Skip to content

Add kernel message on integration test failure#1129

Closed
joshjms wants to merge 1 commit into
vhive-serverless:mainfrom
joshjms:add-dmesg
Closed

Add kernel message on integration test failure#1129
joshjms wants to merge 1 commit into
vhive-serverless:mainfrom
joshjms:add-dmesg

Conversation

@joshjms

@joshjms joshjms commented Aug 21, 2025

Copy link
Copy Markdown

Summary

In this PR, running separate instances of firecracker-containerd also produces the same flake. Adding kernel message logging on failure might help us debug the cause.

Ref: #1113

Implementation Notes ⚒️

  • Briefly outline the overall technical solution. If necessary, identify talking points where the reviewer's attention should be drawn to.

External Dependencies 🍀

Breaking API Changes ⚠️

Simply specify none (N/A) if not applicable.

@joshjms joshjms force-pushed the add-dmesg branch 2 times, most recently from a7eb8ec to 0bf90d3 Compare August 21, 2025 02:08
@joshjms

joshjms commented Aug 21, 2025

Copy link
Copy Markdown
Author

Hmm, the kernel messages don't seem to help.

@joshjms joshjms marked this pull request as draft August 21, 2025 02:58
Signed-off-by: joshjms <joshjms1607@gmail.com>
@joshjms joshjms marked this pull request as ready for review August 21, 2025 10:07
@joshjms

joshjms commented Aug 21, 2025

Copy link
Copy Markdown
Author

I think that even if this doesn't help for the current flake issue, it might help us debug for future errors.

cc @leokondrashov

@leokondrashov

Copy link
Copy Markdown
Contributor

Kernel messages are unlikely to be helpful; it would be more appropriate to collect Firecracker/containerd/vhive logs.

@joshjms

joshjms commented Aug 27, 2025

Copy link
Copy Markdown
Author

I agree, although I think that there will be no harm done with including the kernel messages to debug containerd for a more comprehensive logging. This is also used in containerd's CI.

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