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

Test: Check case-insensitive while checking with group lookup for a overrideuser #6725

Closed
wants to merge 1 commit into from

Conversation

madhuriupadhye
Copy link
Contributor

@madhuriupadhye madhuriupadhye commented May 11, 2023

Check case-insensitive while checking with group lookup for a overrideuser

  Added automation for following bugs:
    https://bugzilla.redhat.com/show_bug.cgi?id=2192708
    https://bugzilla.redhat.com/show_bug.cgi?id=2196838
    https://bugzilla.redhat.com/show_bug.cgi?id=2196816
    https://bugzilla.redhat.com/show_bug.cgi?id=2196839

verify:
#6721
https://issues.redhat.com/browse/SSSD-6094

Copy link
Contributor

@jakub-vavra-cz jakub-vavra-cz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see only cosmetic issues othewise LGTM.

Copy link
Contributor

@sumit-bose sumit-bose left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi,

thank you, the new description looks good, ACK.

bye,
Sumit

@jakub-vavra-cz jakub-vavra-cz added the Ready to push Ready to push label Jun 1, 2023
@pbrezina pbrezina removed the Ready to push Ready to push label Jun 5, 2023
@pbrezina
Copy link
Member

pbrezina commented Jun 5, 2023

Pushed PR: #6725

  • master

  • sssd-2-9

@pbrezina pbrezina added Pushed and removed Accepted labels Jun 5, 2023
@pbrezina pbrezina closed this Jun 5, 2023
@pbrezina pbrezina reopened this Jun 29, 2023
@pbrezina pbrezina added Accepted Ready to push Ready to push and removed Pushed Ready to push Ready to push labels Jun 29, 2023
@pbrezina
Copy link
Member

pbrezina commented Jul 4, 2023

Please, remove the merge commit and rebase on top of latest master.

@jakub-vavra-cz
Copy link
Contributor

Pushed PR: #6725

  • master
    • ea34b80 - Test: Check case-insensitive while checking with group lookup for a overrideuser
  • sssd-2-9
    • 752e002 - Test: Check case-insensitive while checking with group lookup for a overrideuser

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants