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

update VolumeGroupSnapshot to v1beta1 (backport #5021) #5030

Merged
merged 7 commits into from
Dec 17, 2024

Conversation

mergify[bot]
Copy link
Contributor

@mergify mergify bot commented Dec 17, 2024

Updating the required files to move VGS from alpha to beta API.


This is an automatic backport of pull request #5021 done by Mergify.

updating the snapshot version in
the build.env

Signed-off-by: Madhu Rajanna <[email protected]>
(cherry picked from commit 884c47d)
updating the snapshotter image to 8.2.0
and update the flag to enable vgs
feature.

Signed-off-by: Madhu Rajanna <[email protected]>
(cherry picked from commit 23da6d9)
updating the snapshotter image to 8.2.0
and update the flag to enable vgs
feature.

Signed-off-by: Madhu Rajanna <[email protected]>
(cherry picked from commit 5bfff89)
updating the snapshot client and
test to use v1beta1 client and API.

Signed-off-by: Madhu Rajanna <[email protected]>
(cherry picked from commit a17fef0)
updating the snapshot container ARG
to enable the vgs feature.

Signed-off-by: Madhu Rajanna <[email protected]>
(cherry picked from commit eb66ec8)
updating the yamls of vgs to v1beta1

Signed-off-by: Madhu Rajanna <[email protected]>
(cherry picked from commit 21d639d)
Remove the vsg flag present in the older snapshotter
which is meant only for the Alpha Release and we are
no longer installing Alpha CRD.

Signed-off-by: Madhu Rajanna <[email protected]>
(cherry picked from commit a88f86e)
@mergify mergify bot added the ok-to-test Label to trigger E2E tests label Dec 17, 2024
@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/upgrade-tests-cephfs

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/k8s-e2e-external-storage/1.29

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/upgrade-tests-rbd

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/mini-e2e-helm/k8s-1.29

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/k8s-e2e-external-storage/1.30

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/k8s-e2e-external-storage/1.31

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/mini-e2e-helm/k8s-1.30

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/mini-e2e/k8s-1.29

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/mini-e2e/k8s-1.30

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/mini-e2e-helm/k8s-1.31

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/mini-e2e/k8s-1.31

@ceph-csi-bot ceph-csi-bot removed the ok-to-test Label to trigger E2E tests label Dec 17, 2024
@Madhu-1 Madhu-1 added the ready-to-merge This PR is ready to be merged and it doesn't need second review (backports only) label Dec 17, 2024
@nixpanic
Copy link
Member

/retest ci/centos/mini-e2e/k8s-1.31

@nixpanic
Copy link
Member

/retest ci/centos/mini-e2e/k8s-1.31

failed at /go/src/github.com/ceph/ceph-csi/e2e/rbd.go:437:

[FAILED] backend images not matching kubernetes resource count,image count 1 kubernetes resource count 0
  backend image Info:
   [csi-vol-025c1617-5b09-4079-98c2-ce99303eb098]
   images information and status Pool: replicapool, Image: csi-vol-025c1617-5b09-4079-98c2-ce99303eb098, Info: {"name":"csi-vol-025c1617-5b09-4079-98c2-ce99303eb098","id":"1380ae34b28e","size":1073741824,"objects":256,"order":22,"object_size":4194304,"snapshot_count":0,"block_name_prefix":"rbd_data.1380ae34b28e","format":2,"features":["layering"],"op_features":[],"flags":[],"create_timestamp":"Tue Dec 17 11:27:21 2024","access_timestamp":"Tue Dec 17 11:27:21 2024","modify_timestamp":"Tue Dec 17 11:27:21 2024"}
  , Status: 

logs

@mergify mergify bot merged commit 21e9ace into release-v3.13 Dec 17, 2024
41 checks passed
@mergify mergify bot deleted the mergify/bp/release-v3.13/pr-5021 branch December 17, 2024 14:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-to-merge This PR is ready to be merged and it doesn't need second review (backports only)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants