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

[Backport][ipa-4-8] replica: Ensure that ipaapi user is allowed to access ifp #4917

Closed
wants to merge 2 commits into from

Conversation

rcritten
Copy link
Contributor

This PR was opened automatically because PR #4914 was pushed to master and backport to ipa-4-8 is required.

ipa-server-install executes ipa-client-install with the --on-master
flag set, which causes the ipaclient.install.client.sssd_enable_ifp()
function to be called.  This function configures sssd so that the
ipaapi user is allowed to access ifp.  Any FreeIPA replica should also
have sssd configured like this, but in that case we cannot simply pass
the --on-master flag to ipa-client-install because it has other side
effects.  The solution is to call the
ipaclient.install.client.sssd_enable_ifp() function from inside the
ipaserver.install.server.replicainstall.promote_sssd() function.

https://pagure.io/freeipa/issue/8403
@rcritten
Copy link
Contributor Author

PR was ACKed automatically because this is backport of PR #4914. Wait for CI to finish before pushing. In case of questions or problems contact @jsf9k who is author of the original PR.

@rcritten rcritten added the ack Pull Request approved, can be merged label Jul 14, 2020
@rcritten
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@rcritten
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@rcritten rcritten added the re-run Trigger a new run of PR-CI label Jul 14, 2020
@freeipa-pr-ci freeipa-pr-ci removed the re-run Trigger a new run of PR-CI label Jul 14, 2020
@rcritten rcritten added the re-run Trigger a new run of PR-CI label Jul 14, 2020
@freeipa-pr-ci freeipa-pr-ci removed the re-run Trigger a new run of PR-CI label Jul 14, 2020
@rcritten rcritten added the pushed Pull Request has already been pushed label Jul 14, 2020
@rcritten
Copy link
Contributor Author

ipa-4-8:

  • 4b8da1b replica: Ensure the ipaapi user is allowed to access ifp on replicas
  • 6de4b0f replica: Add tests to ensure the ipaapi user is allowed access to ifp on replicas

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