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

Allow to mutate PVCTemplate of Advanced StatefulSet & CloneSet #1118

Merged
merged 1 commit into from
Dec 5, 2022

Conversation

veophi
Copy link
Member

@veophi veophi commented Nov 8, 2022

Signed-off-by: mingzhou.swx [email protected]

Ⅰ. Describe what this PR does

Allow to mutate PVCTemplate of Advanced StatefulSet & CloneSet.

@kruise-bot kruise-bot added the size/L size/L: 100-499 label Nov 8, 2022
@veophi veophi force-pushed the allow-mutate-pvc-template branch 2 times, most recently from a4bb531 to d4f1870 Compare November 9, 2022 04:09
@veophi veophi changed the title Allow to mutate PVCTemplate of Advanced StatefulSet Allow to mutate PVCTemplate of Advanced StatefulSet & CloneSet Nov 9, 2022
@veophi veophi force-pushed the allow-mutate-pvc-template branch from d4f1870 to e1d1457 Compare November 16, 2022 11:33
@willise
Copy link
Contributor

willise commented Nov 30, 2022

@veophi I have a question that if controller should update controllerrevision to record the pvc template's change.

@veophi
Copy link
Member Author

veophi commented Dec 5, 2022

@willise These actions are already in our planning, but this pr only allow users modify the pvcTemplate manually.

@furykerry
Copy link
Member

/lgtm

@furykerry
Copy link
Member

/approve

@kruise-bot
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: furykerry

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@kruise-bot kruise-bot merged commit 53fb7e1 into openkruise:master Dec 5, 2022
@willise
Copy link
Contributor

willise commented Jan 4, 2023

@willise These actions are already in our planning, but this pr only allow users modify the pvcTemplate manually.

@veophi Is there any progress?

@zmberg zmberg added this to the v1.4 milestone Feb 14, 2023
@zmberg zmberg added the kind/enhancement New feature or request label Feb 14, 2023
ppbits pushed a commit to ppbits/kruise that referenced this pull request Apr 4, 2024
@HURUIZHE
Copy link

HURUIZHE commented Jun 24, 2024

@willise These actions are already in our planning, but this pr only allow users modify the pvcTemplate manually.

@veophi Is there any progress?

+1. I really need this feature. Now I can't rollback when I modified the cloneSet PVCTemplate @veophi @willise

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved kind/enhancement New feature or request lgtm size/L size/L: 100-499
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants