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

Make rabbitmq service restart immediately #154

Merged
merged 3 commits into from Nov 21, 2014

Conversation

@wenchma
Copy link
Contributor

commented Oct 29, 2014

Issue: #153

Fixes issue #153

@michaelklishin

This comment has been minimized.

Copy link
Member

commented Oct 29, 2014

I don't have the time to test it right away but from a quick glance, LGTM.

@kramvan1

This comment has been minimized.

Copy link
Contributor

commented Oct 30, 2014

This looks like the right approach for conf changes.

@wenchma

This comment has been minimized.

Copy link
Contributor Author

commented Nov 6, 2014

@michaelklishin do you test this patch? There is no problem, can you give a merge ?
during deploying the openstack with rabbitmq, in the end, rabbitmq-server will be restarted , so throw error messages in openstack components logs. thanks!

@michaelklishin

This comment has been minimized.

Copy link
Member

commented Nov 6, 2014

@wenchma I haven't and I'm not a collaborator on this repo but it looks good to me. @kennonkwok I think this change is good to be merged.

jjasghar pushed a commit that referenced this pull request Nov 21, 2014
JJ Asghar
Merge pull request #154 from wenchma/issue#153
Make rabbitmq service restart immediately

@jjasghar jjasghar merged commit 5b06779 into rabbitmq:master Nov 21, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.