Skip to content

Bump cloudstack-go SDK to v2.19.1 - #8

Merged
Pearl1594 merged 1 commit into
mainfrom
update-go-sdk
Sep 24, 2026
Merged

Pearl1594 merged 1 commit into
mainfrom
update-go-sdk

Conversation

@Pearl1594

Copy link
Copy Markdown
Collaborator

Description of changes:

Updates go-sdk to v2.19.1

What was actually run:

go get github.com/apache/cloudstack-go/v2@v2.19.1
go mod tidy
go build ./..., go vet ./..., go test ./...

Testing performed:
$ go test ./...
? github.com/cloudstack/cloudstack-csi-driver/cmd/cloudstack-csi-driver [no test files]
? github.com/cloudstack/cloudstack-csi-driver/cmd/cloudstack-csi-sc-syncer [no test files]
? github.com/cloudstack/cloudstack-csi-driver/pkg/cloud [no test files]
? github.com/cloudstack/cloudstack-csi-driver/pkg/cloud/fake [no test files]
ok github.com/cloudstack/cloudstack-csi-driver/pkg/driver (cached)
? github.com/cloudstack/cloudstack-csi-driver/pkg/mount [no test files]
ok github.com/cloudstack/cloudstack-csi-driver/pkg/syncer (cached)
ok github.com/cloudstack/cloudstack-csi-driver/pkg/util (cached)

@Pearl1594

Copy link
Copy Markdown
Collaborator Author

@elivlo can you please review this. Thanks.

@elivlo

elivlo commented Sep 22, 2026

Copy link
Copy Markdown

@elivlo can you please review this. Thanks.

Of course, I will do that! I have completely missed the your mention :/

@elivlo

elivlo commented Sep 24, 2026

Copy link
Copy Markdown

@Pearl1594 I rebased this branch locally and ran the CSI in my local cluster. It is working just fine!

LGTM

@Damans227 Damans227 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

clgtm

@Pearl1594
Pearl1594 merged commit 206ccb5 into main Sep 24, 2026
2 checks passed
@Pearl1594
Pearl1594 deleted the update-go-sdk branch September 24, 2026 15:51
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.

3 participants