-
Notifications
You must be signed in to change notification settings - Fork 716
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
[dev-v2.8] rancher-vsphere-csi add 103.1.1+up3.1.2-rancher4 #3804
[dev-v2.8] rancher-vsphere-csi add 103.1.1+up3.1.2-rancher4 #3804
Conversation
This comment was marked as outdated.
This comment was marked as outdated.
c7a2c93
to
17b5d44
Compare
This comment was marked as outdated.
This comment was marked as outdated.
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.
Should this file not exist?
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.
Good catch!
17b5d44
to
1c45d8f
Compare
This comment was marked as outdated.
This comment was marked as outdated.
1c45d8f
to
5252292
Compare
Validation steps
Ex:- longhorn-controller: repository: rancher/hardened-sriov-cni tag: v2.6.3-build20230913
|
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 🚀
Change addressed, approved by another member of hostbusters
Issue: rancher/rancher#45093
Problem
The vSphere CSI chart has updated the default value of
csiAuthCheck
Solution
Bump the commit used in the vSphere CSI chart to introduce the updated default value
Testing
Testing has been done in the scope of rancher/rancher#45093 and the default value change has been verified to work as a part of the rke2 embedded chart tesitng
Engineering Testing
Manual Testing
Automated Testing
QA Testing Considerations
Regressions Considerations
Backporting considerations