Skip to content
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

Vendor update on removing remove eng.vmware.com #662

Merged

Conversation

sakthisunda
Copy link
Contributor

@sakthisunda sakthisunda commented Oct 7, 2024

Description

  • go-vcloud-director version bump that has update on removing eng.vmware.com occcurences and related change
  • make build tested
  • completed CRUD using UI

@arunmk @rocknes @lzichong


This change is Reviewable

Signed-off-by: Sakthi Sundaram <[email protected]>
go.mod Outdated
github.com/vmware/cloud-provider-for-cloud-director v0.0.0-20240725204714-a0a0e916a5ed
github.com/vmware/go-vcloud-director/v2 v2.21.0
github.com/vmware/go-vcloud-director/v2 v2.26.0-alpha.6
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

v2.26.0 is already released right? can't we use a released version instead of an alpha version?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I second this. We can use the 2.26.0 version that was released fairly recently.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

updated to release version. Thanks for bringing this up.

Copy link
Collaborator

@arunmk arunmk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM after using v2.26 released version for go-vcloud-director

@sakthisunda sakthisunda merged commit e6ae1f5 into vmware:1.3.z Oct 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants