Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test/integration/rhel-8.9: integration testing for rhel-8.9 #1366

Merged

Conversation

ryanbsull
Copy link
Contributor

No description provided.

@joe-lawrence
Copy link
Contributor

A few small things:

  • This PR is missing the *.test files in test/integration/rhel-8.9/. You can copy them from the test/integration/rhel-8.8/ directory (note that module-LOADED.test may need the updated linux-6.2 or 9.x version)
  • symvers-disagreement-FAIL.patch - somehow Julien's commit message at the top of the .path file got dropped. Can you copy/paste from and existing version.
  • Commit message
    • add the GA kernel version "kernel-4.18.0-513.5.1.el8_9" in the commit msg for reference
    • module.patch looks very different from 8.8, I think we had to use the linux-6.2 or 9.x version. Can you just add a quick note in the commit msg that we updated this patch/test. Otherwise, the rest appear to be a straight rebase.

@ryanbsull ryanbsull force-pushed the integration-tests-rhel-8.9-final branch 2 times, most recently from 119aa47 to bf7a116 Compare December 4, 2023 17:51
@joe-lawrence joe-lawrence force-pushed the integration-tests-rhel-8.9-final branch from bf7a116 to d37c28f Compare December 5, 2023 16:58
Provides integration tests for rhel-8.9 (kernel-4.18.0-513.5.1.el8_9),
note that module.test is from the linux-6.2/ directory and not
rhel-8.8/

Signed-off-by: Ryan Sullivan <[email protected]>
@joe-lawrence joe-lawrence force-pushed the integration-tests-rhel-8.9-final branch from d37c28f to 4520473 Compare December 5, 2023 16:59
@joe-lawrence joe-lawrence merged commit 8513926 into dynup:master Dec 5, 2023
3 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.

2 participants