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 chart deploy failing #198

Open
Regenhardt opened this issue Nov 27, 2024 · 10 comments
Open

Helm chart deploy failing #198

Regenhardt opened this issue Nov 27, 2024 · 10 comments

Comments

@Regenhardt
Copy link

Not sure why it's failing, not enough experience with both helm charts and github actions. @HaimKortovich could you check out why https://github.com/bagetter/BaGetter/actions/runs/11783976064/job/32822284205 failed?

@Regenhardt
Copy link
Author

@robford123 you know your way around this stuff too, maybe you have an idea?

@robford123
Copy link

Try it now - the branch needed to be gh-pages but I did find an organization level setting for the github token that didn't allow writes, just reads.

@HaimKortovich
Copy link

@Regenhardt Build is returning status code 403 so the gh token probably does not have write access

@Regenhardt
Copy link
Author

Any way to deploy to that repo on the main branch instead of gh-pages? IMO the gh-pages branch as a deployment source is just bad design. Feels like TFS all over again (although it is of course possible it was only my former company using TFS branching like that). A branch should be a version.

So main branch should be the only branch there, possibly readme file to explain it's a helm chart docs repo, /docs directory for the artifact. What do you think?

@Regenhardt
Copy link
Author

Also it's still breaking with the 403: Failed job

@Regenhardt
Copy link
Author

Does the token work cross-repo? Maybe we have to make a deploy key there and inject that here?

@robford123
Copy link

Try adding

permissions:
contents: write

to the job that publishes the helm - just like how the zip file publish job does

@HaimKortovich
Copy link

helm repository is just an index.yaml with tarballs so as @robford123 try adding permissions to write content @Regenhardt

@HaimKortovich
Copy link

@Regenhardt any update on this?

@Regenhardt
Copy link
Author

Will try after the next release as to not scramble the changelog again. Will make a release soon, I would like to merge some open PRs before.

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