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-6] Trust pass args and options #2977

Closed
wants to merge 1 commit into from

Conversation

abbra
Copy link
Contributor

@abbra abbra commented Apr 1, 2019

Refactor com.redhat.idm.trust-fetch.domains oddjob helper to allow
passing administrative credentials and a domain controller to talk to.

This approach allows to avoid rediscovering a domain controller in case
a user actually specified the domain controller when establishing trust.

It also allows to pass through admin credentials if user decides to do
so. The latter will be used later to allow updating trust topology in a
similar oddjob helper.

Resolves: https://pagure.io/freeipa/issue/7895
Reviewed-By: Christian Heimes cheimes@redhat.com
(cherry picked from commit de4a987)

Refactor com.redhat.idm.trust-fetch.domains oddjob helper to allow
passing administrative credentials and a domain controller to talk to.

This approach allows to avoid rediscovering a domain controller in case
a user actually specified the domain controller when establishing trust.

It also allows to pass through admin credentials if user decides to do
so. The latter will be used later to allow updating trust topology in a
similar oddjob helper.

Resolves: https://pagure.io/freeipa/issue/7895
Reviewed-By: Christian Heimes <cheimes@redhat.com>
(cherry picked from commit de4a987)
@abbra abbra added the ipa-4-6 Mark for backport to ipa 4.6 label Apr 1, 2019
@tiran tiran added the re-run Trigger a new run of PR-CI label Apr 1, 2019
@freeipa-pr-ci freeipa-pr-ci removed the re-run Trigger a new run of PR-CI label Apr 1, 2019
@Tiboris Tiboris added the ack Pull Request approved, can be merged label Apr 2, 2019
@Tiboris
Copy link
Member

Tiboris commented Apr 2, 2019

ipa-4-6:

  • 916d4f3 oddjob: allow to pass options to trust-fetch-domains

@Tiboris Tiboris added the pushed Pull Request has already been pushed label Apr 2, 2019
@Tiboris Tiboris closed this Apr 2, 2019
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