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

Helm instructions did not work for me #1625

Open
aisbaa opened this issue Feb 7, 2024 · 3 comments
Open

Helm instructions did not work for me #1625

aisbaa opened this issue Feb 7, 2024 · 3 comments

Comments

@aisbaa
Copy link

aisbaa commented Feb 7, 2024

Unable to use helm chart, I've tried instructions in the readme, was unsuccessful:

> CHART_VER="0.2.2"
> helm install --dry-run \
  -n bazel-buildfarm \
  --create-namespace \
  bazel-buildfarm \
  "https://github.com/bazelbuild/bazel-buildfarm/releases/download/helm%2F${CHART_VER}/buildfarm-${CHART_VER}.tgz"
Error: INSTALLATION FAILED: failed to fetch https://github.com/bazelbuild/bazel-buildfarm/releases/download/helm%2F0.2.2/buildfarm-0.2.2.tgz : 404 Not Found

Tried to manually download it, but it also resulted in 404:

> wget https://github.com/bazelbuild/bazel-buildfarm/releases/download/helm%2F0.2.2/buildfarm-0.2.2.tgz
--2024-02-07 14:40:30--  https://github.com/bazelbuild/bazel-buildfarm/releases/download/helm%2F0.2.2/buildfarm-0.2.2.tgz
Resolving github.com (github.com)... 140.82.121.3
Connecting to github.com (github.com)|140.82.121.3|:443... connected.
HTTP request sent, awaiting response... 404 Not Found
2024-02-07 14:40:31 ERROR 404: Not Found.
@werkt
Copy link
Member

werkt commented Feb 7, 2024

@andrewrothstein the instructions seem to have gotten muddled since your original addition - both are using the same version env var. Can you put up a PR to clear this up?

We could also put notes to do a helm install in the release notes with the most current version and leave the decaying version identifier out of the README.md entirely.

@andrewrothstein
Copy link
Collaborator

@werkt can u drop a tag named ‘helm/0.2.2’ on the main branch? that should bundle the helm chart into released artifacts that these instructions reference.

@werkt
Copy link
Member

werkt commented Feb 15, 2024

I think this means that since landing your most recent change we should tag HEAD at helm/0.2.3?

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

No branches or pull requests

3 participants