Listener update

Updated this class APIs, to use logger::logMessage with placeHolder PEL
instead of using EventLogger::createSyncPel to log a PEL.
This change is required to stream line everything wi

Listener update

Updated this class APIs, to use logger::logMessage with placeHolder PEL
instead of using EventLogger::createSyncPel to log a PEL.
This change is required to stream line everything with logger class.

Test-
- Tested on PST simics
```
```

Change-Id: I0737f8e523c183ea8bdbed549f4424908609c59b
Signed-off-by: Alpana Kumari <alpu8007@gmail.com>

show more ...

bios_handler class update

Updated this class APIs, to use logger::logMessage with placeHolder PEL
instead of using EventLogger::createSyncPel to log a PEL.
This change is required to stream line eve

bios_handler class update

Updated this class APIs, to use logger::logMessage with placeHolder PEL
instead of using EventLogger::createSyncPel to log a PEL.
This change is required to stream line everything with logger class.

Test-
- Tested on PST simics
```
```

Change-Id: I5dc6d96aad95dfa4ec936cc163b118e7b301f333
Signed-off-by: Alpana Kumari <alpu8007@gmail.com>

show more ...

Manager update

Updated this class APIs, to use logger::logMessage with placeHolder PEL
instead of using EventLogger::createSyncPel to log a PEL.
This change is required to stream line everything wit

Manager update

Updated this class APIs, to use logger::logMessage with placeHolder PEL
instead of using EventLogger::createSyncPel to log a PEL.
This change is required to stream line everything with logger class.

Test-
Tested on PST simics.
```
```

Change-Id: I4a5822394a1ff5753a93a50a2bd5c81522c2cf7b
Signed-off-by: Alpana Kumari <alpu8007@gmail.com>

show more ...

ipz_parser update

Updated this class APIs, to use logger::logMessage with placeHolder PEL
instead of using EventLogger::createSyncPel to log a PEL.
This change is required to stream line everything

ipz_parser update

Updated this class APIs, to use logger::logMessage with placeHolder PEL
instead of using EventLogger::createSyncPel to log a PEL.
This change is required to stream line everything with logger class.

Test-
Tested on PST simics.
```
```

Change-Id: I4bc70022df96c84a650b8d901ab1ec6797c49e89
Signed-off-by: Alpana Kumari <alpu8007@gmail.com>

show more ...

ddimm_parser update

Updated this class APIs, to use logger::logMessage with placeHolder PEL
instead of using EventLogger::createSyncPel to log a PEL.
This change is required to stream line everythin

ddimm_parser update

Updated this class APIs, to use logger::logMessage with placeHolder PEL
instead of using EventLogger::createSyncPel to log a PEL.
This change is required to stream line everything with logger class.

Test-
Tested image on PST simics
```
```

Change-Id: I6a943c86ff0fe703eb9e49e00afbd3e86948a6d7
Signed-off-by: Alpana Kumari <alpu8007@gmail.com>

show more ...

parser update

Updated this class APIs, to use logger::logMessage with placeHolder PEL
instead of using EventLogger::createSyncPel to log a PEL.
This change is required to stream line everything with

parser update

Updated this class APIs, to use logger::logMessage with placeHolder PEL
instead of using EventLogger::createSyncPel to log a PEL.
This change is required to stream line everything with logger class.

Test-
Tested on PST simics
```
```

Change-Id: If33a097a99a6f8da07bf3bb2d551df30b1f40866
Signed-off-by: Alpana Kumari <alpu8007@gmail.com>

show more ...

backup_restore update

Updated this class APIs, to use logger::logMessage with placeHolder PEL
instead of using EventLogger::createSyncPel to log a PEL.
This change is required to stream line everyth

backup_restore update

Updated this class APIs, to use logger::logMessage with placeHolder PEL
instead of using EventLogger::createSyncPel to log a PEL.
This change is required to stream line everything with logger class.

Test-
Tested on PST simics
```
```

Change-Id: Ie708bd11c733e5f3deabd18bbbc38985c0e0d0e2
Signed-off-by: Alpana Kumari <alpu8007@gmail.com>

show more ...

single_fab update

Updated this class APIs, to use logger::logMessage with placeHolder PEL
instead of using EventLogger::createSyncPel to log a PEL.
This change is required to stream line everything

single_fab update

Updated this class APIs, to use logger::logMessage with placeHolder PEL
instead of using EventLogger::createSyncPel to log a PEL.
This change is required to stream line everything with logger class.

