Skip to content

common.sh: redirect verbose debug output to file#3783

Open
t-lo wants to merge 1 commit intomainfrom
t-lo/debug-output-to-file
Open

common.sh: redirect verbose debug output to file#3783
t-lo wants to merge 1 commit intomainfrom
t-lo/debug-output-to-file

Conversation

@t-lo
Copy link
Member

@t-lo t-lo commented Mar 5, 2026

common.sh produces very verbose debug output on failure. This leads to the actual error being masked as it is pushed out of the scrollback buffer by many pages of dmesg.

This change redirects this output to a FAILURE log file which is timestapmed.

common.sh produces *very* verbose debug output on failure. This leads to
the actual error being masked as it is pushed out of the scrollback
buffer by many pages of dmesg.

This change redirects this output to a FAILURE log file which is
timestapmed.

Signed-off-by: Thilo Fromm <thilofromm@microsoft.com>
@t-lo t-lo requested a review from a team as a code owner March 5, 2026 14:08
@t-lo t-lo temporarily deployed to development March 5, 2026 14:09 — with GitHub Actions Inactive
@chewi
Copy link
Contributor

chewi commented Mar 5, 2026

I like this a lot, but can we see this file in CI?

@t-lo
Copy link
Member Author

t-lo commented Mar 5, 2026

I like this a lot, but can we see this file in CI?

CI could chose to cat FAILURE-* after a failed run, or to archive the file as an artifact. In most cases though I doubt that mount, df, or dmesg output will contribute significantly to debugging.

@chewi
Copy link
Contributor

chewi commented Mar 5, 2026

CI could chose to cat FAILURE-* after a failed run, or to archive the file as an artifact. In most cases though I doubt that mount, df, or dmesg output will contribute significantly to debugging.

Yeah, I've never found it useful. I don't want to see it in CI normally. If it's not exposed as an artifact, I wonder if there's any point having this data at all.

@krnowak
Copy link
Member

krnowak commented Mar 5, 2026

I have this commit in my selinux-coverage branch, but I was wondering if this is the way to go. There, I'm pushing the build logs to bincache, but I'd need a reassurance that this is a safe thing to do. The commit in question is 8f02179.

@krnowak
Copy link
Member

krnowak commented Mar 5, 2026

I think that the dmesg, df and mount were added by me because at some point we had a rather mysterious weird failures in CI, which I think just went away by themselves (probably got a Flatcar update or something). So maybe an option would be to just drop those.

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