-
Notifications
You must be signed in to change notification settings - Fork 8
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
Correct Powerflex config map name in manifests #149
Conversation
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.
need to discuss this in Moose scrum- will update based on discussions
Note: ConfigMap name in https://github.com/dell/csm-operator/blob/main/samples/storage_csm_powerflex_v230.yaml is vxflexos-config-params and does not match with what is there is 2.4 and 2.5 samples. |
talked about it in scrum- If you look at node.yaml and other files, we are looking for name to be test-vxflexos-xxxx
csm-operator/operatorconfig/driverconfig/powerflex/v2.3.0/driver-config-params.yaml Line 4 in e522e18
|
739ca1d
739ca1d
to
5f5c450
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.
This looks good!
Description
This PR corrects Powerflex config map name in samples and test files
GitHub Issues
List the GitHub issues impacted by this PR:
Checklist:
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Please also list any relevant details for your test configuration