Test-
Tested image on PST simics.
```
```

Change-Id: I06672ff7b38f7edcdabf0b1e2bfe1bfafd5d6078
Signed-off-by: Alpana Kumari <alpu8007@gmail.com>

show more ...

Refactor pvm_keep_and_clear bios attr handler

Replace hardcoded constants with JSON configuration and updated the
logic to remove hardcoded constant keywords and record values for
pvm_keep_and_clear

Refactor pvm_keep_and_clear bios attr handler

Replace hardcoded constants with JSON configuration and updated the
logic to remove hardcoded constant keywords and record values for
pvm_keep_and_clear attribute. These values are now dynamically
retrieved from the Bios-map JSON file to improve maintainability and
flexibility.

Change-Id: I5fc771d5fd571a2b68ee0fae6911a0144aa1dfe7
Signed-off-by: Chanchal Tiwari <chanchaltiwari70021@gmail.com>

show more ...

Refactor pvm_clear_nvram bios attr handler

Replace hardcoded constants with JSON configuration and updated the
logic to remove hardcoded constant keywords and record values for
pvm_clear_nvram attri

Refactor pvm_clear_nvram bios attr handler

Replace hardcoded constants with JSON configuration and updated the
logic to remove hardcoded constant keywords and record values for
pvm_clear_nvram attribute. These values are now dynamically retrieved
from the Bios-map JSON file to improve maintainability and
flexibility.

Change-Id: I09c05bfda86d14ce6680cd131b3b01a3a9885d19
Signed-off-by: Chanchal Tiwari <chanchaltiwari70021@gmail.com>

show more ...

Refactor pvm_create_default_lpar attr handler

Replace hardcoded constants with JSON configuration and updated the
logic to remove hardcoded constant keywords and record values for
pvm_create_default

Refactor pvm_create_default_lpar attr handler

Replace hardcoded constants with JSON configuration and updated the
logic to remove hardcoded constant keywords and record values for
pvm_create_default_lpar attribute. These values are now dynamically
retrieved from the Bios-map JSON file to improve maintainability and
flexibility.

Change-Id: I00561cdef7b3e68179470a12577d8c8fb0dd3114
Signed-off-by: Chanchal Tiwari <chanchaltiwari70021@gmail.com>

show more ...

Refactor hb_memory_mirror_mode bios attr handler

Replace hardcoded constants with JSON configuration and updated the
logic to remove hardcoded constant keywords and record values for
hb_memory_mirro

Refactor hb_memory_mirror_mode bios attr handler

Replace hardcoded constants with JSON configuration and updated the
logic to remove hardcoded constant keywords and record values for
hb_memory_mirror_mode attribute. These values are now dynamically
retrieved from the Bios-map JSON file to improve maintainability and
flexibility.

Change-Id: Ie1271ea81415e2f3e8d78a083ce864ba17460be5
Signed-off-by: Chanchal Tiwari <chanchaltiwari70021@gmail.com>

show more ...

worker class update

Updated this class APIs, to use logger::logMessage with placeHolder PEL
instead of using EventLogger::createSyncPel to log a PEL.
This change is required to stream line everythin

worker class update

Updated this class APIs, to use logger::logMessage with placeHolder PEL
instead of using EventLogger::createSyncPel to log a PEL.
This change is required to stream line everything with logger class.

Test-
- Tested on huygens simcs, can see the PEL i forced to log.
```
"User Data 1": {
"Section Version": "1",
"Sub-section type": "1",
"Created by": "bmc error logging",
"DESCRIPTION": "ParseAndPublish VPD failed for [reason] Standard runtime exception. Reason: DBG: Alpana's patch testing",
"FileName": "/usr/src/debug/openpower-fru-vpd/1.0+git/vpd-manager/src/worker.cpp",
"FunctionName": "std::tuple<bool, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > > vpd::Worker::parseAndPublishVPD(const std::string&)",
"InteranlRc": "0",
"UserData1": "",
"UserData2": ""
},

```

Change-Id: Ib2aa762f7d6e026011ead1c874dd06c782737403
Signed-off-by: Alpana Kumari <alpu8007@gmail.com>

show more ...

Logger Class Update

This commit is modifying logMessage API. The argument pelTuple changed
to optional, so that caller doesn't need to create the pelInfoTuple
structure. Instead, it can be created i

