-
Notifications
You must be signed in to change notification settings - Fork 455
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
feat: add support for vlcm (offline software depots, cluster images) #2143
Conversation
Dependent on the next release of |
ff8bd8c
to
c55ca4a
Compare
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.
Hi Stoyan 👋🏻 -
Could you add the required documenation to the /website/docs/...
directories for the new resource and data source?
I'll get a PR in for updating vmware/govmomi
to v0.36.0 that released today with your upstream changes and then rebase this PR.
Ryan
Thanks for taking care of the govmomi bump. I will add docs for the new entities |
Bump `github.com/vmware/govmomi` from 0.35.0 to 0.36.0. Ref: #2143 Signed-off-by: Ryan Johnson <ryan.johnson@broadcom.com>
Bump `github.com/vmware/govmomi` from 0.35.0 to 0.36.0. Ref: #2143 Signed-off-by: Ryan Johnson <ryan.johnson@broadcom.com>
Bump `github.com/vmware/govmomi` from 0.35.0 to 0.36.0. Ref: #2143 Signed-off-by: Ryan Johnson <ryan.johnson@broadcom.com>
Signed-off-by: Stoyan Zhelyazkov <stoyan.zhelyazkov@broadcom.com>
Signed-off-by: Stoyan Zhelyazkov <stoyan.zhelyazkov@broadcom.com>
Signed-off-by: Stoyan Zhelyazkov <stoyan.zhelyazkov@broadcom.com>
Signed-off-by: Stoyan Zhelyazkov <stoyan.zhelyazkov@broadcom.com>
Updates `CHANGELOG.md` Signed-off-by: Ryan Johnson <ryan.johnson@broadcom.com>
c55ca4a
to
5ec4e70
Compare
Rebased but the CI is failing: --- FAIL: TestProvider (0.00s)
provider_test.go:25: err: resource vsphere_offline_software_depot: key: ValidateFunc is for validating user input, there's nothing to validate on computed-only field
FAIL |
Silly me, I added validations on my computed properties |
Signed-off-by: Stoyan Zhelyazkov <stoyan.zhelyazkov@broadcom.com>
Signed-off-by: Stoyan Zhelyazkov <stoyan.zhelyazkov@broadcom.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.
LGTM!
I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further. |
Description
The goal of this PR is to provide users with the ability to configure custom images for clusters using vLCM.
To achieve that I am adding
Only depots with source type "PULL" are supported. This is intentional and aims to discourage the use of the deprecated VUM functionality.
It allows users to select a base image and a list of custom components from a depot.
Configuring this property for the first time also enables vLCM on the cluster
Acceptance tests
Output of make test
Ran the following acceptance tests:
Release Note
Release note for CHANGELOG:
References
Closes #2142