bulldozer icon indicating copy to clipboard operation
bulldozer copied to clipboard

Add helm chart for bulldozer

Open jpigree opened this issue 6 years ago • 9 comments

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.

jpigree avatar Jul 03 '19 02:07 jpigree

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.

palantirtech avatar Jul 03 '19 02:07 palantirtech

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

jpigree avatar Jul 03 '19 02:07 jpigree

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

LeoQuote avatar Jan 11 '21 07:01 LeoQuote

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

jpigree avatar Jan 11 '21 17:01 jpigree

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.

bluekeyes avatar Jan 12 '21 01:01 bluekeyes

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

LeoQuote avatar Jan 12 '21 03:01 LeoQuote

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?

jpigree avatar Jan 12 '21 17:01 jpigree

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.

trinhpham avatar Apr 07 '22 10:04 trinhpham