-
Notifications
You must be signed in to change notification settings - Fork 24
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
ROX-20541: Temporarily exclude kernels 6.6 from build #1395
Conversation
This issue has been fixed and the TODO statement is misleading here.
Skipping CI for Draft Pull Request. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
kernel-modules/BLOCKLIST
Outdated
~6\.[3-9]\.\d+.* ~2\.[2-5]\.\d+(?:-rc\d+)? | ||
# TODO(ROX-20541) - 6.6 kernel compilation errors | ||
6.6.* * |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You don't need the second parameter as a *
, since the script that processes this file assumes all missing parts are *
. https://github.com/stackrox/collector/blob/24b43e3f04594741fc92957662a4eae41bc53fdb/kernel-modules/build/apply-blocklist.py#L41C8-L41C8
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
And it is explained in the comment header... Removed it !
Description
A patch was introduced to solve build issues on earlier RC versions of 6.6, which is now causing
build issues. It has been mostly reverted upstream.
Block temporarily those kernels while we work on pulling upstream.
Testing
This does not touch the Collector code, we just want to make the drivers build happy.