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

[dev-v2.8] Add Longhorn 1.5.5 chart #3832

Merged
merged 2 commits into from
Apr 30, 2024

Conversation

mantissahz
Copy link
Contributor

@mantissahz mantissahz commented Apr 29, 2024

Release Longhorn v1.5.5 into Rancher 2.8

Longhorn longhorn/longhorn#8356
Rancher rancher/rancher#44761

cc @innobead @PhanLe1010 @khushboo-rancher @roger-ryao

Waiting for Longhorn 1.5.5 images merged into the Rancher image-mirror list.

Copy link

Validation steps

  • Ensure all container images have repository and tag on the same level to ensure that all container images are included in rancher-images.txt which are used by airgap customers.
  Ex:-
    longhorn-controller:
      repository: rancher/hardened-sriov-cni
      tag: v2.6.3-build20230913
  
  • Add a 👍 (thumbs up) reaction to this comment once done. CI won't pass without this reaction to the github-action bot's latest validation comment.
  • Approve the PR to run the CI check.

Copy link
Contributor

@PhanLe1010 PhanLe1010 left a 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.8

Longhorn 8356
Rancher 44761

Signed-off-by: James Lu <james.lu@suse.com>
release Longhorn v1.5.5 into Rancher 2.8

Longhorn 8356
Rancher 44761

Signed-off-by: James Lu <james.lu@suse.com>
Copy link

Validation steps

  • Ensure all container images have repository and tag on the same level to ensure that all container images are included in rancher-images.txt which are used by airgap customers.
  Ex:-
    longhorn-controller:
      repository: rancher/hardened-sriov-cni
      tag: v2.6.3-build20230913
  
  • Add a 👍 (thumbs up) reaction to this comment once done. CI won't pass without this reaction to the github-action bot's latest validation comment.
  • Approve the PR to run the CI check.

@khushboo-rancher
Copy link

Validation - PASSED

Validated with Rancher 2.8.3, k8s - v1.28.8 +rke2r1, installation, uninstallation & upgrade and P0 cases for volume work fine.

Copy link
Contributor

@lucasmlp lucasmlp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🚀

@PhanLe1010 PhanLe1010 merged commit a6ae80e into rancher:dev-v2.8 Apr 30, 2024
9 checks passed
lucasmlp pushed a commit that referenced this pull request May 1, 2024
Signed-off-by: James Lu <james.lu@suse.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants