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 for bulldozer #135

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from
Open

Conversation

jpigree
Copy link

@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
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
Author

jpigree commented Jul 3, 2019

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

@LeoQuote
Copy link

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

@jpigree
Copy link
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
Member

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

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

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.

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.

5 participants