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

upgrade: adtrust update_tdo_gidnumber plugin must check if adtrust is… #724

Closed
wants to merge 1 commit into from

Conversation

flo-renaud
Copy link
Contributor

@flo-renaud flo-renaud commented Apr 20, 2017

… installed

During upgrade, the plugin update_tdo_gidnumber is launched in order to
add a gidnumber to the Trusted Domain Object.
This plugin should not be run when ad trust is not installed, otherwise an
error message is displayed.

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

… installed

During upgrade, the plugin update_tdo_gidnumber is launched in order to
add a gidnumber to the Trusted Domain Object.
This plugin should not be run when ad trust is not installed, otherwise an
error message is displayed.

https://pagure.io/freeipa/issue/6881
@abbra
Copy link
Contributor

abbra commented Apr 20, 2017

LGTM

@abbra abbra added the ack Pull Request approved, can be merged label Apr 20, 2017
@MartinBasti MartinBasti added the pushed Pull Request has already been pushed label Apr 24, 2017
@MartinBasti
Copy link
Contributor

master:

  • 434d9e5 upgrade: adtrust update_tdo_gidnumber plugin must check if adtrust is installed

ipa-4-5:

  • c05bd60 upgrade: adtrust update_tdo_gidnumber plugin must check if adtrust is installed

@flo-renaud flo-renaud deleted the bz1442427 branch May 5, 2017 07:29
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