Skip to content

[Test] test_patching_cluster: fix kernel pinning for RHEL and Fix and stabilize checks for kernel modules - #7602

Merged
gmarciani merged 2 commits into
aws:developfrom
gmarciani:wip/mgiacomo/3170/test-patching-0903-1
Sep 3, 2026
Merged

gmarciani merged 2 commits into
aws:developfrom
gmarciani:wip/mgiacomo/3170/test-patching-0903-1

Conversation

@gmarciani

@gmarciani gmarciani commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

Description of changes

In test_patching_cluster, fix kernel pinning for RHEL and Fix and stabilize checks for kernel modules.
fix kernel pinning on RHEL (with the previous pinning some kernel packages could escape the pinning. Also, fixed the checks on kernel modules to prevent false alarms. Now the test fails the kernel module verification if a mandatory kernel module is not loaded, OR the module is not loaded (may be lazily loaded), but it exists and fails to load. In this way we prevent the false alarms on kernel modules that legitimately do not exist anymore on the new kernel.

Tests

SUCCESS

test-suites:
  patching:
    test_patching.py::test_patching_cluster:
      dimensions:
        - regions: [{{ g4dn_8xlarge_CAPACITY_RESERVATION_2_INSTANCES_2_HOURS_NOPG_ubuntu2404 }}]
          instances: ["g4dn.8xlarge"]
          oss: [{{ UBUNTU_OS_X86_0 }}]
          schedulers: ["slurm"]
          flags: ["patching:full-capped"]
        - regions: [{{ g4dn_8xlarge_CAPACITY_RESERVATION_2_INSTANCES_2_HOURS_NOPG_rhel9 }}]
          instances: ["g4dn.8xlarge"]
          oss: [{{ RHEL_OS_X86_0 }}]
          schedulers: ["slurm"]
          flags: ["patching:full-capped"]
        - regions: [{{ g4dn_8xlarge_CAPACITY_RESERVATION_2_INSTANCES_2_HOURS_NOPG_alinux2023 }}]
          instances: ["g4dn.8xlarge"]
          oss: ["alinux2023"]
          schedulers: ["slurm"]
          flags: ["patching:full"]

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

…ules loaded before/after patching to prevent false alarms.

Now the test fails the kernel module verification if:
1. a mandatory kernel module is not loaded, OR
2. the module is not loaded (may be lazily loaded), but it exists and fails to load.

In this way we prevent the false alarms on kernel modules that legitimately do not exist anymore on the new kernel.
Before this fix, some kernel packages could escape the pinning. With this fix, none of them can.
@gmarciani
gmarciani requested review from a team as code owners September 3, 2026 13:30
@gmarciani gmarciani added skip-changelog-update Disables the check that enforces changelog updates in PRs 3.x Test labels Sep 3, 2026
@gmarciani
gmarciani enabled auto-merge (rebase) September 3, 2026 18:38
@gmarciani
gmarciani merged commit e9a2407 into aws:develop Sep 3, 2026
28 of 29 checks passed
@gmarciani
gmarciani deleted the wip/mgiacomo/3170/test-patching-0903-1 branch September 3, 2026 19:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3.x skip-changelog-update Disables the check that enforces changelog updates in PRs Test

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants