black: re-format

black and isort are enabled in the openbmc-build-scripts on Python files
to have a consistent formatting. Re-run the formatter on the whole
repository.

Change-Id: Ieb4226b16132640

black: re-format

black and isort are enabled in the openbmc-build-scripts on Python files
to have a consistent formatting. Re-run the formatter on the whole
repository.

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

show more ...

python: reformat using black

Code cannot currently be submitted to this repo because it fails
in the unit-test phase. Attempt to reformat the code using `black`
to hopefully make pycodestyle happie

python: reformat using black

Code cannot currently be submitted to this repo because it fails
in the unit-test phase. Attempt to reformat the code using `black`
to hopefully make pycodestyle happier.

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

show more ...

Prevent the reboot command from powering on the host.

Pushing reset button or sending reboot command
should not power on the host if it was off.

Change-Id: I9e307e7b5d181f94bed7176baad23a95920ed8b2

Prevent the reboot command from powering on the host.

Pushing reset button or sending reboot command
should not power on the host if it was off.

Change-Id: I9e307e7b5d181f94bed7176baad23a95920ed8b2
Signed-off-by: Alexander Soldatov <a.soldatov@yadro.com>

show more ...

Port to python 3

Port code to python 3 yet remain backward compatible to python 2.
This port is in preparation for yocto 2.4 upgrade.

Partially resolves openbmc/openbmc#2920

Change-Id: If174ce9444

Port to python 3

Port code to python 3 yet remain backward compatible to python 2.
This port is in preparation for yocto 2.4 upgrade.

Partially resolves openbmc/openbmc#2920

Change-Id: If174ce9444bde4e21fb086764f4def017e423201
Signed-off-by: CamVan Nguyen <ctnguyen@us.ibm.com>

show more ...

Remove use of settings in chassis_control.py

This application is slowly being deprecated. This change
is to allow us to pull the legacy settings object out
of the image.

Tested: I built an image wi

Remove use of settings in chassis_control.py

This application is slowly being deprecated. This change
is to allow us to pull the legacy settings object out
of the image.

Tested: I built an image with these changes plus the removal of
settings and verified our CT regression bucket still passed.

Change-Id: Ia9416b2faba9a0e3b10f433a0e597530f75177e5
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>

show more ...

pep8 fixes

Also ignore the configs/ directory since it only contains
system configuration files named <system>.py.

Change-Id: I0416239c4030ce5bc39473e65e560016cd4ffc75
Signed-off-by: Adriana Kobyla

pep8 fixes

Also ignore the configs/ directory since it only contains
system configuration files named <system>.py.

Change-Id: I0416239c4030ce5bc39473e65e560016cd4ffc75
Signed-off-by: Adriana Kobylak <anoo@us.ibm.com>

show more ...

Update button press handlers to new targets

The targets have changed within openbmc for hard
power off and soft power down. This file will be
deprecated some day but for now it needs to be
in sync

Update button press handlers to new targets

The targets have changed within openbmc for hard
power off and soft power down. This file will be
deprecated some day but for now it needs to be
in sync with the new targets.

Resolves openbmc/openbmc#1593

Change-Id: I1e2949c67fd9b4756a6f18070eea63c8b161c7bf
Signed-off-by: Andrew Geissler <andrewg@us.ibm.com>

show more ...

chassis: Remove sensor subscription

Remove a signal subscription from the deprecated
org.openbmc.SensorThresholds interface.

Change-Id: I75d83324ada77642a8f560b4250fb24190dc540a
Signed-off-by: Brad

chassis: Remove sensor subscription

Remove a signal subscription from the deprecated
org.openbmc.SensorThresholds interface.

Change-Id: I75d83324ada77642a8f560b4250fb24190dc540a
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>

show more ...

Refactor action target names Host-quiesce

The target change is:
obmc-quiesce-host -> obmc-host-quiesce

Change-Id: I713c9c694fbc5b45e0ee86c0e7ccc7f544bc823c
Signed-off-by: Josh D. King <jdking@us.ib

Refactor action target names Host-quiesce

The target change is:
obmc-quiesce-host -> obmc-host-quiesce

Change-Id: I713c9c694fbc5b45e0ee86c0e7ccc7f544bc823c
Signed-off-by: Josh D. King <jdking@us.ibm.com>

show more ...

Refactor action target names Host-start-stop

The target change is:
obmc-chassis-stop -> obmc-host-stop
obmc-chassis-start -> obmc-host-start

Change-Id: Ibba1e8d56e8d911dca309a6e6dc9ad3335bb644d
Sig

Refactor action target names Host-start-stop

The target change is:
obmc-chassis-stop -> obmc-host-stop
obmc-chassis-start -> obmc-host-start

Change-Id: Ibba1e8d56e8d911dca309a6e6dc9ad3335bb644d
Signed-off-by: Josh D. King <jdking@us.ibm.com>

show more ...

LED: Remove methods used to set and clear Identify LEDs.

Now that the LED Group manager support is in, the users can make a
call to phosphor-ledmanager and assert the Identify group and hence
this c

LED: Remove methods used to set and clear Identify LEDs.

Now that the LED Group manager support is in, the users can make a
call to phosphor-ledmanager and assert the Identify group and hence
this code in here is not needed.

Change-Id: I7c268710ee8806215c60015d6b39a0ef19601fd0
Signed-off-by: Vishwanatha Subbanna <vishwa@linux.vnet.ibm.com>

show more ...

Go to quiesce state on checkstop/watchdog error

This change alters the behavior on a host
checkstop or watchdog error. Instead of simply
rebooting, go to the new quiesce state.

Change-Id: I4dd3a3d7

Go to quiesce state on checkstop/watchdog error

This change alters the behavior on a host
checkstop or watchdog error. Instead of simply
rebooting, go to the new quiesce state.

Change-Id: I4dd3a3d7a91607fc56a684873115d5f32de18f6a
Signed-off-by: Michael Tritz <mtritz@us.ibm.com>

show more ...

python: add vim modelines

Change-Id: Ic13d79892e3efe6f7bad357ad747e6fd25299d02
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>

Use super to construct dbus.service.Object

Change-Id: I830decfc523d435c6b9ecdc9ffffec7956c3b85d
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>

chassis: pep8 fixes

Change-Id: I6dc52616ce30660f2e6992f6e1a52d44dfec1cd7
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>

chassis_control: remove unused modules

Change-Id: I65d56a5597c5e79d4ac48d90b5d7437f50cc8023
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>

chassis_control: remove execute permissions

Change-Id: I6ba0bfe7463cf8647d07218174d578f2c96e076a
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>

chassis: Use systemd for chassis on/off

Start a systemd target for chassis state changes.

Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
Change-Id: I3f02360aeec35ff451d4da8a087c1aeb1e7709

chassis: Use systemd for chassis on/off

Start a systemd target for chassis state changes.

Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
Change-Id: I3f02360aeec35ff451d4da8a087c1aeb1e7709e0

show more ...

Start host watchdog after magic sequence

The magic sequence triggers the host to boot, so start the
host watchdog timer (which monitors for the host hanging)
right after that.
There's no need to por

Start host watchdog after magic sequence

The magic sequence triggers the host to boot, so start the
host watchdog timer (which monitors for the host hanging)
right after that.
There's no need to port the check for debug flag because
it's checked as part of the magic sequence.
Resolves openbmc/skeleton#127

Change-Id: Ibad7b73c795c6cc8971bb99b755c8fce2c859f97
Signed-off-by: Adriana Kobylak <anoo@us.ibm.com>

show more ...

Power button power off behavior

Change the power button behavior to mirror the current behavior
on Power and x86 systems:
On short press: soft power off
On long press: hard power off
Resolves openbm

Power button power off behavior

Change the power button behavior to mirror the current behavior
on Power and x86 systems:
On short press: soft power off
On long press: hard power off
Resolves openbmc/openbmc#485

Change-Id: I436c364f6ed072474f942f80d98428df11b4c666
Signed-off-by: Adriana Kobylak <anoo@us.ibm.com>

show more ...

chassis_control: Log an event when emergency shutdown host

According to BMC Firmware Specification, on occasion the BMC will be required
to shut down the host due to high ambient or fan fails. A SEL

chassis_control: Log an event when emergency shutdown host

According to BMC Firmware Specification, on occasion the BMC will be required
to shut down the host due to high ambient or fan fails. A SEL must be
present in this case.

Fixes openbmc/skeleton#64

Change-Id: I3d779fc597961821a7a7f1b0f147314072ae8802
Signed-off-by: Yi Li <adamliyi@msn.com>

show more ...

Remove unnecessary InterfacesAdded signals

Remove ObjectManager signals emitted before a bus name is owned.

This isn't necessary now that the system_manager can detect objects via
NameOwnerChanged.

Remove unnecessary InterfacesAdded signals

Remove ObjectManager signals emitted before a bus name is owned.

This isn't necessary now that the system_manager can detect objects via
NameOwnerChanged.

Change-Id: Ia2b626a926392b50d20c26474e6c9a9fdb3ca2f0
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>

show more ...

Export objects prior to bus name grab

The default behavior in pyphosphor changed such that ObjectManager
and Properties signals are not emitted by default. Enable signals
just prior to bus name gra

Export objects prior to bus name grab

The default behavior in pyphosphor changed such that ObjectManager
and Properties signals are not emitted by default. Enable signals
just prior to bus name grab.

Change-Id: I27a2de4b60482dd843702396f33c7cdccaf17277
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>

show more ...

Defer busname ownership until init complete

Avoid race conditions where applications expecting an object to be
present upon recipt of NameOwnerChanged find that the object hasn't
been created yet.

Defer busname ownership until init complete

Avoid race conditions where applications expecting an object to be
present upon recipt of NameOwnerChanged find that the object hasn't
been created yet.

Change-Id: I4236a3898055a84b00e1eaf37f32e0192b683a0f
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>

show more ...

Disable watchdog timer if debug_mode is set

resolves openbmc/openbmc#246

debug_mode is used when the host is not going to be started.
In this case, we do not want the watchdog timer to be started
b

Disable watchdog timer if debug_mode is set

resolves openbmc/openbmc#246

debug_mode is used when the host is not going to be started.
In this case, we do not want the watchdog timer to be started
because there is no one to keep it from resetting.

This code change prevents the watchdog from being started
during poweron.

Potential side affects could be if we do start the host
in this code path. The watchdog is un-tested when it's
not started but software comes in and tries to reset it.
The code in control_host_obj.c currently enforces the
no-host-power-on when debug_mode is set.

show more ...