Logger Class Update

This commit is modifying logMessage API. The argument pelTuple changed
to optional, so that caller doesn't need to create the pelInfoTuple
structure. Instead, it can be created inside the logMessage API with
the sent details.

Test-
- Can see the PEL on PST simics.
```
"User Data 1": {
"Section Version": "1",
"Sub-section type": "1",
"Created by": "bmc error logging",
"DESCRIPTION": "ParseAndPublish VPD failed for [reason] Standard runtime exception. Reason: DBG: Alpana's patch testing",
"FileName": "/usr/src/debug/openpower-fru-vpd/1.0+git/vpd-manager/src/worker.cpp",
"FunctionName": "std::tuple<bool, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > > vpd::Worker::parseAndPublishVPD(const std::string&)",
"InteranlRc": "0",
"UserData1": "",
"UserData2": ""
},

```

Change-Id: I3d5e3c749e93749f0ea7a88d88531a4b3faedc80
Signed-off-by: Alpana Kumari <alpu8007@gmail.com>

show more ...

File mode support for system VPD

This commit adds code to support read and write of system VPD keywords
in file mode.

Test:
```
Value before updating keyword:
root@p11bmc:~# vpd-tool -r -O /sys/bu

File mode support for system VPD

This commit adds code to support read and write of system VPD keywords
in file mode.

Test:
```
Value before updating keyword:
root@p11bmc:~# vpd-tool -r -O /sys/bus/i2c/drivers/at24/8-0050/eeprom -H -R VSYS -K RB
{
"/sys/bus/i2c/drivers/at24/8-0050/eeprom": {
"RB": "0001"
}
}

root@p11bmc:~# vpd-tool -r -O /system/chassis/motherboard -R VSYS -K RB
{
"/system/chassis/motherboard": {
"RB": "0001"
}
}
root@p11bmc:~# vpd-tool -r -O /system/chassis -R VSYS -K RB
{
"/system/chassis": {
"RB": "0001"
}
}

Update the keyword
root@p11bmc:~# vpd-tool -w -O /sys/bus/i2c/drivers/at24/8-0050/eeprom -H -R VSYS -K RB --value "0003"
Data updated successfully

After updating the value:
root@p11bmc:~# vpd-tool -r -O /sys/bus/i2c/drivers/at24/8-0050/eeprom -H -R VSYS -K RB
{
"/var/lib/vpd/file/sys/bus/i2c/drivers/at24/8-0050/eeprom": {
"RB": "0003"
}
}

root@p11bmc:~# vpd-tool -r -O /system/chassis -R VSYS -K RB
{
"/system/chassis": {
"RB": "0003"
}
}
root@p11bmc:~# vpd-tool -r -O /system/chassis/motherboard -R VSYS -K RB
{
"/system/chassis/motherboard": {
"RB": "0003"
}
}
```

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

show more ...

Refactor hb_field_core_override bios attr handler handler

Replace hardcoded constants with JSON configuration and updated the
logic to remove hardcoded constant keywords and record values for
hb_fie

Refactor hb_field_core_override bios attr handler handler

Replace hardcoded constants with JSON configuration and updated the
logic to remove hardcoded constant keywords and record values for
hb_field_core_override attribute. These values are now dynamically
retrieved from the Bios-map JSON file to improve maintainability and
flexibility.

Change-Id: Ibc86f854070ff14affd47fedbba0f960f2054e70
Signed-off-by: Chanchal Tiwari <chanchaltiwari70021@gmail.com>

show more ...

Revert "Logger Class Update"

The commit changes the signature of API resulting in all other calls to
fail. All the related changes need to be merged in a single shot.

This reverts commit 5c484e9e5a

Revert "Logger Class Update"

The commit changes the signature of API resulting in all other calls to
fail. All the related changes need to be merged in a single shot.

This reverts commit 5c484e9e5a399075ef26d6974ed5c1c7a486e45c.

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

show more ...

Logger Class Update

This commit is modifying logMessage API. The argument pelTuple changed
to optional, so that caller doesn't need to create the pelInfoTuple
structure. Instead, it can be created i

Logger Class Update

This commit is modifying logMessage API. The argument pelTuple changed
to optional, so that caller doesn't need to create the pelInfoTuple
structure. Instead, it can be created inside the logMessage API with
the sent details.

Test-
- Can see the PEL on huygens simics.
```
"User Data 1": {
"Section Version": "1",
"Sub-section type": "1",
"Created by": "bmc error logging",
"DESCRIPTION": "ParseAndPublish VPD failed for [reason] Standard runtime exception. Reason: DBG: Alpana's patch testing",
"FileName": "/usr/src/debug/openpower-fru-vpd/1.0+git/vpd-manager/src/worker.cpp",
"FunctionName": "std::tuple<bool, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > > vpd::Worker::parseAndPublishVPD(const std::string&)",
"InteranlRc": "0",
"UserData1": "",
"UserData2": ""
},

