Skip to content

Add helm chart for bulldozer#135

Open
jpigree wants to merge 2 commits into
palantir:developfrom
jpigree:helm-chart
Open

Add helm chart for bulldozer#135
jpigree wants to merge 2 commits into
palantir:developfrom
jpigree:helm-chart

Conversation

@jpigree
Copy link
Copy Markdown

@jpigree jpigree commented Jul 3, 2019

I had to create an Helm chart to install Bulldozer.Since it is pretty generic, I can share it with the community.

Feel free to provide feedback. If you don't helm charts stored in this repository, I can submit it to the official chart repository but I don't know if it will be accepted.

I stored the config in a secret because it contains credentials.

@palantirtech
Copy link
Copy Markdown
Member

Thanks for your interest in palantir/bulldozer, @jpigree! Before we can accept your pull request, you need to sign our contributor license agreement - just visit https://cla.palantir.com/ and follow the instructions. Once you sign, I'll automatically update this pull request.

@jpigree
Copy link
Copy Markdown
Author

jpigree commented Jul 3, 2019

I have another helm chart for palantir policy-bot aswell if people are interested.

@LeoQuote
Copy link
Copy Markdown

This is good, why is there nobody reviewing this pr?

@jpigree
Copy link
Copy Markdown
Author

jpigree commented Jan 11, 2021

I have no clue. I made this so long ago. Perhaps a maintainer can tell us what's wrong.
@asvoboda @bluekeyes

@bluekeyes
Copy link
Copy Markdown
Contributor

Sorry for letting this sit for so long. Unfortunately, none of the maintainers have the expertise with Helm needed to review this and then keep it updated it once merged. We're starting to do more work with Kubernetes, so that might change in the future, but for now I don't think it is something I'm comfortable including as part of the repository.

@LeoQuote
Copy link
Copy Markdown

I have no clue. I made this so long ago. Perhaps a maintainer can tell us what's wrong.
@asvoboda @bluekeyes

You may set your own helm repo instead, it's very common for many apps

@jpigree
Copy link
Copy Markdown
Author

jpigree commented Jan 12, 2021

Sorry for letting this sit for so long. Unfortunately, none of the maintainers have the expertise with Helm needed to review this and then keep it updated it once merged. We're starting to do more work with Kubernetes, so that might change in the future, but for now I don't think it is something I'm comfortable including as part of the repository.

I believed it was the case but at least now, it is clear. No problem. ;)

You may set your own helm repo instead, it's very common for many apps

I am not using bulldozer anymore so I am not so sure about maintaining it either but I will think about it. Should I close this PR?

@trinhpham
Copy link
Copy Markdown

IMO, the chart should be merged and let the maintenance be community-driven. .
If it gets outdate, people who use it (like me) would raise an issue. And trust me, a lot talented people will be there for help.

@yitzhaktal
Copy link
Copy Markdown

I'll be happy to use this chart, any reason it wasn't merged already?

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.

6 participants