search

bazelbuild / bazel-buildfarm

Bazel remote caching and execution service
https://bazel.build
Apache License 2.0
635 stars 199 forks source link

Helm instructions did not work for me #1625

Open aisbaa opened 4 months ago

aisbaa commented 4 months ago

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 commented 4 months ago

@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 commented 4 months ago

@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 commented 4 months ago

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