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 ASIC namespace ID to core filename, if any #1091

Merged
merged 1 commit into from
Sep 3, 2020

Conversation

renukamanavalan
Copy link
Contributor

@renukamanavalan renukamanavalan commented Sep 2, 2020

- What I did
Look for namespace of this process in crashing process's environ file.If any, add it to core filename as
< binary name >.< timestamp >.< local pid inside container >.< ns > .core.gz.
e.g. orchagent.1599073263.45.5.core.gz <-- 5 is the asic namespace.

For processes that don't have asic namespace, there is no change.
e.g. snmpd.1599073245.27.core.gz

The /etc/sysctl.conf kernel.core_pattern is set to pass pid in host
as last arg.

- How I did it

- How to verify it
Crash a process running in a namespace and another process that is not running in a namepsace and verify, the core filenames.

- Previous command output (if the output of a command-line utility has changed)

- New command output (if the output of a command-line utility has changed)

If any, add it to core filename as
<bnary name>.<timestamp>.<local pid inside container>.<ns>.core.gz.
e.g. orchagent.1599073263.45.5.core.gz  <-- 5 is the asic namespace.

For processes that don't have asic namespace, there is no change.
e.g. snmpd.1599073245.27.core.gz

The /etc/sysctl.conf kernel.core_pattern is set to pass `pid in host`
as last arg.
@lguohan
Copy link
Contributor

lguohan commented Sep 2, 2020

retest this please

@renukamanavalan
Copy link
Contributor Author

retest this please.

@renukamanavalan renukamanavalan merged commit 5aa3e88 into sonic-net:master Sep 3, 2020
abdosi pushed a commit that referenced this pull request Sep 3, 2020
Look for namespace of this process in crashing process's environ file.If any, add it to core filename as
< binary name >.< timestamp >.< local pid inside container >.< ns > .core.gz.
e.g. orchagent.1599073263.45.5.core.gz <-- 5 is the asic namespace.

For processes that don't have asic namespace, there is no change.
e.g. snmpd.1599073245.27.core.gz

The /etc/sysctl.conf kernel.core_pattern is set to pass pid in host
as last arg.
stepanblyschak pushed a commit to stepanblyschak/sonic-utilities that referenced this pull request Apr 28, 2022
Fix error when running 'show lldp table' or 'show lldp neighbor'
through SSH command. (sonic-net#1067)
[201911]: Multi asic show interface support (sonic-net#1070)
[counterpoll] Add new FC group for port buffer drop counters
(sonic-net#1024)
[201911] show interface portchannel support for Multi ASIC
(sonic-net#1071)
Fix a typo in mellanox_buffer_migrator (sonic-net#1090)
Fix pfcwd stats crash with invalid queue name (sonic-net#1077)
[PFCWD] Fix issue with "pfcwd show stats" command during SONiC init
(sonic-net#1018)
enable watchdog before running platform specific reboot plugin
(sonic-net#1037)
Add namespace of the process in the coredump filename. (sonic-net#1091)
@renukamanavalan renukamanavalan deleted the core_with_ns branch September 16, 2022 17:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants