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 auto cluster configuration support #883

Merged
merged 1 commit into from
Apr 20, 2021

Conversation

fatpat
Copy link
Contributor

@fatpat fatpat commented Apr 17, 2021

Pull Request (PR) description

This PR add the clustering management

This Pull Request (PR) fixes the following issues

Fixes #792
Closes #736

@fatpat fatpat force-pushed the cluster_support branch 7 times, most recently from fa40bee to 79e6f6b Compare April 17, 2021 05:47
@fatpat
Copy link
Contributor Author

fatpat commented Apr 17, 2021

Still working on tests...

@fatpat fatpat force-pushed the cluster_support branch 4 times, most recently from 901f8dd to bed4aeb Compare April 19, 2021 09:18
@fatpat
Copy link
Contributor Author

fatpat commented Apr 19, 2021

tests are now all green. Only one is failing due to network issues.

it's good to me

@wyardley @bastelfreak

@fatpat
Copy link
Contributor Author

fatpat commented Apr 19, 2021

it would be great to include this PR in the coming release (Release 11.1.0 #879)

@wyardley wyardley added the enhancement New feature or request label Apr 19, 2021
@wyardley
Copy link
Contributor

@fatpat have you tested this with an actual auto-cluster setup as well?

Only other question is, it seems like the original author still has some credit from the commit email; if this is based on one or more previous PRs, is it worth also mentioning the authors in the commit message?

- added clustername fact and spec tests for it
- added rabbitmq_cluster type and provider with tests
- added related doc
- based from PR#736, credits to @vchepkov
@fatpat
Copy link
Contributor Author

fatpat commented Apr 20, 2021

@fatpat have you tested this with an actual auto-cluster setup as well?

yes this has been tested and we used it in production.

Only other question is, it seems like the original author still has some credit from the commit email; if this is based on one or more previous PRs, is it worth also mentioning the authors in the commit message?

fixed in the commit message. My mistake

@wyardley wyardley merged commit 9e5164b into voxpupuli:master Apr 20, 2021
wyardley added a commit to wyardley/puppet-rabbitmq that referenced this pull request Apr 20, 2021
- Add support for oom_score_adj (voxpupuli#877)
- CLI Environment Fixes (voxpupuli#876)
- make sure the rabbitmq_version method actually returns the version (voxpupuli#874)
- Use mocked facts in tests (voxpupuli#873)
- Enable Puppet 7 support (voxpupuli#881)
- Add auto cluster configuration support (voxpupuli#883)
@wyardley wyardley mentioned this pull request Apr 20, 2021
@fatpat fatpat deleted the cluster_support branch April 21, 2021 02:35
wyardley added a commit to wyardley/puppet-rabbitmq that referenced this pull request May 6, 2021
- Add support for oom_score_adj (voxpupuli#877)
- CLI Environment Fixes (voxpupuli#876)
- make sure the rabbitmq_version method actually returns the version (voxpupuli#874)
- Use mocked facts in tests (voxpupuli#873)
- Enable Puppet 7 support (voxpupuli#881)
- Add auto cluster configuration support (voxpupuli#883)
wyardley added a commit to wyardley/puppet-rabbitmq that referenced this pull request May 6, 2021
- Add support for oom_score_adj (voxpupuli#877)
- CLI Environment Fixes (voxpupuli#876)
- make sure the rabbitmq_version method actually returns the version (voxpupuli#874)
- Use mocked facts in tests (voxpupuli#873)
- Enable Puppet 7 support (voxpupuli#881)
- Add auto cluster configuration support (voxpupuli#883)
- Compatibility with camptocamp/systemd 3.x (voxpupuli#886)
wyardley added a commit to wyardley/puppet-rabbitmq that referenced this pull request May 6, 2021
- Add support for oom_score_adj (voxpupuli#877)
- CLI Environment Fixes (voxpupuli#876)
- make sure the rabbitmq_version method actually returns the version (voxpupuli#874)
- Use mocked facts in tests (voxpupuli#873)
- Enable Puppet 7 support (voxpupuli#881)
- Add auto cluster configuration support (voxpupuli#883)
- Compatibility with camptocamp/systemd 3.x (voxpupuli#886)
- puppetlabs/apt: Allow 8.x (voxpupuli#884)
- puppet/archive: Allow 5.x (voxpupuli#884)
- puppetlabs/stdlib: allow 7.x (voxpupuli#884)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Auto Clustering of nodes should be enabled
3 participants