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 #111

Closed
wants to merge 1 commit into from
Closed

Add helm chart #111

wants to merge 1 commit into from

Conversation

jochbru
Copy link

@jochbru jochbru commented Jul 4, 2023

I created a Helm chart in the following repository: https://github.com/cyso/helm-smokeping-prober

I figured it might be better to contribute it to the source project. If you want to @SuperQ, then go ahead and merge it into this project, i'll then either archive or delete the other repository.

Signed-off-by: Jochem Bruijns <jochem.bruijns@cyso.com>
@SuperQ
Copy link
Owner

SuperQ commented Jul 4, 2023

Maybe this would be a better fit for https://github.com/prometheus-community/helm-charts? This way all of the automated testing tooling is available.

@jochbru
Copy link
Author

jochbru commented Jul 4, 2023

Thanks for the suggestion. I have never contributed there before, but i'll see if I'm able to meet the requirements. I'll have to change some stuff with regards to versioning I think. Are there any other things you would think need attention before it could be merged?

@SuperQ
Copy link
Owner

SuperQ commented Jul 4, 2023

I'm not totally up on the standards used in that repo. But it's a nice collection, which makes it easier for people to find this.

@jochbru
Copy link
Author

jochbru commented Jul 4, 2023 via email

@SuperQ SuperQ closed this Jul 4, 2023
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