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

feat: add helm chart #619

Merged
merged 5 commits into from
Feb 22, 2024
Merged

feat: add helm chart #619

merged 5 commits into from
Feb 22, 2024

Conversation

david972
Copy link
Contributor

issue #615

@david972
Copy link
Contributor Author

The chart working well

There is couple of thing that need to do

  • I did not find the docker image tag with semver
  • need to add some pre-check for the config

@levkk
Copy link
Contributor

levkk commented Feb 22, 2024

I have no way to validate that this is correct, but I'm happy to merge it to give it a try. Thanks for the contribution!

@levkk levkk merged commit 358724f into postgresml:main Feb 22, 2024
@alexelent-carta
Copy link

alexelent-carta commented May 15, 2024

what would the helm repo add command be to use this chart?

Is it possible this never worked due to https://github.com/postgresml/pgcat/actions/runs/8008557639/job/21875234561?

@code-sauce
Copy link

@david972 @alexelent-carta do we now know how to add this repo to helm?

what's the exact command? Locally, it complains the repository doesn't have index.yaml

@david972
Copy link
Contributor Author

hi
for now we'll need to do it locally as the publishing failed
The log is available anymore i cannot investigate
I'll do it later

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.

4 participants