[GitHub] [cloudstack] DaanHoogland commented on issue #3839: FEATURE-3823: kvm agent hooks

[GitBox <gi...@apache.org>]

DaanHoogland commented on issue #3839: FEATURE-3823: kvm agent hooks
URL: https://github.com/apache/cloudstack/pull/3839#issuecomment-588241546
 
 
   @bwsw I'm happy to put some effort into getting this merged. Yur code looks neat, even though I don't undertand it fully yet. A few things putting me off in your latest reply. Please have patience with me?
   Firstly:
   
   >     My PR is for public cloud operators, who percept CS as a constructor;
   
   I imagine you mean *this* PR, right?
   
   >     Your PR is for private cloud operators who percept CS as a final system.
   
   What PR is that? Not "who's!", can you please refer to bits of code and not people? I can see some frustration in this comment and I have been involved in this conflict between private clouds an public clouds a lot. My question is genuine. On a side note, It's all apache's PRs ;)
   
   > Case1. In your PR there is no way to select any of the available NVME drives installed in the platform during VM deployment and attach it to the VM using some advanced logic.
   
   (what PR?) can you add/explain how you are adding that info? I would expect some kind of dialog going back to the MS to provide the user with a list of available resources but can not see that in the code.
   
   > Case2. when VM is deployed, for every CS account, I create VXLAN unmanaged by CS. I create a bridge and would like to add a NIC into configuration dynamically. 
   
   So how do you guarantee the same vxlan's aren't also used by ACS? (And also, parallel to case 1 above, how is the vxlan id fed back into ACS)
   
   > Case 3. I would like to find any of available Quadro RTX 4000 GPU and pass it into VM.
   
   In this case you do not want the user to add it, right? but have it automagically added?
   
   > Case 4. Read Sakari Poussa's request from mailing list:
   
   Can you add a reference to the mail thread in lists.apache.org please? The snippet contains references to context.
   
   @svenvogel is asking for documentation above. Are you already using this in production? Can you add a PR to https://github.com/apache/cloudstack-documentation, please?
   
   @weizhouapache , @GabrielBrascher , @kiwiflyer thoughts/reviews?

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services