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

Build Helm chart release and serve it using github pages #432

Closed
wants to merge 1 commit into from

Conversation

prune998
Copy link
Contributor

One-line summary

Issue : #1234 (only if appropriate)

Description

This PR is adding a github action to build a package out of the Helm chart when it is updated (version changed in the Chart.yaml file).
The release will be added into the Chart Repository index that is served by Github Pages.

Requirement:

  • Github pages is active and serving the content from the gh-pages branch

Side notes: I also have a README.md file to upload into the gh-pages branch when created

Types of Changes

What types of changes does your code introduce? Keep the ones that apply:

  • Refactor/improvements
  • Documentation / non-code

@mikkeloscar
Copy link
Contributor

@prune998 I have now setup a gh-pages branch and defined the output folder as charts. Can you update this PR to use charts instead of docs, then we can try it out.

@prune998
Copy link
Contributor Author

looking into it, thanks @mikkeloscar

@prune998
Copy link
Contributor Author

the docs folder is the one on master, where the chart files are. So I guess we should be good to go.
Sadly, I can't test it until we merge this PR...

@prune998
Copy link
Contributor Author

PR for the gh-pages branch: #433

@prune998 prune998 force-pushed the master branch 2 times, most recently from 4ddd7d5 to 7ef969e Compare May 20, 2022 12:48
Signed-off-by: Sebastien THOMAS <sebastien.thomas@wunderkind.co>
@sppwf
Copy link

sppwf commented Mar 9, 2023

Hi Guys,

What about this PR?

Thanks
Sergiu Plotnicu

@mikkeloscar
Copy link
Contributor

Closing in favor of #539

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.

None yet

3 participants