Skip to content

Conversation

tom-rm-meyer-ISST
Copy link
Contributor

@tom-rm-meyer-ISST tom-rm-meyer-ISST commented Aug 26, 2025

Description

Workaround to keep the latest chart running for R25.09.

Bitnami changed their container image availability to provide versioned and security hardened images only for subscribed companies.

We follow this immediate recommendation and:

  • switch image to bitnamilegacy images
  • add WARNING into migration notes

Pre-review checks

Please ensure to do as many of the following checks as possible, before asking for committer review:

  • DEPENDENCIES are up-to-date. Dash license tool. Committers can open IP issues for restricted libs.
  • Copyright and license header are present on all affected files
  • If helm chart has been changed, the chart version has been bumped to either next major, minor or patch level (compared to released chart).

@tom-rm-meyer-ISST tom-rm-meyer-ISST marked this pull request as ready for review August 26, 2025 16:10
Copy link
Member

@ReneSchroederLJ ReneSchroederLJ left a comment

Choose a reason for hiding this comment

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

Thank you for your quick work. The changes look good. Please update the changelog

@tom-rm-meyer-ISST
Copy link
Contributor Author

Thank you for your quick work. The changes look good. Please update the changelog

Right now we don't have a CHANGELOG.md for the chart. I could start one right now but would only include the 4.1.1 change made here.

@ReneSchroederLJ
Copy link
Member

Thank you for your quick work. The changes look good. Please update the changelog

Right now we don't have a CHANGELOG.md for the chart. I could start one right now but would only include the 4.1.1 change made here.

I don't think it's necessary to start a changelog now

Copy link
Member

@ReneSchroederLJ ReneSchroederLJ left a comment

Choose a reason for hiding this comment

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

LGTM thank you

Copy link
Member

@stephanbcbauer stephanbcbauer left a comment

Choose a reason for hiding this comment

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

LGTM

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.

3 participants