diff options
author | Konstantin Aladyshev <aladyshev22@gmail.com> | 2021-06-18 16:08:48 +0300 |
---|---|---|
committer | Konstantin Aladyshev <aladyshev22@gmail.com> | 2021-08-02 19:11:49 +0300 |
commit | 8bb79f27b6e61b856bd75fc0316e66326eadef8f (patch) | |
tree | 7a5e168e0c240f6e48c08121687d43994e1ea859 | |
parent | d153e1b511d0a175020fdb77824493972222a65b (diff) | |
download | openbmc-8bb79f27b6e61b856bd75fc0316e66326eadef8f.tar.xz |
phosphor-settings-manager: redesign boot setting override feature
Current realization of a boot override feature has several downsides
which are described in detail in the mailing list letter
https://lists.ozlabs.org/pipermail/openbmc/2021-May/026533.html
Redesign interfaces under '/xyz/openbmc_project/control/host0/boot'
object to solve all the problems, in particular:
- add 'Enable' interface under '/xyz/openbmc_project/control/host0/boot'
path to store overall boot override enabled state,
- drop BootSource/BootMode/BootType interfaces from the
'/xyz/openbmc_project/control/host0/boot/one_time' object. In the new
design there is no need to store such properties twice,
- by default make override feature disabled and permanent.
Signed-off-by: Konstantin Aladyshev <aladyshev22@gmail.com>
Change-Id: Ie59ba703a6f39f8e1d4934cce89cbe8cd5a826e4
-rw-r--r-- | meta-phosphor/recipes-phosphor/settings/phosphor-settings-defaults/defaults.yaml | 14 | ||||
-rw-r--r-- | meta-phosphor/recipes-phosphor/settings/phosphor-settings-manager/boot_type.override.yml | 18 |
2 files changed, 10 insertions, 22 deletions
diff --git a/meta-phosphor/recipes-phosphor/settings/phosphor-settings-defaults/defaults.yaml b/meta-phosphor/recipes-phosphor/settings/phosphor-settings-defaults/defaults.yaml index a4119e6cc..d51ba0369 100644 --- a/meta-phosphor/recipes-phosphor/settings/phosphor-settings-defaults/defaults.yaml +++ b/meta-phosphor/recipes-phosphor/settings/phosphor-settings-defaults/defaults.yaml @@ -25,20 +25,16 @@ Properties: BootMode: Default: Mode::Modes::Regular + - Interface: xyz.openbmc_project.Object.Enable + Properties: + Enabled: + Default: 'false' /xyz/openbmc_project/control/host0/boot/one_time: - - Interface: xyz.openbmc_project.Control.Boot.Source - Properties: - BootSource: - Default: Source::Sources::Default - - Interface: xyz.openbmc_project.Control.Boot.Mode - Properties: - BootMode: - Default: Mode::Modes::Regular - Interface: xyz.openbmc_project.Object.Enable Properties: Enabled: - Default: 'true' + Default: 'false' /xyz/openbmc_project/control/host0/power_cap: - Interface: xyz.openbmc_project.Control.Power.Cap diff --git a/meta-phosphor/recipes-phosphor/settings/phosphor-settings-manager/boot_type.override.yml b/meta-phosphor/recipes-phosphor/settings/phosphor-settings-manager/boot_type.override.yml index da1501302..458961177 100644 --- a/meta-phosphor/recipes-phosphor/settings/phosphor-settings-manager/boot_type.override.yml +++ b/meta-phosphor/recipes-phosphor/settings/phosphor-settings-manager/boot_type.override.yml @@ -11,22 +11,14 @@ Properties: BootType: Default: Type::Types::EFI + - Interface: xyz.openbmc_project.Object.Enable + Properties: + Enabled: + Default: 'false' /xyz/openbmc_project/control/host0/boot/one_time: - - Interface: xyz.openbmc_project.Control.Boot.Source - Properties: - BootSource: - Default: Source::Sources::Default - - Interface: xyz.openbmc_project.Control.Boot.Mode - Properties: - BootMode: - Default: Mode::Modes::Regular - - Interface: xyz.openbmc_project.Control.Boot.Type - Properties: - BootType: - Default: Type::Types::EFI - Interface: xyz.openbmc_project.Object.Enable Properties: Enabled: - Default: 'true' + Default: 'false' |