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

ci: In "Tetragon Go Test" add vmlinux in artifact when test fails #3526

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ScriptSathi
Copy link
Contributor

@ScriptSathi ScriptSathi commented Mar 20, 2025

To help debugging between arch, I think it is a great idea to upload the vmlinux file when this test fails

This will help fixing #3492

Motivation

TestSpecs uses ksys_lseek symbol which is inlined (I think) on ARM but not on x86_64. If we upload the vmlinux, we can load the BTF spec and check what hooks are shared
#3414 (comment)

Description

Kprobes are not exactly the same between ARM and x86_64. To help understand why some tests fail on one architecture and not the other, it can be helpful to use the vmlinux file directly from the CI instead of deploying a specific VM with the same architecture for debugging.

Kprobe are not extactly the same between arm and x86_64. So to help
understand why some tests fails on one side and not the other, it can be
helpful to use the vmlinux directly from the CI instead of deploying a
specific VM with the same architecture to debug

Signed-off-by: Tristan d'Audibert <[email protected]>
@ScriptSathi ScriptSathi force-pushed the pr/ScriptSathi/add-vmlinux-workflow branch from 6ecf9a5 to 5b40d47 Compare March 23, 2025 20:23
Copy link

netlify bot commented Mar 23, 2025

Deploy Preview for tetragon ready!

Name Link
🔨 Latest commit 5b40d47
🔍 Latest deploy log https://app.netlify.com/sites/tetragon/deploys/67e06da824349b0008475877
😎 Deploy Preview https://deploy-preview-3526--tetragon.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@ScriptSathi ScriptSathi requested a review from kkourt March 24, 2025 09:47
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