Skip to content

[autobackport: sssd-2-11] Tests: Fix ipa multihost test_authentication_indicators#8528

Merged
jakub-vavra-cz merged 1 commit intoSSSD:sssd-2-11from
sssd-bot:SSSD-sssd-backport-pr8525-to-sssd-2-11
Mar 17, 2026
Merged

[autobackport: sssd-2-11] Tests: Fix ipa multihost test_authentication_indicators#8528
jakub-vavra-cz merged 1 commit intoSSSD:sssd-2-11from
sssd-bot:SSSD-sssd-backport-pr8525-to-sssd-2-11

Conversation

@sssd-bot
Copy link
Contributor

This is an automatic backport of PR#8525 Tests: Fix ipa multihost test_authentication_indicators to branch sssd-2-11, created by @aborah-sudo.

Please make sure this backport is correct.

Note

The commits were cherry-picked without conflicts.

You can push changes to this pull request

git remote add sssd-bot git@github.com:sssd-bot/sssd.git
git fetch sssd-bot refs/heads/SSSD-sssd-backport-pr8525-to-sssd-2-11
git checkout SSSD-sssd-backport-pr8525-to-sssd-2-11
git push sssd-bot SSSD-sssd-backport-pr8525-to-sssd-2-11 --force

Original commits
a6d0f0c - Tests: Fix ipa multihost test_authentication_indicators

Backported commits

  • b35fe7b - Tests: Fix ipa multihost test_authentication_indicators

Original Pull Request Body

Provide sleep time to test

Provide sleep time to test

Reviewed-by: Jakub Vávra <jvavra@redhat.com>
(cherry picked from commit a6d0f0c)
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.

LGTM.

@jakub-vavra-cz jakub-vavra-cz added Accepted Tests Trivial A single reviewer is sufficient to review the Pull Request labels Mar 17, 2026
@jakub-vavra-cz jakub-vavra-cz merged commit b85a4dd into SSSD:sssd-2-11 Mar 17, 2026
17 of 26 checks passed
Copy link

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

The pull request introduces a time.sleep(5) call within the test_authentication_indicators function in src/tests/multihost/ipa/test_misc.py. This change appears to be a pragmatic fix to address potential timing issues or race conditions in the test environment, ensuring that the ipa sudorule-add-user command has sufficient time to propagate before subsequent commands are executed. While adding fixed delays to tests can sometimes indicate underlying synchronization problems or lead to slower test execution, this specific modification, being a test-level adjustment for stability, does not present any issues that meet the 'high' or 'critical' severity criteria for the core codebase or test framework itself.

@sssd-bot
Copy link
Contributor Author

The pull request was accepted by @jakub-vavra-cz with the following PR CI status:


🟢 CodeQL (success)
🔴 rpm-build:centos-stream-10-x86_64:upstream (failure)
🔴 rpm-build:fedora-42-x86_64:upstream (failure)
🔴 rpm-build:fedora-43-x86_64:upstream (failure)
🔴 rpm-build:fedora-44-x86_64:upstream (failure)
🔴 rpm-build:fedora-rawhide-x86_64:upstream (failure)
🟢 Build / make-distcheck (success)
🟡 ci / intgcheck (centos-10) (in_progress)
🟡 ci / intgcheck (fedora-42) (in_progress)
🟡 ci / intgcheck (fedora-43) (in_progress)
🟡 ci / intgcheck (fedora-44) (in_progress)
🔴 ci / intgcheck (fedora-45) (failure)
🟢 ci / prepare (success)
🟡 ci / system (centos-10) (in_progress)
🟡 ci / system (fedora-42) (in_progress)
🔴 ci / system (fedora-43) (failure)
🔴 ci / system (fedora-44) (failure)
🔴 ci / system (fedora-45) (failure)
🟢 Static code analysis / codeql (success)
🟢 Static code analysis / pre-commit (success)
🟢 Static code analysis / python-system-tests (success)


There are unsuccessful or unfinished checks. Make sure that the failures are not related to this pull request before merging.

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

Labels

Accepted Tests Trivial A single reviewer is sufficient to review the Pull Request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants