Update error type for runtime error

In case there is a generic runtime exception thrown by VPD-Manager,
"FirmwareError" error type needs to be used to call out BMC firmware.

This is required as cur

Update error type for runtime error

In case there is a generic runtime exception thrown by VPD-Manager,
"FirmwareError" error type needs to be used to call out BMC firmware.

This is required as current implementation does not have any call out
in case of runtime exceptions and phosphor-logging downgrades the
severity in case of no call out.

Change-Id: I3ccd2f2e9cd947d4334f6676d955a320828320c8
Signed-off-by: Sunny Srivastava <sunnsr25@in.ibm.com>

show more ...

Infrastructure to support single FAB feature

This commit implements infrastructure to support single FAB feature.

Single FAB feature updates the IM value of the system planar or creates
PEL based o

Infrastructure to support single FAB feature

This commit implements infrastructure to support single FAB feature.

Single FAB feature updates the IM value of the system planar or creates
PEL based on the IM value read from the planar and cache, considering
the system mode and type.

Change-Id: I77fe927de9e37e0ba70e0f4d661a25c586911433
Signed-off-by: Rekha Aparna <vrekhaaparna@ibm.com>

show more ...

Move to sync PEL creation

Creating sync PELs by default. Async PEL should be created only when
required.

Change-Id: I73ace657a4e35b3483b65bd70d4a18bbe67883a1
Signed-off-by: Sunny Srivastava <sunnsr

Move to sync PEL creation

Creating sync PELs by default. Async PEL should be created only when
required.

Change-Id: I73ace657a4e35b3483b65bd70d4a18bbe67883a1
Signed-off-by: Sunny Srivastava <sunnsr25@in.ibm.com>

show more ...

meson: reformat with meson formatter

Apply the `meson format` results.

Change-Id: I0a8cf196acaeef29e0520a0ff5a0ed9349737e06
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>

Fix string comparison by trimming newline char

Fixed an issue where the API isFieldModeEnabled always returns true due
to newline character at the end of the string returned from executeCmd
API. Tri

Fix string comparison by trimming newline char

Fixed an issue where the API isFieldModeEnabled always returns true due
to newline character at the end of the string returned from executeCmd
API. Trimmed the newline before comparison to ensure correct string
comparison.

Change-Id: I1d4aab72d062a6bfd2a6e5c21643a749366bbed5
Signed-off-by: Rekha Aparna <vrekhaaparna@ibm.com>

show more ...

API to update planar IM to P11 series

This commit implements an API to update the IM value in planar EEPROM to
P11 series(6000x).

Change-Id: Iedfcba0eee278ee13bfdf8ef26eb1a894424e743
Signed-off-by:

API to update planar IM to P11 series

This commit implements an API to update the IM value in planar EEPROM to
P11 series(6000x).

Change-Id: Iedfcba0eee278ee13bfdf8ef26eb1a894424e743
Signed-off-by: Souvik Roy <souvikroyofficial10@gmail.com>

show more ...

API to check is field mode enabled

This commit implements an API to check if field mode is enabled.

Change-Id: Icb2ae1df2588e5cb3fd81e30bf739745de5fbb42
Signed-off-by: Rekha Aparna <vrekhaaparna@ib

API to check is field mode enabled

This commit implements an API to check if field mode is enabled.

Change-Id: Icb2ae1df2588e5cb3fd81e30bf739745de5fbb42
Signed-off-by: Rekha Aparna <vrekhaaparna@ibm.com>

show more ...

API to detect powerVS image

PowerVS systems mandate running of image with specific prefix.
The commit adds API to check for those prefix and confirm if it is a
powerVS image or not.

Change-Id: Ie1c

API to detect powerVS image

PowerVS systems mandate running of image with specific prefix.
The commit adds API to check for those prefix and confirm if it is a
powerVS image or not.

Change-Id: Ie1cb012d57068d77fbfeeb2cbd6e4e54a542d32f
Signed-off-by: Sunny Srivastava <sunnsr25@in.ibm.com>

show more ...

PEL to log failure in detecting system VPD

To capture any failure while initiating timer to detect system VPD on
DBus or in the initial stage of triggering collection from JSON. PEL is
logged with c

PEL to log failure in detecting system VPD

To capture any failure while initiating timer to detect system VPD on
DBus or in the initial stage of triggering collection from JSON. PEL is
logged with critical severity and required error type.

Change-Id: I5af76afb29cf61ee0e4e6d713c8c83be586af6df
Signed-off-by: Sunny Srivastava <sunnsr25@in.ibm.com>

show more ...

API to update IM value on system planar

This commit implements code to update IM value on system planar
EEPROM path.

Change-Id: Ib4e8570d1ad5c86dd4573aca9882031282342a88
Signed-off-by: Rekha Aparna

API to update IM value on system planar

This commit implements code to update IM value on system planar
EEPROM path.

Change-Id: Ib4e8570d1ad5c86dd4573aca9882031282342a88
Signed-off-by: Rekha Aparna <vrekhaaparna@ibm.com>

show more ...

Update FRU part number for PowerVS systems

Error log makes use of FRU part number instead of part number in the
call outs.
Hence the code is updated to change FRU part number instead of part
number

Update FRU part number for PowerVS systems

Error log makes use of FRU part number instead of part number in the
call outs.
Hence the code is updated to change FRU part number instead of part
number in case of PowerVS systems.
Similarly, spare part number of Decorator.Asset interface is updated
instead of part number.

Change-Id: I31b57e23863b250f7fb036b23c7a192fc1bbec5c
Signed-off-by: Sunny Srivastava <sunnsr25@in.ibm.com>

show more ...

API to read IM value from system planar

This commit implements an API to read IM value from system planar using
the IM keyword offset value.

Change-Id: I69f48895aa814e31a4ce23bcf4f8aa954449c969
Sig

API to read IM value from system planar

This commit implements an API to read IM value from system planar using
the IM keyword offset value.

Change-Id: I69f48895aa814e31a4ce23bcf4f8aa954449c969
Signed-off-by: Anupama B R <anupama.b.r1@ibm.com>

show more ...

Update error handling in main file

Main file need not handle each exception separately. Instead a common
PEL needs to be logged based on the type of exception received with
critical severity as the

Update error handling in main file

Main file need not handle each exception separately. Instead a common
PEL needs to be logged based on the type of exception received with
critical severity as the service failed to start.

Change-Id: Ic432a2ab2caf6dac70c376219fce6597e7782309
Signed-off-by: Sunny Srivastava <sunnsr25@in.ibm.com>

show more ...

Handling of PELs while collecting system VPD

The commit updates handling of PEL in the path of system VPD collection.
Any error in the flow needs to be handled at worker end and should not
be propag

Handling of PELs while collecting system VPD

The commit updates handling of PEL in the path of system VPD collection.
Any error in the flow needs to be handled at worker end and should not
be propagated to Manager preventing service from failing.

Change-Id: Ia06e5a4debe2fd3f8541d7975509449e86b23fcc
Signed-off-by: Sunny Srivastava <sunnsr25@in.ibm.com>

show more ...

Error handling for GpioMonitor class

In case of any exception in the construction of GpioMonitor class, VPD
Manager need not skip rest of the processings.
Instead, code has been modified to handle t

Error handling for GpioMonitor class

In case of any exception in the construction of GpioMonitor class, VPD
Manager need not skip rest of the processings.
Instead, code has been modified to handle the exception locally by
logging appropriate PEL and allowing VPD-Manager to continue with rest
of the processings.

Change-Id: Iccf421da175ec264a8aff149c360ba7ad105fbe9
Signed-off-by: Sunny Srivastava <sunnsr25@in.ibm.com>

show more ...

Check and update JSON symlink on every reboot

