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

Hosting of the Harmony Chart #33

Closed
jfavellar90 opened this issue May 2, 2023 · 3 comments
Closed

Hosting of the Harmony Chart #33

jfavellar90 opened this issue May 2, 2023 · 3 comments
Assignees

Comments

@jfavellar90
Copy link
Contributor

One of the points open in our list of TODOs to make this project usable in pŕoduction is the hosting of the Harmony chart. Currently, to install this chart correctly, we need to clone the whole openedx-k8s-harmony repository and use Helm to install it from the source code. Additionally, we need to push the artifacts of the dependency charts to prevent running extra commands when installing the main chart. This is not ideal from an operator's perspective.

We should stick to the standard and host the chart in a Helm repository to make its installation more intuitive:

helm repo add openedx-harmony <repo_url>
helm install <release_name> openedx-harmony/harmony-chart --values values.yaml

There's a tool called chart-releaser that helps to add Helm chart artifacts to GitHub Releases and host such releases in GitHub pages. To automate the process, the chart-releaser-action can be included in the repo workflow to create new releases of the chart and update properly the GitHub Pages hosting. The prerequisites to use the action are included here.

This releaser is used by most of the well-known Helm charts (ingress-nginx, prometheus-charts, among others) to handle the releases and the hosting. However, the discussion is open to bring other options to the table.

@jfavellar90
Copy link
Contributor Author

Guys, I stepped up and created a PR to implement the release and hosting mechanisms of the harmony chart using chart-releaser. Feel free to check it out and provide feedback :)

@felipemontoya
Copy link
Member

Updated PR: #40

@felipemontoya
Copy link
Member

This was fixed by PR40 so I'm closing now

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

No branches or pull requests

2 participants