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 a helm chart to deploy chaoskube #295

Merged
merged 1 commit into from
Mar 17, 2021
Merged

Conversation

ghouscht
Copy link
Contributor

@ghouscht ghouscht commented Jan 5, 2021

Quickly hacked together, needs some detailing. Waiting on feedback 😃 See also #275 You can already run helm template chart/chaoskube/ to see/apply the rendered manifests

Copy link
Owner

@linki linki left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ghouscht Thanks for tackling this. Would be nice to have a chart for chaoskube again. I left some questions. I'm fine with merging it as soon as you tell me it's ready.

chart/chaoskube/Chart.yaml Outdated Show resolved Hide resolved
chart/chaoskube/README.md Show resolved Hide resolved
chart/chaoskube/templates/clusterrolebinding.yaml Outdated Show resolved Hide resolved
@ghouscht
Copy link
Contributor Author

Thanks for the comments, I will review and address the missing points later (quite busy this week). The only thing I'm unsure at the moment is how to release and publish the chart. I'll look around how others do this and propose a solution.

@linki
Copy link
Owner

linki commented Jan 13, 2021

Thanks!

@ghouscht ghouscht marked this pull request as draft January 21, 2021 07:10
@ghouscht ghouscht changed the title WIP: add a helm chart to deploy chaoskube add a helm chart to deploy chaoskube Jan 21, 2021
@ghouscht ghouscht marked this pull request as ready for review February 8, 2021 19:49
@linki
Copy link
Owner

linki commented Mar 17, 2021

It's working. Thanks a lot @ghouscht 🙇‍♂️ 🥳

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