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

Publish versioned chart #226

Closed
cehoffman opened this issue Mar 26, 2024 · 17 comments
Closed

Publish versioned chart #226

cehoffman opened this issue Mar 26, 2024 · 17 comments
Assignees
Labels
enhancement New feature or request

Comments

@cehoffman
Copy link

What is missing?

A native helm way of installing the chart without cloning the git repository.

Why do we need it?

To integrate with our deployment pipeline, we have to setup a extra pre process that pulls the git repository to extract the helm chart for use in the remaining parts of our pipeline.

Environment

  • Twingate Kubernetes Operator version:

    `0.6.2

Anything else we need to know?:

Publishing this through GitHub packages should be fine, but we haven't use that explicitly. We've used the GitHub pages method of publishing the chart many times though.

@cehoffman cehoffman added the enhancement New feature or request label Mar 26, 2024
Copy link

linear bot commented Mar 26, 2024

@ekampf
Copy link
Contributor

ekampf commented Mar 26, 2024

Yep. Definately something we need to do but not so simple so might take some time...

@svyatoslavmo
Copy link

@ekampf any ETA on this one? It would provide easier deployment path

@ekampf
Copy link
Contributor

ekampf commented Apr 24, 2024

@svyatoslavmo no ETA I can share yet but it is at the top of my list

@ekampf
Copy link
Contributor

ekampf commented May 7, 2024

@svyatoslavmo would love some feedback on #265
Publishing to an OCI repo on github solves answers your needs right?

@svyatoslavmo
Copy link

Yes, thank you! @ekampf
I would also suggest to version Helm chart and operator together, as Grafana does.

@ekampf
Copy link
Contributor

ekampf commented May 8, 2024

@svyatoslavmo isnt the reason Helm chart has version and appVersion as separate properties is to avoid this?
Helm chart rarely changes do we really want to update its version with every release?

@svyatoslavmo
Copy link

Yes, because it's one version for all changes. Your end user will bump just the chart version, regardless what's been changed.
check this out

@ekampf
Copy link
Contributor

ekampf commented May 17, 2024

I see... ok will make it work on #265

Copy link

This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 5 days.

@github-actions github-actions bot added the Stale label Jul 17, 2024
@ekampf ekampf removed the Stale label Jul 17, 2024
@cehoffman
Copy link
Author

cehoffman commented Jul 30, 2024

I'm ok with either method in general, but for the way my company currently manages these type of resources moving the chart version in lock step with the application version is easiest. Hosting on GitHub OCI is also perfectly fine.

@AlexisDaciuk
Copy link

Any updates on this? we are using the Operator in our clusters but it's not great to have to install it by hand

Copy link

This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 5 days.

@github-actions github-actions bot added the Stale label Nov 24, 2024
@AlexisDaciuk
Copy link

bump

@ekampf
Copy link
Contributor

ekampf commented Dec 18, 2024

@AlexisDaciuk sorry it has been taking so long... PR is almost ready at #479

@HaddadJoe
Copy link

Seems like the PR is ready, we would also benefit from that. Any timelines for the deployment?

@ekampf ekampf closed this as completed Dec 23, 2024
@ekampf
Copy link
Contributor

ekampf commented Dec 24, 2024

@HaddadJoe PR is merged and new operator released 😃

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

Successfully merging a pull request may close this issue.

5 participants