Skip to content

Tests: Add sleep time in multihost test#8597

Merged
madhuriupadhye merged 1 commit into
SSSD:sssd-2-9from
madhuriupadhye:backport_2-9
Apr 13, 2026
Merged

Tests: Add sleep time in multihost test#8597
madhuriupadhye merged 1 commit into
SSSD:sssd-2-9from
madhuriupadhye:backport_2-9

Conversation

@madhuriupadhye

@madhuriupadhye madhuriupadhye commented Apr 13, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Backport of #8525

@madhuriupadhye madhuriupadhye added Waiting for review Trivial A single reviewer is sufficient to review the Pull Request no-backport This should go to target branch only. labels Apr 13, 2026
gemini-code-assist[bot]
gemini-code-assist Bot reviewed Apr 13, 2026
View reviewed changes

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

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code Review

This pull request introduces a 5-second delay in the test_authentication_indicators test case within src/tests/multihost/ipa/test_misc.py. The sleep is added after a user is added to a sudo rule and before executing sudo -l, likely to account for propagation delays in the environment. There are no review comments to evaluate, and I have no feedback to provide.

@jakub-vavra-cz jakub-vavra-cz self-assigned this Apr 13, 2026
@madhuriupadhye @sssd-bot
Tests: Add sleep time in multihost test
7c52d49 
Backport of SSSD#8525

Signed-off-by: Madhuri Upadhye <[email protected]>
Reviewed-by: Jakub Vávra <[email protected]>
@sssd-bot

sssd-bot commented Apr 13, 2026

Copy link
Copy Markdown
Contributor

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

🟢 CodeQL (success)
🟢 rpm-build:centos-stream-9-x86_64:upstream (success)
🟢 Build / make-distcheck (success)
🟢 ci / prepare (success)
🟡 ci / system (centos-9) (in_progress)
🟢 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.

@madhuriupadhye
madhuriupadhye merged commit 66ec847 into SSSD:sssd-2-9 Apr 13, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jakub-vavra-cz jakub-vavra-cz jakub-vavra-cz approved these changes
+1 more reviewer
@gemini-code-assist gemini-code-assist[bot] gemini-code-assist[bot] left review comments
Reviewers whose approvals may not affect merge requirements

Assignees

@jakub-vavra-cz jakub-vavra-cz

Labels

Accepted no-backport This should go to target branch only. Trivial A single reviewer is sufficient to review the Pull Request

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@madhuriupadhye @sssd-bot @jakub-vavra-cz