ci: update passkey TMT plan for native CentOS Stream 10 execution#8731
Merged
Conversation
ikerexxe
added
Waiting for review
Trivial
Contributor
Author
|
Setting |
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request modifies the plans/passkey.fmf file to ensure that the dnf upgrade command is executed conditionally upon the success of $COPR_ENABLE_CMD for both the client and IPA master containers. I have no feedback to provide as no review comments were submitted.
ikerexxe
added
backport-to-sssd-2-12
backport-to-sssd-2-13
and removed
no-backport
ikerexxe
changed the title
Contributor
Author
|
/packit build |
Contributor
|
The ci is red @ikerexxe . |
ikerexxe
force-pushed
the
tmt-fixes
branch
3 times, most recently
from
2358e5f to
f012f50
Compare
Contributor
Author
|
/packit build |
Contributor
Author
|
All current CI failures are unrelated: Github runners are also failing for the same reason in other PRs and Testing Farm is failing due to a known issue with the LDAP server. |
jakub-vavra-cz
approved these changes
Jun 8, 2026
jakub-vavra-cz
left a comment
jakub-vavra-cz
left a comment
Contributor
There was a problem hiding this comment.
LGTM the tf c10s pipeline is green.
justin-stephenson
approved these changes
Jun 8, 2026
If COPR_ENABLE_CMD fails, the script will now fail immediately instead of proceeding to dnf upgrade with packages from default repositories, ensuring tests run against the correct SSSD version. Signed-off-by: Iker Pedrosa <ipedrosa@redhat.com> Reviewed-by: Jakub Vávra <jvavra@redhat.com> Reviewed-by: Justin Stephenson <jstephen@redhat.com>
Update passkey.fmf to use the `hardware.memory` specification. Changed from fixed 16GB to ">= 16 GB" requirement as per Testing Farm documentation at https://docs.testing-farm.io/Testing%20Farm/0.1/test-request.html#ram Signed-off-by: Iker Pedrosa <ipedrosa@redhat.com> Reviewed-by: Jakub Vávra <jvavra@redhat.com> Reviewed-by: Justin Stephenson <jstephen@redhat.com>
With 16GB RAM allocation, CentOS Stream 10 tests can now run directly on matching VM and containers. Thus, update the plan to take this into account and detect the distribution accordingly. Signed-off-by: Iker Pedrosa <ipedrosa@redhat.com> Reviewed-by: Jakub Vávra <jvavra@redhat.com> Reviewed-by: Justin Stephenson <jstephen@redhat.com>
Contributor
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
distro.shscript.