Skip to content
This repository was archived by the owner on Apr 4, 2024. It is now read-only.

Helm charts - Fix Bitnami dependency#1616

Merged
bors[bot] merged 1 commit into
bors-ng:masterfrom
JulianPasquale:fix-helm-dependencies
Mar 6, 2023
Merged

Helm charts - Fix Bitnami dependency#1616
bors[bot] merged 1 commit into
bors-ng:masterfrom
JulianPasquale:fix-helm-dependencies

Conversation

@JulianPasquale

Copy link
Copy Markdown
Contributor

There is an open issue in Bitnami that won't let us build the chart dependencies.

Reproduction steps

  1. Go to deploy/charts/bors-ng on the master branch of the bors-ng repo
  2. Add the Bitnami repo
    • helm repo add bitnami https://charts.bitnami.com/bitnami
  3. Try to build the dependencies
    • helm dependency build

You'll see an error like this:
image

Test solution

Pre: Run helm repo remove bitnami to avoid name collision.

  1. Go to deploy/charts/bors-ng on the fix-helm-dependencies branch of the fork repo.
  2. Add the Bitnami repo
    • helm repo add bitnami https://raw.githubusercontent.com/bitnami/charts/e0169502c14943782da50fc1977483fecd6912b5/bitnami
  3. Try to build the dependencies
    • helm dependency build

Update completes without errors
image

I'm not a Helm expert so any feedback is welcome 🙂 .

@notriddle

Copy link
Copy Markdown
Member

bors r+

Okay, that'll do, at least for now.

@bors

bors Bot commented Mar 6, 2023

Copy link
Copy Markdown
Contributor

Build succeeded:

@bors bors Bot merged commit 9b8ae39 into bors-ng:master Mar 6, 2023
notriddle added a commit to bors-ng/bors-ng.github.io that referenced this pull request Apr 1, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants