Replace bitnami images #1248
Replace bitnami images #1248
Conversation
longshuicy
requested review from
Copilot and
lmarini
and removed request for
lmarini
There was a problem hiding this comment.
Pull Request Overview
This PR migrates from the deprecated Bitnami Helm charts repository to the Bitnami legacy archive repository and updates the chart version and dependencies.
- Updates all Bitnami repository URLs from
https://charts.bitnami.com/bitnamito the archive location - Bumps the chart version from 1.9.2 to 1.9.3
- Updates the Elasticsearch dependency from version 19.17.6 to 19.21.2
Reviewed Changes
Copilot reviewed 3 out of 4 changed files in this pull request and generated 5 comments.
| File | Description |
|---|---|
| deployments/kubernetes/charts/clowder2/README.md | Updates repository URL and helm command in documentation |
| deployments/kubernetes/charts/clowder2/Chart.yaml | Updates dependency repository URLs and increments chart version |
| deployments/kubernetes/charts/clowder2/Chart.lock | Reflects updated repository URLs and dependency versions |
| deployments/kubernetes/charts/README.md | Updates repository URL and helm command in documentation |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| - name: minio | ||
| version: ~12 | ||
| repository: https://charts.bitnami.com/bitnami | ||
| repository: https://raw.githubusercontent.com/bitnami/charts/archive-full-index/bitnami/ |
There was a problem hiding this comment.
Inconsistency between the repository URL in the README and Chart.yaml. The README instructions use https://raw.githubusercontent.com/bitnami/charts/archive-full-index/bitnami (without trailing slash), but the Chart.yaml dependencies use https://raw.githubusercontent.com/bitnami/charts/archive-full-index/bitnami/ (with trailing slash). These should match to ensure the helm repo add command aligns with the repository URLs referenced in the dependencies.
| - name: rabbitmq | ||
| version: ~10 | ||
| repository: https://charts.bitnami.com/bitnami | ||
| repository: https://raw.githubusercontent.com/bitnami/charts/archive-full-index/bitnami/ |
There was a problem hiding this comment.
Inconsistency between the repository URL in the README and Chart.yaml. The README instructions use https://raw.githubusercontent.com/bitnami/charts/archive-full-index/bitnami (without trailing slash), but the Chart.yaml dependencies use https://raw.githubusercontent.com/bitnami/charts/archive-full-index/bitnami/ (with trailing slash). These should match to ensure the helm repo add command aligns with the repository URLs referenced in the dependencies.
| - name: elasticsearch | ||
| version: ~19 | ||
| repository: https://charts.bitnami.com/bitnami | ||
| repository: https://raw.githubusercontent.com/bitnami/charts/archive-full-index/bitnami/ |
There was a problem hiding this comment.
Inconsistency between the repository URL in the README and Chart.yaml. The README instructions use https://raw.githubusercontent.com/bitnami/charts/archive-full-index/bitnami (without trailing slash), but the Chart.yaml dependencies use https://raw.githubusercontent.com/bitnami/charts/archive-full-index/bitnami/ (with trailing slash). These should match to ensure the helm repo add command aligns with the repository URLs referenced in the dependencies.
| - name: keycloak | ||
| version: ~13 | ||
| repository: https://charts.bitnami.com/bitnami | ||
| repository: https://raw.githubusercontent.com/bitnami/charts/archive-full-index/bitnami/ |
There was a problem hiding this comment.
Inconsistency between the repository URL in the README and Chart.yaml. The README instructions use https://raw.githubusercontent.com/bitnami/charts/archive-full-index/bitnami (without trailing slash), but the Chart.yaml dependencies use https://raw.githubusercontent.com/bitnami/charts/archive-full-index/bitnami/ (with trailing slash). These should match to ensure the helm repo add command aligns with the repository URLs referenced in the dependencies.
| - name: mongodb | ||
| version: ~12 | ||
| repository: https://charts.bitnami.com/bitnami | ||
| repository: https://raw.githubusercontent.com/bitnami/charts/archive-full-index/bitnami/ |
There was a problem hiding this comment.
Inconsistency between the repository URL in the README and Chart.yaml. The README instructions use https://raw.githubusercontent.com/bitnami/charts/archive-full-index/bitnami (without trailing slash), but the Chart.yaml dependencies use https://raw.githubusercontent.com/bitnami/charts/archive-full-index/bitnami/ (with trailing slash). These should match to ensure the helm repo add command aligns with the repository URLs referenced in the dependencies.
Replace bitnami with bitnami legacy and update the helm version