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

Add 6.8 kernel to CI build. Update patch versions of existing kernels #1995

Merged

Conversation

ddelnano
Copy link
Member

Summary: Add 6.8 kernel to CI build. Update patch versions of existing kernels

In addition to updating the kernels, I've removed the use of the gs://pixie-dev-public/kernel-build bucket. Please see the changes commit by commit to see them independently.

Relevant Issues: N/A

Type of change: /kind cleanup

Test Plan: bpf-build-all-kernels build should succeed

…ersion

Signed-off-by: Dom Del Nano <ddelnano@gmail.com>
…artifacts

Signed-off-by: Dom Del Nano <ddelnano@gmail.com>
Signed-off-by: Dom Del Nano <ddelnano@gmail.com>
@ddelnano ddelnano requested review from a team as code owners August 26, 2024 22:45
…8 and later kernels

Signed-off-by: Dom Del Nano <ddelnano@gmail.com>
@ddelnano ddelnano force-pushed the ddelnano/add-6.8-kernel-update-patch-versions branch from 32379df to a9a274f Compare August 27, 2024 14:25
Signed-off-by: Dom Del Nano <ddelnano@gmail.com>
Copy link
Member

@vihangm vihangm left a comment

Choose a reason for hiding this comment

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

Build/CI changes lgtm.
@oazizi000 can you take a quick look at the bpf test changes?

@ddelnano ddelnano merged commit a62f9a9 into pixie-io:main Aug 29, 2024
45 checks passed
@ddelnano ddelnano deleted the ddelnano/add-6.8-kernel-update-patch-versions branch August 29, 2024 16:54
ddelnano added a commit that referenced this pull request Sep 3, 2024
Summary: Update reference to old kernel label since most recent update

The 6.1.8 kernel was updated to the latest patch version in #1995.

Relevant Issues: N/A

Type of change: /kind cleanup

Test Plan: Build without specifying
`--//bazel/test_runners/qemu_with_kernel:kernel_version` succeeds

Signed-off-by: Dom Del Nano <ddelnano@gmail.com>
ddelnano added a commit that referenced this pull request Sep 4, 2024
Summary: Update GitHub matrix to use correct 6.1.x kernel

Builds after #1995 would fail since GitHub would attempt to run the
6.1.8 kernel build, which bazel no longer understands. This also updates
the qemu test runner kernel that was incorrectly updated in #1999. I
thought I had exercised that default value with my testing, but it
didn't trigger that default value.

Relevant Issues: N/A

Type of change: /kind bugfix

Test Plan: GitHub build on #2002 which pulls in the matrix change is no
longer failing
ddelnano added a commit to ddelnano/pixie that referenced this pull request Sep 23, 2024
ddelnano added a commit to ddelnano/pixie that referenced this pull request Sep 23, 2024
…io#1999)

Summary: Update reference to old kernel label since most recent update

The 6.1.8 kernel was updated to the latest patch version in pixie-io#1995.

Relevant Issues: N/A

Type of change: /kind cleanup

Test Plan: Build without specifying
`--//bazel/test_runners/qemu_with_kernel:kernel_version` succeeds

Signed-off-by: Dom Del Nano <ddelnano@gmail.com>
GitOrigin-RevId: df3992f
ddelnano added a commit to ddelnano/pixie that referenced this pull request Sep 23, 2024
Summary: Update GitHub matrix to use correct 6.1.x kernel

Builds after pixie-io#1995 would fail since GitHub would attempt to run the
6.1.8 kernel build, which bazel no longer understands. This also updates
the qemu test runner kernel that was incorrectly updated in pixie-io#1999. I
thought I had exercised that default value with my testing, but it
didn't trigger that default value.

Relevant Issues: N/A

Type of change: /kind bugfix

Test Plan: GitHub build on pixie-io#2002 which pulls in the matrix change is no
longer failing

GitOrigin-RevId: 9ae53a8
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