Symlink can exist whereas the underneath IM value can change in some
scenarios.
In such cases, even when the symlink is present, the file being pointed

Check and update JSON symlink on every reboot

Symlink can exist whereas the underneath IM value can change in some
scenarios.
In such cases, even when the symlink is present, the file being pointed
via symlink needs to be updated.

The commit, on every reboot, checks and updates the existing symlink if
required.

Change-Id: I850bfaba41c28ca30443aa09263c712e3a4e4af9
Signed-off-by: Sunny Srivastava <sunnsr25@in.ibm.com>

show more ...

Move single FAB API under the separate class

SingleFab class provides functionalities to support single FAB feature.
This commit moves the single FAB related API under SingleFab class.

Change-Id: I

Move single FAB API under the separate class

SingleFab class provides functionalities to support single FAB feature.
This commit moves the single FAB related API under SingleFab class.

Change-Id: I8d55cd2134f5812f83f5fc5033df097ab1b1f8f2
Signed-off-by: Anupama B R <anupama.b.r1@ibm.com>

show more ...

Update PN under asset interface

In case of powerVS configuration, PN is updated for some of the FRUs.
The same needs to be reflected under Asset interface to keep the values
in sync.

The commit imp

Update PN under asset interface

In case of powerVS configuration, PN is updated for some of the FRUs.
The same needs to be reflected under Asset interface to keep the values
in sync.

The commit implements changes to explicitly update the Asset interface
under PIM once the data has been successfully updated in EEPROM.

Change-Id: I945520bf7669e44d98202f903bd59efdea880a75
Signed-off-by: Sunny Srivastava <sunnsr25@in.ibm.com>

show more ...

API to get error type and error msg

Error type to be set should be strictly dependent on the type of
exception being caught. So that correct interface can be called from
message registry while loggi

API to get error type and error msg

Error type to be set should be strictly dependent on the type of
exception being caught. So that correct interface can be called from
message registry while logging a PEL.
Also, the error message logged should have a uniform format across
different kind of errors.

The commit implements API to get the pre-defined error type and message
format for a given exception.
The user needs to call this API and need not set error type on wish.

Change-Id: I3aeddd2986f12ff18b3a061533490db20ebede34
Signed-off-by: Sunny Srivastava <sunnsr25@in.ibm.com>

show more ...

Fix for AMM gets disabled after system reboot

Enable Active Memory Mirroring(AMM) mode in GUI memory tab under
‘Resource Management’. On reboot of the system and once system reaches
ready state, AMM

Fix for AMM gets disabled after system reboot

Enable Active Memory Mirroring(AMM) mode in GUI memory tab under
‘Resource Management’. On reboot of the system and once system reaches
ready state, AMM mode in the GUI shows as disabled.

Bios handler in VPD manager caused this issue as it’s comparing AMM VPD
as character value with wrong int value.

This commit implements changes to fix the above issue.

Change-Id: I758ede97e17ecf76fbd4ee05287172ce1534201c
Signed-off-by: Anupama B R <anupama.b.r1@ibm.com>

show more ...

Refactor exception handling for json_utility APIs

This commit adds code to handle exceptions for some of the json_utility
APIs. Each API will have output returned in case of error based on the
retur

Refactor exception handling for json_utility APIs

This commit adds code to handle exceptions for some of the json_utility
APIs. Each API will have output returned in case of error based on the
return type of the API instead of throwing an exception.

Also updated the caller of the json_utility APIs to handle the return
value.

Change-Id: I154a90d5edbbca7441f75a153b908914de83471d
Signed-off-by: RekhaAparna01 <vrekhaaparna@ibm.com>

show more ...

Refactor get keyword value API exception handling

This commit refactors vpd specific utility API used to read value of a
keyword,in order to handle any exceptions thrown by it locally. All
utility m

Refactor get keyword value API exception handling

