Skip to content

test/integration, .packit.yml: add rhel-9.8 tests - #1532

Merged
usebeforefree merged 1 commit into
dynup:masterfrom
usebeforefree:integration-rhel9.8
Oct 30, 2026
Merged

usebeforefree merged 1 commit into
dynup:masterfrom
usebeforefree:integration-rhel9.8

Conversation

@usebeforefree

Copy link
Copy Markdown
Contributor

Added rebased kernel-5.14.0-687.5.1.el9_8 tests for rhel-9.8 and invoked them trough packit.

@ykopkova
ykopkova requested review from Tomcat-42 and joe-lawrence and removed request for Tomcat-42 October 26, 2026 17:20
@usebeforefree
usebeforefree force-pushed the integration-rhel9.8 branch 2 times, most recently from 4f27191 to 167aea8 Compare October 27, 2026 17:39
@usebeforefree

usebeforefree commented Oct 27, 2026 •

Copy link
Copy Markdown
Contributor Author

Investigating the added test failures:

https://artifacts.osci.redhat.com/testing-farm/d02c7bfd-6711-498f-a65f-bfc9293b1acc/

Found this closed PR: #1509

@usebeforefree

Copy link
Copy Markdown
Contributor Author
build: warn-detect-FAIL
RHEL distribution detected
Downloading kernel source for 5.14.0-687.60.1.el9_8.x86_64
Unpacking kernel source
ERROR: gcc/kernel version mismatch
gcc version:    GCC: (GNU) 11.5.0 20240719 (Red Hat 11.5.0-15)
kernel version: GCC: (GNU) 11.5.0 20240719 (Red Hat 11.5.0-14)
Install the matching gcc version (recommended) or use --skip-compiler-check
to skip the version matching enforcement (not recommended)
ERROR: kpatch build failed. Check /root/.kpatch/build.log for more details.
combine: skipping rhel-9.8/warn-detect-FAIL.patch
build: combined module
Using cache at /root/.kpatch/src
ERROR: gcc/kernel version mismatch
gcc version:    GCC: (GNU) 11.5.0 20240719 (Red Hat 11.5.0-15)
kernel version: GCC: (GNU) 11.5.0 20240719 (Red Hat 11.5.0-14)
Install the matching gcc version (recommended) or use --skip-compiler-check
to skip the version matching enforcement (not recommended)
ERROR: kpatch build failed.
ERROR: combined build failed
can't find test-COMBINED.ko, skipping
1 errors encountered

@usebeforefree

Copy link
Copy Markdown
Contributor Author

I am not sure if the solution would be to just pass the --skip-compiler-check like we do in tier2 tests? @ykopkova what do you think?

@usebeforefree

usebeforefree commented Oct 29, 2026 •

Copy link
Copy Markdown
Contributor Author

I implemented all @joe-lawrence 's suggestions from the PR above:

  • Can we use the updated module.patch/module-LOADED.test that uses xfs and not nfsd (see the rhel-8.10 / rhel-10.2 versions). For whatever reason we didn't make that change in prior rhel-9 rebases
  • Likewise, for some reason the rhel-9 tests dropped new-function.patch and symvers-disagreement-FAIL.patch somewhere along the way. Can you see if those can be added back to the 9.8 suite?

Also pointed to a test-plan branch with --skip-compiler-check argument added to the test.

edit:

Calling only the rhel9.8 tests, since we're not touching anything else for now.

@usebeforefree

Copy link
Copy Markdown
Contributor Author

/packit test

@ykopkova

Copy link
Copy Markdown
Contributor

I am not sure if the solution would be to just pass the --skip-compiler-check like we do in tier2 tests? @ykopkova what do you think?

Yep, use --skip-compiler-check for this one

@usebeforefree

Copy link
Copy Markdown
Contributor Author

/packit test

@usebeforefree

Copy link
Copy Markdown
Contributor Author

@ykopkova
ykopkova self-requested a review October 30, 2026 13:37

@ykopkova ykopkova left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM, Thank you, Marko!

Added rebased kernel-5.14.0-687.5.1.el9_8 tests for rhel-9.8 and invoked them through
packit.

Signed-off-by: Marko Kostić <mkostic@redhat.com>
@usebeforefree
usebeforefree merged commit 6f50a78 into dynup:master Oct 30, 2026
3 of 32 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants