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

Add helm chart release action #181

Merged
merged 2 commits into from
Sep 13, 2024
Merged

Add helm chart release action #181

merged 2 commits into from
Sep 13, 2024

Conversation

HaimKortovich
Copy link

Adds upon PR #117 which addressed issue #116.
This would provide a helm chart repo facility for folks using GitOps utilities.

@seriouz
Copy link

seriouz commented Sep 9, 2024

@robford123 @Regenhardt I think we should create an extra project for the Helm Chart GitHub Pages. I don't like the idea of using the bagetter/BaGetter GitHub pages for the Helm releases.

@HaimKortovich
Copy link
Author

I've seen other projects doing the same since it won't be visible to the user since it's going to be in another path

@Regenhardt
Copy link

I think the existing website would fit, as it's hosted under the general bagetter.com, which can hold both docs and other things (even a demo, should we ever go there).

Although I could also imagine a subdomain like helm.bagetter.com or something. Not sure if this helm repository is conceptually big enough for its own subdomain.

@seriouz
Copy link

seriouz commented Sep 9, 2024

We could just try how it afffects the main site. But I have literally zero knowledge about Helm nor I could test it.

@HaimKortovich
Copy link
Author

I really need this for my k8s cluster, can we get this reviewed?

Copy link

@robford123 robford123 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hopefully this will publish the packaged chart(s) to a separate repo

@robford123 robford123 merged commit 087ca20 into bagetter:main Sep 13, 2024
@robford123
Copy link

@Regenhardt @seriouz @HaimKortovich

I made a few updates to the PR to hopefully make it publish to a different repo.

Created a draft release - need others to decide if/when we're ok to release and test this publishing out. It will probably still need more configuration on the github pages settings to make it functional.

https://github.com/bagetter/BaGetter/releases/tag/untagged-6898337acf0a91fb9ede

@Regenhardt
Copy link

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

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

Successfully merging this pull request may close these issues.

5 participants