This commit refactors vpd specific utility API used to read value of a
keyword,in order to handle any exceptions thrown by it locally. All
utility methods should handle exceptions locally and log a journal log
in case of failure. The caller of the utility APIs should check the
return value to detect success/failure.

This commit also changes the caller of this API throughout the repo, in
order to check the return value.

Test:

```
- Install bitbaked image on Everest system
- After BMC boots, BMC should reach Ready state
- Check vpd-manager service status, should be active(running)
- Check no restarts in vpd-manager service
- Check vpd-manager "CollectionStatus" = "Completed"
- Check extra interfaces are processed properly
- Check backup restore working properly
```

Change-Id: I965313f512553ed5d39373dd871754e1a8fed5f3
Signed-off-by: Souvik Roy <souvikroyofficial10@gmail.com>

show more ...

API to get error info based on the exception

This commit implements an event logger API to extract the error info
based on the exception type.

Change-Id: I132306a6d3127253435e53f61328843fa8f1be24
S

API to get error info based on the exception

This commit implements an event logger API to extract the error info
based on the exception type.

Change-Id: I132306a6d3127253435e53f61328843fa8f1be24
Signed-off-by: Anupama B R <anupama.b.r1@ibm.com>

show more ...

API to read IM value from persisted location

This commit implements API to read IM value from PIM persisted
location.

Change-Id: I9cd4f6ed9ded2dd4f42c65902509f7bea9e15849
Signed-off-by: Anupama B R

API to read IM value from persisted location

This commit implements API to read IM value from PIM persisted
location.

Change-Id: I9cd4f6ed9ded2dd4f42c65902509f7bea9e15849
Signed-off-by: Anupama B R <anupama.b.r1@ibm.com>

show more ...

vpd-tool force reset command implementation

This commit implements force reset command option in vpd-tool
application.

The commit implements clearing of Phosphor Inventory Manager’s (PIM)
persisted

vpd-tool force reset command implementation

This commit implements force reset command option in vpd-tool
application.

The commit implements clearing of Phosphor Inventory Manager’s (PIM)
persisted data, restarts the PIM and VPD manager services. In turn VPD
manager triggers the VPD collection for the FRUs listed in the system
config JSON.

Output:
```
Create mismatch value, by updating keyword value only on DBus.
busctl call xyz.openbmc_project.Inventory.Manager /xyz/openbmc_project/inventory xyz.openbmc_project.Inventory.Manager Notify a{oa{sa{sv}}} 1 "/system/chassis/motherboard" 1 "com.ibm.ipzvpd.VSYS" 1 "BR" ay 2 0x32 0x33

Data On Hardware:
root@rainvpdteam:/tmp# vpd-tool -O /sys/bus/i2c/drivers/at24/8-0050/eeprom -R VSYS -K BR -r -H
{
"/sys/bus/i2c/drivers/at24/8-0050/eeprom": {
"BR": "S0"
}
}

Data on DBus:
root@rainvpdteam:/tmp# vpd-tool -O /system/chassis/motherboard -R VSYS -K BR -r
{
"/system/chassis/motherboard": {
"BR": "23"
}
}

On force reset command:
root@rainvpdteam:/tmp# ./vpd-tool -f
Stopping vpd-manager service.
Restarting xyz.openbmc_project.Inventory.Manager service.
Starting vpd-manager service.

After force reset command, DBus got updated with hardware data:
root@rainvpdteam:/tmp# vpd-tool -O /system/chassis/motherboard -R VSYS -K BR -r
{
"/system/chassis/motherboard": {
"BR": "S0"
}
}

root@rainvpdteam:/tmp# vpd-tool -O /sys/bus/i2c/drivers/at24/8-0050/eeprom -R VSYS -K BR -r -H
{
"/sys/bus/i2c/drivers/at24/8-0050/eeprom": {
"BR": "S0"
}
}
```

Change-Id: Ic236265817d793892902f63835667167365b8cdc
Signed-off-by: Anupama B R <anupama.b.r1@ibm.com>

show more ...