```

Change-Id: I1b617ca611507d15e7592bc88a2995200feee06c
Signed-off-by: Alpana Kumari <alpu8007@gmail.com>

show more ...

BIOS JSON parsing and refactor Backup/Restore API

Updated IbmBiosHandler to extract the BIOS configuration file path from
The System Configuration JSON. Additionally, updated the
backUpOrRestoreBios

BIOS JSON parsing and refactor Backup/Restore API

Updated IbmBiosHandler to extract the BIOS configuration file path from
The System Configuration JSON. Additionally, updated the
backUpOrRestoreBiosAttributes API to align with these changes,ensuring
BIOS attributes are processed correctly based on the system-specific
configuration and bios json.

Change-Id: I9e97733b5c1440e8fb346ceb3fe7cb8328fb5684
Signed-off-by: Chanchal Tiwari <chanchaltiwari70021@gmail.com>

show more ...

GpioEventHandler :Handle uninitialized Worker

This commit adds changes in GpioEventHandler to handle the scenario
where Worker instant is not initialized. GpioEventHandler APIs depend on
Worker inst

GpioEventHandler :Handle uninitialized Worker

This commit adds changes in GpioEventHandler to handle the scenario
where Worker instant is not initialized. GpioEventHandler APIs depend on
Worker instance, so if Worker instance is not initialized, we should
fail GpioEventHandler instantiation as well.

Test:
```
- Build vpd-manager for rbmc-prototype
- Install vpd-manager binary on an rbmc-prototype system
- Restart vpd-manager
- Observe vpd-manager service started successfully
Jan 27 08:47:42 rbmc-prototype systemd[1]: Starting VPD Manager...
Jan 27 08:47:42 rbmc-prototype vpd-manager[29695]: ## Error: "fieldmode" not defined
Jan 27 08:47:42 rbmc-prototype vpd-manager[29696]: ## Error: "vpdmode" not defined
Jan 27 08:47:42 rbmc-prototype vpd-manager[29694]: FileName: /usr/src/debug/openpower-fru-vpd/1.0+git/vpd-manager/oem-handler/ibm_handler.cpp, Line: 1174 Failed to set collection status for path /sys/bus/i2c/drivers/at24/1-0051/eepromReason: The FRU path is not found in the JSON.
Jan 27 08:47:42 rbmc-prototype vpd-manager[29694]: FileName: /usr/src/debug/openpower-fru-vpd/1.0+git/vpd-manager/oem-handler/ibm_handler.cpp, Line: 198 Failed to initialize event listener. Error: Cannot instantiate Listener as Worker is not initialized
Jan 27 08:47:42 rbmc-prototype vpd-manager[29694]: FileName: /usr/src/debug/openpower-fru-vpd/1.0+git/vpd-manager/src/manager_main.cpp, Line: 46 Start VPD-Manager event loop
Jan 27 08:47:42 rbmc-prototype systemd[1]: Started VPD Manager.
```

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

show more ...

Return in case of invalid JSON

In case the system config JSON does not has mandatory tag, log error
message and return from the method.
No further processing is required as the system config JSON is

Return in case of invalid JSON

In case the system config JSON does not has mandatory tag, log error
message and return from the method.
No further processing is required as the system config JSON is
considered to be invalid in case mandatory tag is missing.

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

show more ...

Manager: handle uninitialized Worker in D-Bus APIs

This commit adds changes in vpd-manager APIs to check for uninitialized
Worker instance. vpd-manager API implementations are using Worker
instance

Manager: handle uninitialized Worker in D-Bus APIs

This commit adds changes in vpd-manager APIs to check for uninitialized
Worker instance. vpd-manager API implementations are using Worker
instance and should fail if Worker instance is uninitialized.

Test:
```
- Build vpd-manager for rbmc-prototype
- Install vpd-manager binary on an rbmc-prototype system
- Restart vpd-manager
- Observe vpd-manager service started successfully
Jan 27 08:47:42 rbmc-prototype systemd[1]: Starting VPD Manager...
Jan 27 08:47:42 rbmc-prototype vpd-manager[29695]: ## Error: "fieldmode" not defined
Jan 27 08:47:42 rbmc-prototype vpd-manager[29696]: ## Error: "vpdmode" not defined
Jan 27 08:47:42 rbmc-prototype vpd-manager[29694]: FileName: /usr/src/debug/openpower-fru-vpd/1.0+git/vpd-manager/oem-handler/ibm_handler.cpp, Line: 1174 Failed to set collection status for path /sys/bus/i2c/drivers/at24/1-0051/eepromReason: The FRU path is not found in the JSON.
Jan 27 08:47:42 rbmc-prototype vpd-manager[29694]: FileName: /usr/src/debug/openpower-fru-vpd/1.0+git/vpd-manager/oem-handler/ibm_handler.cpp, Line: 198 Failed to initialize event listener. Error: Cannot instantiate Listener as Worker is not initialized
Jan 27 08:47:42 rbmc-prototype vpd-manager[29694]: FileName: /usr/src/debug/openpower-fru-vpd/1.0+git/vpd-manager/src/manager_main.cpp, Line: 46 Start VPD-Manager event loop
Jan 27 08:47:42 rbmc-prototype systemd[1]: Started VPD Manager.

