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

Removing Helm chart to deploy RabbitmqCluster object #803

Merged
merged 2 commits into from
Aug 16, 2021
Merged

Conversation

Zerpet
Copy link
Collaborator

@Zerpet Zerpet commented Aug 16, 2021

Note to reviewers: remember to look at the commits in this PR and consider if they can be squashed

Summary Of Changes

We are not expecting this chart to be in use, and we no longer use it internally.

Additional Context

N/A

Local Testing

Tests and packaging tasks have been removed from CI.

We are not expecting this chart to be in use, and we no longer use it
internally.

Signed-off-by: Aitor Perez Cedres <[email protected]>
@Zerpet Zerpet merged commit fec7f9c into main Aug 16, 2021
@Zerpet Zerpet deleted the delete-helm-chart branch August 16, 2021 15:59
@kvendingoldo
Copy link

@Zerpet can you tell why Helm was deleted? Should I use kubectl plugin instead?

@Zerpet
Copy link
Collaborator Author

Zerpet commented Sep 13, 2021

@kvendingoldo the main reason was adoption. Since we didn't publish this chart to Helm Hub, and it was used mostly within our team, we believe this chart was not used at all. I use the kubectl plugin nowadays to deploy RabbitmqCluster objects.

@politician
Copy link

@Zerpet that's unfortunate. Helm is much better when used with GitOps to manage version upgrades (And GitOps is gaining in popularity).

There is a helm chart over @bitnami but nothing beats an officially maintained chart and there are easy ways to publish it

@MarcialRosales
Copy link
Collaborator

@politician bitnami cluster-operator helm chart is an officially maintained chart. It is been actively maintained and tested against latest RabbitMQ kubernetes cluster-operator by bitnami team.

@politician
Copy link

@MarcialRosales Ok I didn't know it was considered as the official chart. Maybe the official documentation should reflect that? I must say, I am always looking for the Helm install instructions because I know my life will be easier for my workflow (GitOps) and am sure many others are in the same boat.

@MarcialRosales
Copy link
Collaborator

@politician You are absolutely right. Thanks for the suggestion. We will make sure that our official documentation reflects it as it is the first point of contact to the product. Thanks!

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