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] Resolve user/group names in idoverride*-find ack #2655

Closed
wants to merge 2 commits into from

Conversation

tiran
Copy link
Member

@tiran tiran commented Dec 7, 2018

Manual backport of #2648 to IPA 4.6 branch. I'm Auto-ACK this PR because it was only a trivial merge conflict.

Resolve user/group names in idoverride*-find

ipa idoverrideuser-find and ...group-find have an --anchor argument. The
anchor argument used to support only anchor UUIDs like
':IPA:domain:UUID' or ':SID:S-sid'. The find commands now detect regular
user or group names and translate them to anchors.

Add integration tests for idviews

Add several tests to verify new anchor override and general idview
override functionality.

Run idviews integration tests in nightly

Fixes: pagure.io/freeipa/issue/6594

ipa idoverrideuser-find and ...group-find have an --anchor argument. The
anchor argument used to support only anchor UUIDs like
':IPA:domain:UUID' or ':SID:S-sid'. The find commands now detect regular
user or group names and translate them to anchors.

Fixes: https://pagure.io/freeipa/issue/6594
Signed-off-by: Christian Heimes <cheimes@redhat.com>
Add several tests to verify new anchor override and general idview
override functionality.

Fixes: https://pagure.io/freeipa/issue/6594
Signed-off-by: Christian Heimes <cheimes@redhat.com>
@tiran tiran added the ack Pull Request approved, can be merged label Dec 7, 2018
@rcritten rcritten added the pushed Pull Request has already been pushed label Dec 7, 2018
@rcritten
Copy link
Contributor

rcritten commented Dec 7, 2018

ipa-4-6:

  • 9229c83 Resolve user/group names in idoverride*-find
  • f1badca Add integration tests for idviews

@rcritten rcritten closed this Dec 7, 2018
@tiran tiran deleted the backport_pr2648_ipa-4-6 branch March 29, 2019 09:09
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
2 participants