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

Build fixes for linux 6.7 #1492

Merged
merged 7 commits into from
Mar 13, 2024
Merged

Build fixes for linux 6.7 #1492

merged 7 commits into from
Mar 13, 2024

Conversation

ovalenti
Copy link
Contributor

@ovalenti ovalenti commented Jan 9, 2024

Description

Some changes in the inode structure introduced in 6.7. We cherry-pick the fix, and block previous probe
versions (2.6&2.7) on 6.7 and up.

There is an issue with the size of some programs in the ebpf probe as well.

Those changes should end-up in the 3.18 branch, as we shall overwrite them in master with the next "big falco pull"

Checklist

  • Investigated and inspected CI test results

Copy link

openshift-ci bot commented Jan 9, 2024

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

Stringy
Stringy previously approved these changes Jan 9, 2024
@ovalenti ovalenti marked this pull request as ready for review January 10, 2024 06:48
@ovalenti ovalenti requested a review from a team as a code owner January 10, 2024 06:48
@Stringy
Copy link
Collaborator

Stringy commented Jan 11, 2024

Just a heads up that I blocked 6.7 along with some other fixes here ac5a735 and this PR will need to be rebased and 6.7 unblocked when it comes to merging

@ovalenti ovalenti closed this Feb 27, 2024
@ovalenti ovalenti reopened this Mar 4, 2024
@ovalenti ovalenti marked this pull request as draft March 4, 2024 11:00
@ovalenti ovalenti closed this Mar 4, 2024
@ovalenti ovalenti reopened this Mar 5, 2024
@Stringy Stringy dismissed their stale review March 5, 2024 15:20

work has restarted on this change

@ovalenti ovalenti force-pushed the fix/kernel_6.7 branch 2 times, most recently from 325ba80 to 2425333 Compare March 6, 2024 15:39
@ovalenti ovalenti changed the base branch from master to giles/new-drivers-branch March 6, 2024 15:40
@ovalenti ovalenti force-pushed the fix/kernel_6.7 branch 3 times, most recently from e41481b to 467e763 Compare March 7, 2024 10:39
@ovalenti ovalenti changed the base branch from giles/new-drivers-branch to giles/fix-duplicate-driver-artifacts March 7, 2024 10:41
@ovalenti ovalenti force-pushed the fix/kernel_6.7 branch 3 times, most recently from 9813feb to a09b248 Compare March 7, 2024 14:18
Base automatically changed from giles/fix-duplicate-driver-artifacts to master March 8, 2024 11:33
@ovalenti
Copy link
Contributor Author

Pending PR merge in the falco fork: stackrox/falcosecurity-libs#71

@ovalenti ovalenti marked this pull request as ready for review March 12, 2024 10:43
Copy link
Collaborator

@Molter73 Molter73 left a comment

Choose a reason for hiding this comment

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

LGTM!

@ovalenti ovalenti added the run-multiarch-builds Run steps for non-x86 archs. label Mar 13, 2024
@ovalenti ovalenti merged commit c787a47 into master Mar 13, 2024
184 checks passed
@ovalenti ovalenti deleted the fix/kernel_6.7 branch March 13, 2024 16:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants