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

[flare] Warn user if not root #1509

Merged
merged 1 commit into from
Apr 6, 2015
Merged

[flare] Warn user if not root #1509

merged 1 commit into from
Apr 6, 2015

Conversation

degemer
Copy link
Member

@degemer degemer commented Apr 3, 2015

As otherwise supervisorctl status won't be collected.

Ask for email even to update a case, as it is now the 'id' to check if
it is the good case (ie a case open by this customer).

@degemer degemer added the bugfix label Apr 3, 2015
@degemer degemer added this to the 5.3.0 milestone Apr 3, 2015
@degemer degemer force-pushed the quentin/update-flare branch from 853dd30 to d477c77 Compare April 4, 2015 17:27
As otherwise `supervisorctl status` won't be collected.

Ask for email even to update a case, as it is now the 'id' to check if
it is the good case (ie a case open by this customer).
@degemer degemer force-pushed the quentin/update-flare branch from d477c77 to e47d733 Compare April 4, 2015 17:52
@remh
Copy link
Contributor

remh commented Apr 6, 2015

Looks good to me!

remh added a commit that referenced this pull request Apr 6, 2015
@remh remh merged commit 240383e into master Apr 6, 2015
@degemer degemer deleted the quentin/update-flare branch April 6, 2015 22:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants