8623918b | 22-Dec-2021 |
Tim Lee <timlee660101@gmail.com> |
Correct osState string to match xyz naming format for fixing automation test
Symptom: "Verify Dump After Host Watchdog Error Injection" automation test item failed.
Root cause: Due to "Is OS Booted
Correct osState string to match xyz naming format for fixing automation test
Symptom: "Verify Dump After Host Watchdog Error Injection" automation test item failed.
Root cause: Due to "Is OS Booted" keyword got failed in lib/state_manager.robot.
According openbmc-test-automation/lib/variables.py The "os_state" must match to "xyz.openbmc_project.State.OperatingSystem.Status.OSStatus.Standby".
But, "osState" string in x86-power-control just return "Standby" instead of "xyz.openbmc_project.State.OperatingSystem.Status.OSStatus.Standby". Thus keyword "Is OS Booted" return failed then cause test item got failed.
Reference: openbmc-test-automation/lib/state_manager.robot: Is OS Booted [Documentation] Check OS status. ${os_state}= Get Host State Attribute OperatingSystemState Should Be Equal ${OS_BOOT_COMPLETE} ${os_state}
openbmc-test-automation/lib/variables.py: STATE_DBUS_BASE = 'xyz.openbmc_project.State.' OS_BOOT_COMPLETE = STATE_DBUS_BASE + \ 'OperatingSystem.Status.OSStatus.Standby'
Solution: Correct osState string to match xyz naming format and consistent with auto test.
Tested: Host Power ON: busctl call xyz.openbmc_project.State.Host /xyz/openbmc_project/state/host0 \ org.freedesktop.DBus.Properties \ GetAll s "xyz.openbmc_project.State.OperatingSystem.Status" a{sv} 1 "OperatingSystemState" s "xyz.openbmc_project.State.OperatingSystem.Status.OSStatus.Standby"
Host Power OFF: busctl call xyz.openbmc_project.State.Host /xyz/openbmc_project/state/host0 \ org.freedesktop.DBus.Properties \ GetAll s "xyz.openbmc_project.State.OperatingSystem.Status" "xyz.openbmc_project.State.OperatingSystem.Status.OSStatus.Inactive"
Signed-off-by: Tim Lee <timlee660101@gmail.com> Change-Id: Ia63100a6756104a645c2d789d3b5be560a1305ff
show more ...
|
3efcf37b | 29-Dec-2021 |
Andrei Kartashev <a.kartashev@yadro.com> |
use same consumer for each gpio request
The `consumer` field of line_request supposed to represent consumer application. Currently x86-power-control uses different names for different cases: applica
use same consumer for each gpio request
The `consumer` field of line_request supposed to represent consumer application. Currently x86-power-control uses different names for different cases: application name for inputs and function name for outputs.
There is no benefits in use function name as consumer, so use application name in both cases.
Tested: gpioinfo | grep -i power-control see all gpios, used by x86-power-control Signed-off-by: Andrei Kartashev <a.kartashev@yadro.com> Change-Id: I1baef7bd4bb9b0c2efcb423f513f56d244a0cbdd
show more ...
|
50fe8ee3 | 04-Jan-2022 |
Andrei Kartashev <a.kartashev@yadro.com> |
introduce PersistentState manager
This commit brings new mechanism to store application state over system reboot. State is saved to json file and restored on application startup by easy to use class
introduce PersistentState manager
This commit brings new mechanism to store application state over system reboot. State is saved to json file and restored on application startup by easy to use class.
Currently only PowerState is managed by this class but more states can be added in future.
Tested: verify that power state saved to file and restored on startup Change-Id: I91fb46f262827dda89f438eefea38d59e02ba25c Signed-off-by: Andrei Kartashev <a.kartashev@yadro.com>
show more ...
|
c46ebb49 | 10-Nov-2021 |
Jason M. Bills <jason.m.bills@linux.intel.com> |
Change to lg2
To take advantage of the improved structured logging, change x86-power-control logging to lg2.
Also fixed up some minor formatting issues and added more detail to clarify some of the
Change to lg2
To take advantage of the improved structured logging, change x86-power-control logging to lg2.
Also fixed up some minor formatting issues and added more detail to clarify some of the debug messages.
Tested: Ran a power off, on, cycle, and reset and confirmed that the logs look the same.
Change-Id: Iad601e39a54e7e1dc0f26352711cb1b3d5f83522 Signed-off-by: Jason M. Bills <jason.m.bills@linux.intel.com>
show more ...
|
85e111ed | 11-Nov-2021 |
Logananth Sundararaj <logananth.s@hcl.com> |
Changes for dbus based gpio config.
Updating dbus match search filter function with more specific interface name to avoid match function event handler getting called for multiple not relevant proper
Changes for dbus based gpio config.
Updating dbus match search filter function with more specific interface name to avoid match function event handler getting called for multiple not relevant properties.
Updating the variable type to match with the return type from sdbusplus method call
TESTED:Tested and verified in YosemiteV2.
Signed-off-by: Logananth Sundararaj <logananth.s@hcl.com> Change-Id: Ib910b8b7c174247e7199eb0785fa9c81f6a54aac
show more ...
|
9fd8ac2b | 21-Oct-2021 |
Jason M. Bills <jason.m.bills@linux.intel.com> |
Fix issues flagged by werror
This fixes the issues that are flagged by werror in preparation for enabling a meson build.
Change-Id: Icf05316d21390188f84da5b59f97319e3718e728 Signed-off-by: Jason M.
Fix issues flagged by werror
This fixes the issues that are flagged by werror in preparation for enabling a meson build.
Change-Id: Icf05316d21390188f84da5b59f97319e3718e728 Signed-off-by: Jason M. Bills <jason.m.bills@linux.intel.com>
show more ...
|
0d57f4e3 | 21-Oct-2021 |
Jason M. Bills <jason.m.bills@linux.intel.com> |
Re-organize source
After removing i2c support, we can reduce the depth of the source code and remove the power-control-x86 folder.
We can also simplify the CMake files.
Tested: Confirmed that ipmi
Re-organize source
After removing i2c support, we can reduce the depth of the source code and remove the power-control-x86 folder.
We can also simplify the CMake files.
Tested: Confirmed that ipmi power control commands all still work correctly.
Change-Id: I67104e40555d4d84c2b946fad000e3a541828272 Signed-off-by: Jason M. Bills <jason.m.bills@linux.intel.com>
show more ...
|