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

fix managed-entries printing IPA not installed #762

Closed
wants to merge 1 commit into from

Conversation

stlaz
Copy link
Contributor

@stlaz stlaz commented May 4, 2017

ipa-managed-entries would print "IPA is not configured on this system."
even though this is not true if run as a normal user. Add check for
root running the script.

https://pagure.io/freeipa/issue/6928

ipa-managed-entries would print "IPA is not configured on this system."
even though this is not true if run as a normal user. Add check for
root running the script.

https://pagure.io/freeipa/issue/6928
@pvomacka pvomacka self-assigned this May 5, 2017
@pvomacka pvomacka added the ack Pull Request approved, can be merged label May 9, 2017
@tkrizek
Copy link
Contributor

tkrizek commented May 10, 2017

master:

  • 6522c4a fix managed-entries printing IPA not installed

@tkrizek tkrizek added the pushed Pull Request has already been pushed label May 10, 2017
@tkrizek tkrizek closed this May 10, 2017
@stlaz stlaz deleted the managed branch July 7, 2017 12:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ack Pull Request approved, can be merged pushed Pull Request has already been pushed
Projects
None yet
3 participants