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

Create helm chart #104

Open
DavidNix opened this issue Sep 14, 2022 · 5 comments
Open

Create helm chart #104

DavidNix opened this issue Sep 14, 2022 · 5 comments

Comments

@DavidNix
Copy link
Contributor

It seems to a popular way to install operators. The fact this is a private repo can be handled by Flux in the same way we need to download the private docker image.

GH Pages seems to be the most popular way for hosting charts.

However, you can also host them on ghcr. GHCR will work the same way as authenticating a private docker image on ghcr.

@DavidNix DavidNix self-assigned this Sep 14, 2022
@DavidNix
Copy link
Contributor Author

Note, I can't use the GH pages method because they are public by default. You can only get private gh pages if you have an enterprise account.

@DavidNix
Copy link
Contributor Author

Moving out of Sprint 2. I attempted this and it's going to be more work than the value it will bring. I was able to use the existing kustomize instead.

Also, this was complicated by the fact the operator is still closed source.

Eventually, we will need a helm chart, so I will keep this issue open but de-prioritized.

@DavidNix DavidNix removed their assignment Nov 7, 2022
@morgoved
Copy link

i can help to you;)

@DavidNix
Copy link
Contributor Author

i can help to you;)

Thanks! It seems straight forward with GH pages now that this repo is open source.

@justyntemme
Copy link
Contributor

justyntemme commented Mar 2, 2023

'helm create operator' will generate a preconfigured helm chart. We will need to do a few things however as the kubernetes configuration requires more interaction between nodes than the default configuration. We will also need to change the image to reflect that of the production tagged docker image for the operator. I will create a branch in my fork and make the changes for review.

EDIT: Generally I would be suggesting a 'deploy' directory with the helm charts living in there, however looking at the GitHub pages we can work from that way no problem. I have a current PR out that will build docker images on pull requests, we can use the 'helm test' strategy with https://github.com/marketplace/actions/helm-chart-testing to get more testing integration coverage on PR's.

One thing to note, is that by using githubs architecture we do make it difficult for someone who may want to fork the helm repo directory within the suggested 'deploy' directory. I am going to research the GitHub suggestion and see if there are any reasons the deploy directory would be better for a platform agnostic approach so other cosmos projects can utilize the work we do here.

EDITEDIT: I have created the PR #224 that will begin the work. Once the chat Is generated I will take it and make the required changes, then reference those custom changes in the releaser stage so that's handled in the generation as well.

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