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

tests: mark xfail for test_selinux_user_optimized on fed<=28 #2839

Closed
wants to merge 1 commit into from

Conversation

flo-renaud
Copy link
Contributor

The test TestUserPermissions::test_selinux_user_optimized is
testing the fix for SSSD issue 3819, but the fix is not
available in fedora 28. Hence mark the test as xfail when
executed on fedora <=28 (our nightly tests also run on fed 28).

For full ref: fixed in sssd 1.16.4, Fedora 28 provides
1.16.3-2.fc28 only, while Fedora 29 provides 2.0.0-3.fc29.

related ticket : https://pagure.io/SSSD/sssd/issue/3819

The test TestUserPermissions::test_selinux_user_optimized is
testing the fix for SSSD issue 3819, but the fix is not
available in fedora 28. Hence mark the test as xfail when
executed on fedora <=28 (our nightly tests also run on fed 28).

For full ref: fixed in sssd 1.16.4, Fedora 28 provides
1.16.3-2.fc28 only, while Fedora 29 provides 2.0.0-3.fc29.

related ticket : https://pagure.io/SSSD/sssd/issue/3819
@flo-renaud flo-renaud added trivial needs review Pull Request is waiting for a review ipa-next Mark as master (4.13) only labels Feb 15, 2019
@flo-renaud
Copy link
Contributor Author

Marking as ipa-next only as the test in included only in the master branch (see #2777).

@rcritten rcritten added ack Pull Request approved, can be merged and removed needs review Pull Request is waiting for a review labels Feb 15, 2019
@flo-renaud flo-renaud added the pushed Pull Request has already been pushed label Feb 15, 2019
@flo-renaud
Copy link
Contributor Author

master:

  • e63c6b2 tests: mark xfail for test_selinux_user_optimized on fed<=28

@flo-renaud flo-renaud closed this Feb 15, 2019
@flo-renaud flo-renaud deleted the mark_xfail branch February 20, 2019 14:45
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-next Mark as master (4.13) only pushed Pull Request has already been pushed trivial
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants