phosphor-debug-collector, branch master

build: pkg anti-pattern: use defaults

2019-02-13T23:14:00+00:00

Use the defaults in the pkg check where the default error message is sufficient to identify which package is missing. Change-Id: If14318cbb16f40cac4bd94b5b7d81cf9d3aacfdc Signed-off-by: Patrick Venture

Specify plugins to be invoked for chekstop error

2019-01-30T07:04:46+00:00

Set the config value for the scripts that need to be invoked during dump collection for checkstop error. Checkstop dump enum type has value 4. Change-Id: Ic75fa0f800ed459b8e493b191eec18f144664655 Signed-off-by: Marri Devender Rao

Refactor to pass errors to watch through config YAML

2019-01-22T21:57:01+00:00

Add errors to watch for through error YAML file Add support for checkstop error type Tested: Generating dumps for elog, core, checkstop type errors. Change-Id: Idd00ace2e3d0c472a74ec142e6d150e55e843a6f Signed-off-by: Marri Devender Rao

Add phosphor-hostlogger plugin to dreport

2018-11-14T15:39:25+00:00

phosphor-hostlogger project stores host's console output data, such as boot logs or Linux kernel messages printed to the system console. https://github.com/openbmc/phosphor-hostlogger Tested: Run dreport inside the BMC console, the result dump archive must contains host console logs. Change-Id: Icc45888a43b42de78c93aa0bc0ba7a295f0a76e3 Signed-off-by: Artem Senichev Tested-by: Artem Senichev

MAINTAINERS: Updated reviewers

2018-10-23T13:35:19+00:00

Change-Id: Ide3f7c1e4e96ae07a99055a2a14e5d376e51d6b4 Signed-off-by: Jayanth Othayoth

Enable clang code format

2018-10-18T20:33:14+00:00

Change-Id: Ib640ef7cea99ff505965182ec9900a2d53a8986e Signed-off-by: Jayanth Othayoth Signed-off-by: Brad Bishop

internal: Add missing Manager forward declaration

2018-10-18T20:33:14+00:00

This allows us to not depend on specific include ordering. Change-Id: I4d2a552eb4b4d5dcf0aaf9ad9be3b86265ca469a Signed-off-by: Brad Bishop

phosphor-debug-collector: use c++17

2018-10-04T17:17:16+00:00

Update configure.ac to choose the c++17 standard Change-Id: I3636e54a746c9f9312f5d01ac83113204ed78a46 Signed-off-by: Vernon Mauery

elog_watch: Fix parsing of elog add requests

2018-06-12T23:53:21+00:00

Requests come in the form "oa{sa{sv}}". However, the way sdbusplus was interpreting the type of our message "a{oa{sa{sv}}}" since tuples are not allowed to consume multiple arguments during the read call as that would be ambiguous. This fixes the type issues. Prior to the change to sdbusplus that introduces error handling for the read calls, the sd_bus_message_{enter,exit}_container were failing during the read on the pair. Luckily this produces the expected result for the read and our old code was "working". This also cleans up an unnecessary string move. Tested: Builds and no longer produces errors on zaius when elogs are added. Change-Id: Ifc5394f3f361e8932c939376bd0bf5b4e3ca589c Signed-off-by: William A. Kennington III

Add error handling for message parsing

2018-05-29T20:36:39+00:00

Some of the signals we try and parse for error logs are not structured in the format we expect when parsing. When error logging is enabled in sdbusplus this causes the phosphor-dump-manager to crash. Fix this crashing by logging errors and ignoring the bad signals. Change-Id: Ieadcb7e95f622005382f3c4957bf1535e3bb1ef9 Signed-off-by: William A. Kennington III