-
Notifications
You must be signed in to change notification settings - Fork 145
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
fix(setting): do not delete the storage class set in the setting default-longhorn-static-storage-class
#3141
Conversation
@mantissahz Is it ready? |
Not yet, and I'm testing the modification of Phan's suggestions. |
a3ae29b
to
7fca998
Compare
default-longhorn-static-storage-class
7fca998
to
30e3cb1
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Delete the old storage class set in reconciling the `default-longhorn-static-storage-class` setting. Create the storage with a label if the storage class named as the value of `default-longhorn-static-storage-class` setting does not exist. ref: longhorn/longhorn 9391 Signed-off-by: James Lu <james.lu@suse.com>
30e3cb1
to
b71b6c6
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@mergify backport v1.7.x v1.6.x |
✅ Backports have been created
|
Which issue(s) this PR fixes:
Issue # longhorn/longhorn#9391
What this PR does / why we need it:
Delete the old storage class set in reconciling the
default-longhorn-static-storage-class
setting.Create the storage with a label if the storage class named as the value of
default-longhorn-static-storage-class
setting does not exist.Special notes for your reviewer:
Additional documentation or context