- Execute GetExpandedLocationCode API from busctl and see vpd-manager
does not crash

FileName: /usr/src/debug/openpower-fru-vpd/1.0+git/vpd-manager/src/manager.cpp, Line: 489 Cannot get expanded location code as Worker instance is not initialized.
```

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

show more ...

Listener: handle Worker not initialized scenario

This commit adds changes in Listener instantiation if Worker is not
initialized. Listener APIs require Worker instance, so if Worker
instance is not

Listener: handle Worker not initialized scenario

This commit adds changes in Listener instantiation if Worker is not
initialized. Listener APIs require Worker instance, so if Worker
instance is not initialized, Listener instantiation should also fail.

Test:
```
- Build vpd-manager for rbmc-prototype
- Install vpd-manager binary on an rbmc-prototype system
- Restart vpd-manager
- Observe vpd-manager service started successfully
Jan 27 08:47:42 rbmc-prototype systemd[1]: Starting VPD Manager...
Jan 27 08:47:42 rbmc-prototype vpd-manager[29695]: ## Error: "fieldmode" not defined
Jan 27 08:47:42 rbmc-prototype vpd-manager[29696]: ## Error: "vpdmode" not defined
Jan 27 08:47:42 rbmc-prototype vpd-manager[29694]: FileName: /usr/src/debug/openpower-fru-vpd/1.0+git/vpd-manager/oem-handler/ibm_handler.cpp, Line: 1174 Failed to set collection status for path /sys/bus/i2c/drivers/at24/1-0051/eepromReason: The FRU path is not found in the JSON.
Jan 27 08:47:42 rbmc-prototype vpd-manager[29694]: FileName: /usr/src/debug/openpower-fru-vpd/1.0+git/vpd-manager/oem-handler/ibm_handler.cpp, Line: 198 Failed to initialize event listener. Error: Cannot instantiate Listener as Worker is not initialized
Jan 27 08:47:42 rbmc-prototype vpd-manager[29694]: FileName: /usr/src/debug/openpower-fru-vpd/1.0+git/vpd-manager/src/manager_main.cpp, Line: 46 Start VPD-Manager event loop
Jan 27 08:47:42 rbmc-prototype systemd[1]: Started VPD Manager.
```

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

show more ...

Read VPD collection mode

VPD collection mode is used by multiple classes. Instead of reading it
in each class, the mode is read once in the manager class and passed to
other classes to avoid multipl

Read VPD collection mode

VPD collection mode is used by multiple classes. Instead of reading it
in each class, the mode is read once in the manager class and passed to
other classes to avoid multiple reads.

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

show more ...

Add API to check JSON validity

The commit adds an API to check if the JSON passed in the constructor of
BackupAndRestore class in having all the required tags or not.

This has been added so that we

Add API to check JSON validity

The commit adds an API to check if the JSON passed in the constructor of
BackupAndRestore class in having all the required tags or not.

This has been added so that we detect any failure case right at the time
of initializing the class and fail early in case of any issue.

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

show more ...