fermyon / spin-containerd-shim-installer

Helm chart that installs the containerd runtime shim for Spin on Kubernetes
Apache License 2.0
4 stars 4 forks source link

helm install issue #10

Open michelleN opened 1 year ago

michelleN commented 1 year ago

I'm trying to install via helm and this is the error I am getting:

helm install spin-containerd-shim-installer oci://ghcr.io/fermyon/charts/spin-containerd-shim-installer --version 0.1.0

Error: INSTALLATION FAILED: failed to download "oci://ghcr.io/fermyon/charts/spin-containerd-shim-installer" at version "0.1.0"

Is this install line still up to date? If I change versions, I get the same error message.

vdice commented 12 months ago

Interesting, I was able to install both 0.1.0 and the latest 0.9.2 versions. I wonder if there was an intermittent issue in ghcr.io? Note I did have to set HELM_EXPERIMENTAL_OCI=1 in my shell. This happened to be on a kind cluster. Can you try again @michelleN?

Good point, we should update the README.md to install the latest 0.9.2 version (the version matrix needs a few updates as well...)

$ helm install spin-containerd-shim-installer oci://ghcr.io/fermyon/charts/spin-containerd-shim-installer --version 0.1.0
NAME: spin-containerd-shim-installer
LAST DEPLOYED: Mon Nov 27 09:56:33 2023
NAMESPACE: default
STATUS: deployed
REVISION: 1
TEST SUITE: None
NOTES:
Thank you for installing spin-containerd-shim-installer.

Your release is named spin-containerd-shim-installer.

To read the logs use:

  $ kubectl logs daemonset/spin-containerd-shim-installer -c installer --timestamps=true --prefix=true -f

$ helm ls
NAME                            NAMESPACE   REVISION    UPDATED                                 STATUS      CHART                                   APP VERSION
spin-containerd-shim-installer  default     1           2023-11-27 09:56:33.650069 -0700 MST    deployed    spin-containerd-shim-installer-0.1.0    0.6.0
$ helm install spin-containerd-shim-installer oci://ghcr.io/fermyon/charts/spin-containerd-shim-installer --version 0.9.2
NAME: spin-containerd-shim-installer
...

$ helm ls
NAME                            NAMESPACE   REVISION    UPDATED                                 STATUS      CHART                                   APP VERSION
spin-containerd-shim-installer  default     1           2023-11-27 09:57:28.339256 -0700 MST    deployed    spin-containerd-shim-installer-0.9.2    0.9.2
michelleN commented 12 months ago

I've set HELM_EXPERIMENTAL_OCI=1 and am still getting the error.

michelleN commented 12 months ago

I upgraded helm and tried again and got a slightly different error:

helm install spin-containerd-shim-installer oci://ghcr.io/fermyon/charts/spin-containerd-shim-installer --version 0.9.2
Error: INSTALLATION FAILED: failed to authorize: failed to fetch oauth token: unexpected status from GET request to https://ghcr.io/token?scope=repository%3Afermyon%2Fcharts%2Fspin-containerd-shim-installer%3Apull&service=ghcr.io: 403 Forbidden
michelleN commented 12 months ago

I ended up cloning the repo and installing what is in the chart directory locally but that points to 0.7.0 versions of the necessary images. Do I need to update those to 0.10.0 for Spin 2.0?

vdice commented 12 months ago

@michelleN one thing to check: do you happen to have an older/invalid credential for the ghcr.io registry on your system? That might lead to weird/auth errors. I think I've been hit with this... even attempting to pull public images (or charts) returns errors.

Yes, it looks like updates are needed to this project to bump to the 0.10.0 shim version for Spin 2.0. Other updates include naming changes... I see the -v1 suffix has been dropped from resource names, etc in the shim project. Oh, I see updates in release branchs (eg https://github.com/fermyon/spin-containerd-shim-installer/commit/69d9c8f83a5955a44717fdd679ada3c134252246 for the recent v0.9.2 tag), but these changes should also be on main, as you've pointed out. cc @jpflueger

jpflueger commented 12 months ago

Spot on, I've been pushing up some changes for the latest releases but I do need to merge those changes back into main. I have a bunch of scattered changes at the moment but I'll try to get them cleaned up here over the next week.

jpflueger commented 11 months ago

@michelleN I did update the readme and the main branch with v0.9.x releases. I've also pushed up a release for v0.10.0 (but haven't merged that into main yet) so you should be able to install with this command:

helm install spin-containerd-shim-installer oci://ghcr.io/fermyon/charts/spin-containerd-shim-installer --version 0.10.0