[Backport][ipa-4-6] ipatests: filter_users should be applied correctly if SSSD starts offline

[amore17]

This is manual back-port of : #3349

[flo-renaud]

Hi @amore17
pylint failed with

************* Module ipatests.test_integration.test_sssd

ipatests/test_integration/test_sssd.py:16: [E0611(no-name-in-module), ] No name 'osinfo' in module 'ipaplatform')

ipatests/test_integration/test_sssd.py:16: [E0401(import-error), ] Unable to import 'ipaplatform.osinfo')

You will need to adapt the PR because osinfo was introduced after ipa-4-6.

[amore17]

@flo-renaud As per suggestion of @tiran I have removed back-port of 1dadcd6 and added xfail in another commit as sssd bulids are not available in fedora-27

[flo-renaud]

Hi @amore17

• the nightly definition needs to be updated in order to execute this test (ipatests/prci_definitions/nightly_ipa-4-6.yaml)
• The backport also takes the tests from ipatests: add tests for cached_auth_timeout in sssd.conf #3602 but this was not planned for ipa-4-6, are you sure you need the whole file?

[amore17]

Hi @amore17

* the nightly definition needs to be updated in order to execute this test (ipatests/prci_definitions/nightly_ipa-4-6.yaml)

* The backport also takes the tests from #3602 but this was not planned for ipa-4-6, are you sure you need the whole file?

Adding postponed label.
As there is test which is part of #3602 which is not back-ported yet to ipa-4-6
Pending review on @wladich

[flo-renaud]

Hi @amore17
the backport looks ok but you should add a temp commit to launch the test in PRCI and check that it is properly marked as xfail.

[flo-renaud]

Hi @amore17
thanks for the update. The test was properly marked as xfailed, you can remove the temp commit.

[flo-renaud]

Hi @amore17
thanks for the update, tests are green, ACK.

[fcami]

ipa-4-6:

• 4b70132 ipatests: filter_users should be applied correctly.
• d3b740e Mark xfail for test_is_user_filtered