-
Notifications
You must be signed in to change notification settings - Fork 112
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add support for vGPU policies #1167
Conversation
…quests return asterisks for the pre shared key Signed-off-by: Adam Jasinski <jasinskia@vmware.com>
Signed-off-by: Adam Jasinski <jasinskia@vmware.com>
Signed-off-by: Adam Jasinski <jasinskia@vmware.com>
Signed-off-by: Adam Jasinski <jasinskia@vmware.com>
Signed-off-by: Adam Jasinski <jasinskia@vmware.com>
Signed-off-by: Adam Jasinski <jasinskia@vmware.com>
Signed-off-by: Adam Jasinski <jasinskia@vmware.com>
Signed-off-by: Adam Jasinski <jasinskia@vmware.com>
Signed-off-by: Adam Jasinski <jasinskia@vmware.com>
Signed-off-by: Adam Jasinski <jasinskia@vmware.com>
Signed-off-by: Adam Jasinski <jasinskia@vmware.com>
Signed-off-by: Adam Jasinski <jasinskia@vmware.com>
Reminder: create changelog entry when we consider the feature complete |
Signed-off-by: Adam Jasinski <jasinskia@vmware.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's nice to finally complete the Compute policy circle with this addition 😄
Quick scan of trivial things/nitpicks, will do a more deep review after that:
Signed-off-by: Adam Jasinski <jasinskia@vmware.com>
Signed-off-by: Adam Jasinski <jasinskia@vmware.com>
Signed-off-by: Adam Jasinski <jasinskia@vmware.com>
Signed-off-by: Adam Jasinski <jasinskia@vmware.com>
Signed-off-by: Adam Jasinski <jasinskia@vmware.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should add 1167-experimental.md
to define this new resource and data source
Signed-off-by: Adam Jasinski <jasinskia@vmware.com>
Signed-off-by: Adam Jasinski <jasinskia@vmware.com>
Signed-off-by: Adam Jasinski <jasinskia@vmware.com>
Signed-off-by: Adam Jasinski <jasinskia@vmware.com>
Signed-off-by: Adam Jasinski <jasinskia@vmware.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am good. There is one caveat that is not related to this PR and SDK needs fixing. Doing it in vmware/go-vcloud-director#636
Signed-off-by: Adam Jasinski <jasinskia@vmware.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for this great feature, @adezxc !
Signed-off-by: Adam Jasinski <jasinskia@vmware.com>
This PR adds some new things:
New resource and datasource:
vcd_vm_vgpu_policy
New datasource:
vcd_vgpu_profile
New field in
vcd_org_vdc
:vm_vgpu_policy_ids
New field in
vcd_resource_pool
datasource:cluster_moref
There is a hack and a questionable decision regarding the field
vcd_vm_vgpu_policy.provider_vdc_scope.cluster_names
: I'm not sure if there can be more than one cluster on which the provider VDC resides, so I have settled with a solution for ease of use: if a user doesn't specify any clusters, the provider tries to do it automatically by fetching the resource pool on which the PVDC is running.