-
Notifications
You must be signed in to change notification settings - Fork 718
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.7] Add Longhorn 1.5.5 chart #3833
Conversation
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
Could QA help to test @khushboo-rancher ?
release Longhorn v1.5.5 into Rancher 2.7 Longhorn 8356 Rancher 44762 Signed-off-by: James Lu <james.lu@suse.com>
release Longhorn v1.5.5 into Rancher 2.7. Longhorn 8356 Rancher 44762 Signed-off-by: James Lu <james.lu@suse.com>
dd2cfd8
to
6ae159d
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 🚀
@@ -6,31 +6,31 @@ | |||
engine: | |||
- repository: longhornio/longhorn-engine | |||
+ repository: rancher/mirrored-longhornio-longhorn-engine |
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.
Here the images were patched to use the mirrored ones.
@@ -30,25 +30,25 @@ image: | |||
longhorn: | |||
engine: | |||
repository: longhornio/longhorn-engine |
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.
Here the images are not the mirrored ones.
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.
Oh. This one is in exclude
folder which means the whole file will be removed. I think it shouldn't affect anything. WDYT?
Validation - PASSED Validated with Rancher 2.7.12, installation, uninstallation, upgrade and p0 test cases for volume worked fine. I didn't observe any longhornio images in the deployment, all the components are deployed with mirrored images. |
Signed-off-by: James Lu <james.lu@suse.com>
Release Longhorn v1.5.5 into Rancher 2.7
Longhorn longhorn/longhorn#8356
Rancher rancher/rancher#44762
cc @innobead @PhanLe1010 @khushboo-rancher @roger-ryao
Waiting for Longhorn 1.5.5 images merged into the Rancher image-mirror list.