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

rbd: Enable featuregates for volume expansion recovery #3078

Merged
merged 3 commits into from
May 25, 2022

Conversation

humblec
Copy link
Collaborator

@humblec humblec commented May 4, 2022

The enabled feature gate helps us to recover from volume expansion
failure.

Additional Reference#

Reference below URL for this feature gate details:
https://github.com/kubernetes-csi/external-resizer/

Additional Note for reviewer:

No E2E as its a generic one and not specific to the Ceph CSI driver as like AnnotateFSResize.

Signed-off-by: Humble Chirammal [email protected]

@humblec humblec marked this pull request as draft May 4, 2022 06:22
@mergify mergify bot added the component/rbd Issues related to RBD label May 4, 2022
@humblec humblec force-pushed the handle branch 6 times, most recently from 8ef271e to 60be4a3 Compare May 6, 2022 05:01
@mergify
Copy link
Contributor

mergify bot commented May 9, 2022

This pull request now has conflicts with the target branch. Could you please resolve conflicts and force push the corrected changes? 🙏

@humblec humblec force-pushed the handle branch 2 times, most recently from 8540f32 to cc690f0 Compare May 10, 2022 09:52
@humblec humblec changed the title rbd: Enable featuregates for volume expansion recovery and pvc delete rbd: Enable featuregates for volume expansion recovery May 17, 2022
@humblec humblec marked this pull request as ready for review May 17, 2022 07:47
@humblec humblec requested a review from Madhu-1 May 17, 2022 07:47
Copy link
Collaborator

@Madhu-1 Madhu-1 left a comment

Choose a reason for hiding this comment

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

changes LGTM but @humblec What are the cases the volume expansion can fail in cephcsi?

@humblec
Copy link
Collaborator Author

humblec commented May 23, 2022

changes LGTM but @humblec What are the cases the volume expansion can fail in cephcsi?

it could be anything, more or less, this is a fallback mechanism for the user to come back from an error scenario.

@humblec
Copy link
Collaborator Author

humblec commented May 23, 2022

@Mergifyio rebase

@mergify
Copy link
Contributor

mergify bot commented May 23, 2022

rebase

✅ Branch has been successfully rebased

@humblec humblec force-pushed the handle branch 2 times, most recently from 0e19a11 to 2ac5baa Compare May 23, 2022 10:23
@humblec
Copy link
Collaborator Author

humblec commented May 23, 2022

@Madhu-1 comments addressed, ptal. thanks.

@humblec
Copy link
Collaborator Author

humblec commented May 25, 2022

@Madhu-1 ptal. comments are addressed

humblec added 3 commits May 25, 2022 11:19
The enabled feature gates helps us to recover from volume expansion
failure.

Signed-off-by: Humble Chirammal <[email protected]>
This commit enable the mentioned feature gate which helps to
recover from volume expansion failures.

Signed-off-by: Humble Chirammal <[email protected]>
this commit enable RecoverVolumeExpansionFailure featuregate in
minikube deployment.

Signed-off-by: Humble Chirammal <[email protected]>
@humblec humblec added the ci/retry/e2e Label to retry e2e retesting on approved PR's label May 25, 2022
@humblec
Copy link
Collaborator Author

humblec commented May 25, 2022

@Mergifyio refresh

@mergify
Copy link
Contributor

mergify bot commented May 25, 2022

refresh

✅ Pull request refreshed

@ceph-csi-bot
Copy link
Collaborator

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

@ceph-csi-bot
Copy link
Collaborator

@humblec "ci/centos/mini-e2e-helm/k8s-1.21" test failed. Logs are available at location for debugging

@ceph-csi-bot
Copy link
Collaborator

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

@ceph-csi-bot
Copy link
Collaborator

@humblec "ci/centos/mini-e2e-helm/k8s-1.23" test failed. Logs are available at location for debugging

@ceph-csi-bot
Copy link
Collaborator

@Mergifyio requeue

@mergify
Copy link
Contributor

mergify bot commented May 25, 2022

requeue

✅ The queue state of this pull request has been cleaned. It can be re-embarked automatically

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci/retry/e2e Label to retry e2e retesting on approved PR's component/rbd Issues related to RBD
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants