[bitnami/keycloak] fix default image repositories to bitnamilegacy for pullable tags#36559
[bitnami/keycloak] fix default image repositories to bitnamilegacy for pullable tags#36559LordRobinCbz wants to merge 1 commit into
Conversation
…r pullable tags Update chart defaults and README so helm install works without manual image overrides after the Bitnami catalog migration (bitnami#35164). Signed-off-by: lordrobincbz <robincrombez.pro@gmail.com>
|
Hello @javsalgar , First of all, I am sorry to tell you that I didn't ran correctly the readme-generator-for-helm (need specific version of NodeJs, CLI tool only avaialble on UNIX, custom Windows installation that I have doesn't allow me to do it correctly). I guess some files need to be edited to (chart.lock ?) But I am not sure, can you tell me ? This PR is not mandatory for my current works, because I can override used images, so take your time :) Many thanks |
|
Thank you for the contribution and for the detailed description! We appreciate you taking the time to investigate this and put together a fix. However, we won't be changing the default image repositories to The reason is that If you need to use the legacy images (e.g. to keep an existing deployment running on older tags), you can override the repository at install/upgrade time. We understand this is inconvenient, but keeping these overrides explicit ensures users are aware they are opting into archived images. |
|
Hello @carrodher No problem, I understand your reasoning. That said, because of the issue described in my report, the current Keycloak Helm chart cannot be deployed successfully without several manual overrides. From my understanding, the current approach means keeping a chart that does not work out of the box rather than relying on the legacy images. I understand the concerns regarding archived and unmaintained images, and I agree that they should not become the default for new installations. However, would it be possible to publish the missing tags on Docker Hub (or provide another compatible solution) so that the chart can function correctly without requiring users to rewrite multiple image definitions? From a user's perspective, someone trying to deploy the chart will immediately run into this issue. This may lead them to conclude that the chart is broken and potentially choose an alternative solution instead. That's why I'm wondering whether leaving the chart in a non-functional state is preferable to providing a fix that restores a working out-of-the-box experience. |
|
Hi @LordRobinCbz, those are fair points and we appreciate the thoughtful follow-up. You are right that the chart does not work out of the box without manual overrides, and we understand that this creates a poor first-time experience. However, publishing new versioned tags to The supported path for a fully functional, out-of-the-box Helm chart experience is a Bitnami Secure Images (BSI) subscription. BSI provides:
For users who need a temporary workaround without a subscription, the manual |
Description of the change
Updates default container image repositories in the Keycloak chart from
docker.io/bitnami/*todocker.io/bitnamilegacy/*for Keycloak, keycloak-config-cli, and the bundled PostgreSQL subchart.Since the Bitnami catalog migration (#35164), versioned tags referenced by chart
25.2.0return 404 on Docker Hub:docker.io/bitnami/keycloak:26.3.3-debian-12-r0docker.io/bitnami/postgresql:17.6.0-debian-12-r0docker.io/bitnami/keycloak-config-cli:6.4.0-debian-12-r11Equivalent tags exist under
docker.io/bitnamilegacy/*.This PR includes four files:
bitnami/keycloak/Chart.yaml25.2.0→25.2.1; updateannotations.imagesbitnami/keycloak/values.yamlpostgresql.imageblock with@parammetadata; setallowInsecureImages: truebitnami/keycloak/README.mdBenefits
helm install/helm upgradeworks without manual--set image.repository=bitnamilegacy/...overrides.postgresql.image.*values.Possible drawbacks
bitnamilegacy/*images are archived and receive no further updates or security patches (same limitation as the documented FAQ workaround).global.security.allowInsecureImages: truedisables built-in image repository verification for non-bitnami/*paths.image.repositoryback tobitnami/*are unaffected.registry-1.docker.io/bitnamichartsare not retroactively updated; consumers must use a new chart revision after merge/publication.Applicable issues
Additional information
Reproduction (before fix)
helm install keycloak oci://registry-1.docker.io/bitnamicharts/keycloak --version 25.2.0Observed kubelet error:
Docker Hub tag check (2026-06-24)
bitnami/*bitnamilegacy/*26.3.3-debian-12-r017.6.0-debian-12-r06.4.0-debian-12-r11Validation (after fix)
Regenerate README (required before opening PR)
Install tool: https://github.com/bitnami/readme-generator-for-helm
Commit message (with DCO)
Checklist
Chart.yamlaccording to semver. This is not necessary when the changes only affect README.md files.README.mdusing readme-generator-for-helm