Skip to content

Log list of coredumps with coredumpctl#8

Merged
pothos merged 2 commits intoflatcar-masterfrom
kai/add-coredump-list
Apr 6, 2020
Merged

Log list of coredumps with coredumpctl#8
pothos merged 2 commits intoflatcar-masterfrom
kai/add-coredump-list

Conversation

@pothos
Copy link
Copy Markdown
Member

@pothos pothos commented Apr 3, 2020

That a program crashed is not visible in the mayday tar ball unless the
kernel log is searched for it and users may miss that a program crashed.

Include the output of coredumpctl list in the tar ball so that the user
can see which programs crashed and whether a core dump is still available
on the machine for further inspection.

How to use/test

make
./bin/mayday -o o.tar.gz -c default.json
tar -axf o.tar.gz --wildcards '*coredumpctl_list*' -O
TIME                            PID   UID   GID SIG COREFILE  EXE
Wed 2020-03-11 14:11:43 CET  3981385  1000  1000   6 missing /your/prog

That a program crashed is not visible in the mayday tar ball unless the
kernel log is searched for it and users may miss that a program crashed.

Include the output of `coredumpctl list` in the tar ball so that the user
can see which programs crashed and whether a core dump is still available
on the machine for further inspection.
@pothos pothos requested a review from a team April 3, 2020 16:44
Copy link
Copy Markdown

@margamanterola margamanterola left a comment

Choose a reason for hiding this comment

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

This LGTM. But, in order to get this to work I needed to:

  1. Install systemd-coredump
  2. Run mayday with sudo (at least in Debian, coredumpctl doesn't really do much without root rights).

I tried to find if there's somewhere where tool dependencies are listed or where things that require root rights are documented but I didn't find either... Is this info not stored anywhere?

@pothos
Copy link
Copy Markdown
Member Author

pothos commented Apr 6, 2020

Yes, the instructions in the readme are very basic. I'll add something there.

@pothos pothos merged commit 73e4d2f into flatcar-master Apr 6, 2020
@pothos pothos deleted the kai/add-coredump-list branch April 6, 2020 13:08
pothos added a commit to flatcar-archive/coreos-overlay that referenced this pull request Apr 6, 2020
This pulls in flatcar/mayday#8
to include "coredumpctl list" in the tar ball.
pothos added a commit to flatcar-archive/coreos-overlay that referenced this pull request Apr 6, 2020
This pulls in flatcar/mayday#8
to include "coredumpctl list" in the tar ball.
pothos added a commit to flatcar-archive/coreos-overlay that referenced this pull request Apr 6, 2020
This pulls in flatcar/mayday#8
to include "coredumpctl list" in the tar ball.
pothos added a commit to flatcar-archive/coreos-overlay that referenced this pull request Apr 6, 2020
This pulls in flatcar/mayday#8
to include "coredumpctl list" in the tar ball.
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