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

Add helm chart #10

Closed
wants to merge 11 commits into from
Closed

Add helm chart #10

wants to merge 11 commits into from

Conversation

cklingspor
Copy link
Contributor

Hey everyone,

I've created a first draft of a helm chart for the parquet-exporter. I do not have a setup where I can use AWS as of now, therefore the actual push to a S3 bucket is not yet tested. However, I was able to install the chart and run the cronjob in my minikube.

  • Used helm-docs to create a rough documentation around the chart
  • The chart itself is relatively straight forward I think

Let me know what you think!

@mattray
Copy link
Contributor

mattray commented Apr 3, 2024

Do we want to move this to https://github.com/opencost/opencost-helm-chart/ and maybe put it under charts/opencost-parquet-exporter? That would keep OpenCost Helm charts in 1 location. @lmello or @sntxrr want to weigh in?

@cklingspor
Copy link
Contributor Author

Do we want to move this to https://github.com/opencost/opencost-helm-chart/ and maybe put it under charts/opencost-parquet-exporter? That would keep OpenCost Helm charts in 1 location. @lmello or @sntxrr want to weigh in?

Sure thing. I did put it here because it was listed as a todo here. Therefore I assumed that it was meant to be here. But I can close here and move the code to https://github.com/opencost/opencost-helm-chart/

@mattray
Copy link
Contributor

mattray commented Apr 5, 2024

@cklingspor if you want to close this one and re-open it over in opencost-helm-charts that would be appreciated.

@cklingspor
Copy link
Contributor Author

Continued here: opencost/opencost-helm-chart#195

@cklingspor cklingspor deleted the add_helm_chart branch April 8, 2024 07:18
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.

2 participants