[autobackport: sssd-2-13] ci: update passkey TMT plan for native CentOS Stream 10 execution#8787
Conversation
![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 simplifies the Packit configuration by integrating the centos-stream-10 target directly into the main test job. It also updates the passkey.fmf test plan to use the new hardware requirements format, dynamically detect the distribution (Fedora vs. CentOS Stream) for setting container tags and COPR commands, enable EPEL on CentOS Stream, and chain SSH commands safely. Feedback was provided regarding the restriction of loading the vhci_hcd kernel module to Fedora only, which would cause CentOS Stream 10 executions to fail since it also requires this module. A suggestion was made to support both Fedora and CentOS.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
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> (cherry picked from commit 334449b)
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> (cherry picked from commit 167b5a9)
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> (cherry picked from commit 943ecbb)
Signed-off-by: Iker Pedrosa <ipedrosa@redhat.com> Reviewed-by: Jakub Vávra <jvavra@redhat.com> Reviewed-by: Justin Stephenson <jstephen@redhat.com>
|
The pull request was accepted by @justin-stephenson with the following PR CI status: 🟢 rpm-build:centos-stream-10-x86_64:upstream (success) There are unsuccessful or unfinished checks. Make sure that the failures are not related to this pull request before merging. |
sssd-bot
force-pushed
the
SSSD-sssd-backport-pr8731-to-sssd-2-13
branch
from
e9dc329 to
8c25081
Compare
4 participants
This is an automatic backport of PR#8731 ci: update passkey TMT plan for native CentOS Stream 10 execution to branch sssd-2-13, created by @ikerexxe.
Please make sure this backport is correct.
Note
The commits were cherry-picked without conflicts.
You can push changes to this pull request
Original commits
334449b - ci: fix error handling in passkey TMT plan SSH commands
167b5a9 - ci: use hardware.memory syntax for TMT passkey tests
943ecbb - ci: update passkey TMT plan for native CentOS Stream 10 execution
Backported commits
Original Pull Request Body
distro.shscript.