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

Update regexp for rabbitmq_nodename fact #545

Merged
merged 1 commit into from
Jun 29, 2017

Conversation

SergK
Copy link
Contributor

@SergK SergK commented May 29, 2017

rabbitmqctl status returns node status without trailing points
on rabbitmq: 3.6.10, e.g:
root@rabbitmq:~# rabbitmqctl status
Status of node rabbit@rabbitmq
[{pid,16527},
....

So let's align regexp as well

rabbitmqctl status returns node status without trailing points
on rabbitmq: 3.6.10, e.g:
   root@rabbitmq:~# rabbitmqctl status
     Status of node rabbit@rabbitmq
     [{pid,16527},
       ....

So let's align regexp as well
@baurmatt
Copy link
Contributor

This is definitely needed. @SergK Can you check the tests?

This PR could be related to https://tickets.puppetlabs.com/browse/MODULES-3376

@SergK
Copy link
Contributor Author

SergK commented Jun 21, 2017

@baurmatt I'd love to merge this PR, but the first step - we need to merge fix for CI #546 and after rebase this PR become green. So I need at least some feedback from core reviewers

@SergK
Copy link
Contributor Author

SergK commented Jun 21, 2017

btw, we can merge #548 as well to fix CI

@SergK
Copy link
Contributor Author

SergK commented Jun 21, 2017

@hunner could you please review and comment, thank you

@hunner hunner merged commit f7c3a4a into voxpupuli:master Jun 29, 2017
hunner added a commit that referenced this pull request Jun 29, 2017
Update regexp for rabbitmq_nodename fact
stsnel added a commit to thehyve/puppet-rabbitmq that referenced this pull request Jul 27, 2017
Backport pull request 545 of the main puppet-rabbitmq
repository, which fixes the rabbitmq_nodename fact.

We need to backport it because of dependency issues with
newer versions of the module.
@wyardley wyardley added the bug Something isn't working label Sep 5, 2017
slm0n87 pushed a commit to slm0n87/puppet-rabbitmq that referenced this pull request Mar 7, 2019
…regexp

Update regexp for rabbitmq_nodename fact
cegeka-jenkins pushed a commit to cegeka/puppet-rabbitmq that referenced this pull request Mar 26, 2021
…regexp

Update regexp for rabbitmq_nodename fact
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants