Skip to content

Conversation

@Nikhil-Ladha
Copy link
Contributor

We should continue to cleanup the volume info like the omap data, mappings from the group if the image is not part of the goup anymore.

we should continue to cleanup the volume info like the
omap data, mappings from the group if the image is not
part of the goup anymore.

Signed-off-by: Nikhil-Ladha <[email protected]>
Copy link
Contributor

@Rakshith-R Rakshith-R left a comment

Choose a reason for hiding this comment

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

LGTM

@Rakshith-R Rakshith-R requested a review from a team April 1, 2025 08:22
@nixpanic nixpanic changed the title rbd: clenaup volume info from group if the image is not part of group anymore rbd: cleanup volume info from group if the image is not part of group anymore Apr 1, 2025
@nixpanic
Copy link
Member

nixpanic commented Apr 1, 2025

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

@nixpanic
Copy link
Member

nixpanic commented Apr 1, 2025

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

@nixpanic
Copy link
Member

nixpanic commented Apr 1, 2025

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

@nixpanic
Copy link
Member

nixpanic commented Apr 1, 2025

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

@nixpanic
Copy link
Member

nixpanic commented Apr 1, 2025

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

@nixpanic
Copy link
Member

nixpanic commented Apr 1, 2025

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

@nixpanic
Copy link
Member

nixpanic commented Apr 1, 2025

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

@nixpanic
Copy link
Member

nixpanic commented Apr 1, 2025

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

@nixpanic
Copy link
Member

nixpanic commented Apr 1, 2025

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

@nixpanic
Copy link
Member

nixpanic commented Apr 1, 2025

/test ci/centos/upgrade-tests

@nixpanic
Copy link
Member

nixpanic commented Apr 1, 2025

/retest ci/centos/mini-e2e-helm/k8s-1.32

@nixpanic
Copy link
Member

nixpanic commented Apr 1, 2025

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

@nixpanic
Copy link
Member

nixpanic commented Apr 1, 2025

@nixpanic
Copy link
Member

nixpanic commented Apr 1, 2025

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

Failed with:

FAILED] backend images not matching kubernetes resource count,image count 1 kubernetes resource count 0
  backend image Info:
   [csi-vol-44565caa-7b43-444a-b3db-7afeacf63e38]
   images information and status Pool: replicapool, Image: csi-vol-44565caa-7b43-444a-b3db-7afeacf63e38, Info: {"name":"csi-vol-44565caa-7b43-444a-b3db-7afeacf63e38","id":"1682f6ed7226","size":1073741824,"objects":256,"order":22,"object_size":4194304,"snapshot_count":0,"block_name_prefix":"rbd_data.1682f6ed7226","format":2,"features":["layering"],"op_features":[],"flags":[],"create_timestamp":"Tue Apr  1 09:36:59 2025","access_timestamp":"Tue Apr  1 09:36:59 2025","modify_timestamp":"Tue Apr  1 09:36:59 2025"}

logs

@mergify mergify bot merged commit 23fce43 into ceph:devel Apr 1, 2025
26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component/rbd Issues related to RBD

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants