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

ipa-replica-install --setup-adtrust: check for package ipa-server-tru… #2471

Closed
wants to merge 1 commit into from

Conversation

flo-renaud
Copy link
Contributor

…st-ad

When adding the option --setup-adtrust to ipa-replica-install,
we need to check that the package freeipa-server-trust-ad is
installed.
To avoid relying on OS-specific commands like yum, the check is instead
ensuring that the file /usr/share/ipa/smb.conf.empty is present
(this file is delivered by the package).
When the check is unsuccessful, ipa-replica-install exits with an error
message.

Fixes: https://pagure.io/freeipa/issue/7602

@flo-renaud flo-renaud added ipa-4-7 ipa-4-6 Mark for backport to ipa 4.6 needs review Pull Request is waiting for a review labels Oct 23, 2018
@abbra
Copy link
Contributor

abbra commented Oct 23, 2018

ACK

@abbra
Copy link
Contributor

abbra commented Oct 23, 2018

Travis builds are failing now due to https://bugzilla.redhat.com/show_bug.cgi?id=1574478

@abbra abbra added ack Pull Request approved, can be merged and removed needs review Pull Request is waiting for a review labels Oct 23, 2018
@tiran tiran added the re-run Trigger a new run of PR-CI label Oct 23, 2018
@tiran
Copy link
Member

tiran commented Oct 23, 2018

rerun, RunPytest Failed to publish artifacts

@freeipa-pr-ci freeipa-pr-ci removed the re-run Trigger a new run of PR-CI label Oct 23, 2018
…st-ad

When adding the option --setup-adtrust to ipa-replica-install,
we need to check that the package freeipa-server-trust-ad is
installed.
To avoid relying on OS-specific commands like yum, the check is instead
ensuring that the file /usr/share/ipa/smb.conf.empty is present
(this file is delivered by the package).
When the check is unsuccessful, ipa-replica-install exits with an error
message.

Fixes: https://pagure.io/freeipa/issue/7602
@flo-renaud
Copy link
Contributor Author

Rebased to master to pick the annobin fix.

@flo-renaud flo-renaud added the pushed Pull Request has already been pushed label Oct 24, 2018
@flo-renaud
Copy link
Contributor Author

master:

  • 4600e62 ipa-replica-install --setup-adtrust: check for package ipa-server-trust-ad

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 ipa-4-6 Mark for backport to ipa 4.6 pushed Pull Request has already been pushed
Projects
None yet
4 participants