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 install issue #10

Open
michelleN opened this issue Nov 27, 2023 · 7 comments
Open

helm install issue #10

michelleN opened this issue Nov 27, 2023 · 7 comments

Comments

@michelleN
Copy link
Member

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
Copy link
Member

vdice commented Nov 27, 2023

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
Copy link
Member Author

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

@michelleN
Copy link
Member Author

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
Copy link
Member Author

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
Copy link
Member

vdice commented Nov 29, 2023

@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 69d9c8f for the recent v0.9.2 tag), but these changes should also be on main, as you've pointed out. cc @jpflueger

@jpflueger
Copy link
Contributor

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 jpflueger mentioned this issue Dec 20, 2023
@jpflueger
Copy link
Contributor

@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

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