Skip to content

[autobackport: sssd-2-9] intg: remove ent_test.py#8286

Merged
alexey-tikhonov merged 1 commit intoSSSD:sssd-2-9from
sssd-bot:SSSD-sssd-backport-pr8283-to-sssd-2-9
Dec 9, 2025
Merged

[autobackport: sssd-2-9] intg: remove ent_test.py#8286
alexey-tikhonov merged 1 commit intoSSSD:sssd-2-9from
sssd-bot:SSSD-sssd-backport-pr8283-to-sssd-2-9

Conversation

@sssd-bot
Copy link
Contributor

@sssd-bot sssd-bot commented Dec 9, 2025

This is an automatic backport of PR#8283 intg: remove ent_test.py to branch sssd-2-9, created by @pbrezina.

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-pr8283-to-sssd-2-9
git checkout SSSD-sssd-backport-pr8283-to-sssd-2-9
git push sssd-bot SSSD-sssd-backport-pr8283-to-sssd-2-9 --force

Original commits
97fa9e7 - intg: remove ent_test.py

Backported commits

  • 24ade55 - intg: remove ent_test.py

Original Pull Request Body

These tests randomly fails, making CI red. They do not test SSSD, just
internal intg tests tooling and since the intg tests are going away,
it does not make sense to spend time on it or even mark it as flaky.

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

This pull request is a backport that removes the ent_test.py file. According to the description, this file contains flaky tests for tooling that is being deprecated. The changes correctly remove the test file and its corresponding entry in src/tests/intg/Makefile.am. This is a sensible cleanup to improve CI stability. The changes are correct and I have no concerns.

@alexey-tikhonov alexey-tikhonov added the no-backport This should go to target branch only. label Dec 9, 2025
These tests randomly fails, making CI red. They do not test SSSD, just
internal intg tests tooling and since the intg tests are going away,
it does not make sense to spend time on it or even mark it as flaky.

Reviewed-by: Alexey Tikhonov <atikhono@redhat.com>
Reviewed-by: Iker Pedrosa <ipedrosa@redhat.com>
(cherry picked from commit 97fa9e7)
@sssd-bot
Copy link
Contributor Author

sssd-bot commented Dec 9, 2025

The pull request was accepted by @alexey-tikhonov 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.

@sssd-bot sssd-bot force-pushed the SSSD-sssd-backport-pr8283-to-sssd-2-9 branch from 24ade55 to 2070a5e Compare December 9, 2025 17:43
@alexey-tikhonov alexey-tikhonov merged commit d7af71e into SSSD:sssd-2-9 Dec 9, 2025
7 of 8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Accepted no-backport This should go to target branch only.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants