Age | Commit message (Collapse) | Author | Files | Lines |
|
This drop adds socket-id for first console. This was left out before
but with the new design this parameter is required and enforced by
obmc-console code. It is decided to set "socket-id = console0"
initially because that is the current path used by bmcweb but in future
you can change this value to whatever is appropriate.
The console information is now available through the DBUS interface.
Each console server register it's object path on DBUS. The leaf node of
the object path is used to create a console path. For example in the
following console0 the leaf node "console0" is used to make up the
console path and the GUI will use console path "/console/console0" to
connect to the console.
busctl tree xyz.openbmc_project.Console.console0
`-/xyz
`-/xyz/openbmc_project
`-/xyz/openbmc_project/console
`-/xyz/openbmc_project/console/console0
Here is a discord thread where new design was discussed.
https://discord.com/channels/775381525260664832/1083551792094249051
Tested:
Tested on the rainier system.
Related commits:
1) phosphor-dbus-interface: https://gerrit.openbmc.org/c/openbmc/phosphor-dbus-interfaces/+/61486
2) obmc-console: https://gerrit.openbmc.org/c/openbmc/obmc-console/+/62496
3) bmcweb: https://gerrit.openbmc.org/c/openbmc/bmcweb/+/62525
Change-Id: I5fd06f3aac69872a3e5032d592a9b59711795cfe
Signed-off-by: Ninad Palsule <ninadpalsule@us.ibm.com>
|
|
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: Iced76041c1207dc9ab28a83fd849ba3cd0a07bb5
|
|
As noted in the following mailing list thread:
https://lists.ozlabs.org/pipermail/openbmc/2022-November/032457.html
and implemented via this series of commits:
https://gerrit.openbmc.org/q/topic:op-power-control-ref
Some systemd services had their names changed. Update the service files
in this repo to reflect this.
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
Change-Id: Idaca9c1a46f8b5d673b4912e3027fc9268aa5dba
|
|
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
Change-Id: I8e54833ac78e540e9dd5011533d53ff9a3af6763
|
|
Romulus has a Qemu model called romulus-bmc.
Change-Id: Iea41fc4fcfdb3cea8e2fb9dc286dfca2f304184f
Signed-off-by: Joel Stanley <joel@jms.id.au>
|
|
All of these abstractions have exactly one implementation. Remove the
indirection to improve at a glance comprehension.
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
Change-Id: I5d701aff6d0876fa3b2d16c841cbdcb0433b221f
|
|
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
Change-Id: I65febeeec11e6e5b40ee728f42cfbe77023dae1e
|
|
We also need to move the conf-notes.txt files.
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: Ic44e015c0216b526de4fec277ad42f162bca1f33
|
|
The latest poky commit is requiring us to have all of
our template configs in a subdirectory instead of directly in
the `conf` directory. Without this we end up with errors during
setup like:
```
Error: TEMPLATECONF value (which is .../openbmc/meta-facebook/meta-bletchley/conf) must point to meta-some-layer/conf/templates/template-name
```
Fix this by moving all of our template files into the 'default'
template subdirectory (following the pattern of poky) and modifying
`setup` as necessary to follow.
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: Iecefde73d55acbb6bc63ae3d68c4311adaf327ae
|
|
The OpenBMC u-boot tree now has support for the AST2500 based Romulus
and can use a single configuration for NCSI and non-NCSI boards.
Change-Id: I6daa155fcafa0128b119d67bf47cf629ac4b7977
Signed-off-by: Joel Stanley <joel@jms.id.au>
|
|
Harvey.Wu (2):
dbus-sdr: fix SEL record Generator ID to fit SPEC
dbus-sdr: fix the vector size in sdrWriteTable and sdrStatsTable
Jian Zhang (1):
Add data types to Value to extend setDbusProperty
Vernon Mauery (1):
fix header install
Willy Tu (4):
cleanup: Remove all warning errors for the Meson build support
build: Add meson build for phosphor-host-ipmi
build: move all shared_library/module to library with version suffix
build: Remove dynamicsensor library if it is not enabled
Change-Id: I92503c6625d1ec2bf88c0a28afac9b4eeb8fbd75
Signed-off-by: Andrew Geissler <openbmcbump-github@yahoo.com>
Signed-off-by: Willy Tu <wltu@google.com>
|
|
Don't use /usr/bin/env, it's an antipattern.
Signed-off-by: Matt Spinler <spinler@us.ibm.com>
Change-Id: If388f5427bb2ca45718d0ca99452bf3f147861dd
|
|
This was added with cde0f094f for libseccomp, but shortly after that
upstream moved libseccomp from meta-security to core (241c7d2e6). As
such, meta-security is no longer used or required.
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
Change-Id: I371e54b11f6336720dfc6edf0ef733d22b7fb4f2
|
|
Per [1][2], Yocto is starting to deprecate ABORT and has replaced
the "ABORT" action in BB_DISKMON_DIRS entries with "HALT".
1. https://wiki.yoctoproject.org/wiki/Inclusive_language
2. https://git.yoctoproject.org/poky/commit/?id=4f77505d94a8f6260933f457e9848d1d2fa98ce5
Tested:
Built obmc-phosphor-image successfully and eliminate the following
warnings:
```
WARNING: The BB_DISKMON_DIRS "ABORT" action has been renamed to
"HALT", update configuration
```
Signed-off-by: George Liu <liuxiwei@inspur.com>
Change-Id: If57d0ded9fac41e23b31b01e2a0e309ac7388148
|
|
Palmetto and Romulus were using phosphor-webui. phosphor-webui built on
AngularJS, and AngularJS goes End of Life December 31, 2021.
So Move Plametto and Romulus to use webui-vue.
webui-vue is a modern replacement for phoshor-webui. Weui-vue is built
on vue.js, uses Redfish, and has additional features like
- Ability to easily theme to meet brand guidelines
- Accessibility
- Full Redfish
- Improved user experience based on user feedback
- Language translation-ready
- Modern front-end framework with an active community and future
development roadmap
Tested: manually tested on Palmetto and Romulus system. Able to log in
into the system using the latest webui-vue. Able to retrieve
dump log and another log, all basic functionality is working.
Signed-off-by: Abhishek Patel <Abhishek.Patel@ibm.com>
Change-Id: I883a84552383f3d7f94d9227a11fc7410bc43ec5
|
|
* Deprecate N-1 release (hardknott).
* Enable N+1 release (kirkstone).
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I39e027e02dab64b4390b46ffbd9c299c858f403e
|
|
e.g.
WARNING: /home/peter/openbmc/meta-ibm/meta-romulus/recipes-phosphor/occ/romulus-occ-control-config-native.bb: QA Issue: romulus-occ-control-config-native: native/nativesdk class is not inherited last, this can result in unexpected behaviour. Classes inherited after native/nativesdk: openpower-occ-control.bbclass [native-last]
WARNING: /home/peter/openbmc/meta-ibm/meta-romulus/recipes-phosphor/logging/romulus-phosphor-logging-callouts-native.bb: QA Issue: romulus-phosphor-logging-callouts-native: native/nativesdk class is not inherited last, this can result in unexpected behaviour. Classes inherited after native/nativesdk: phosphor-logging.bbclass [native-last]
Signed-off-by: Peter Foley <pefoley@google.com>
Change-Id: I31be11e264e1701215d2eadd28dc2249153a5d61
|
|
Per [1], Yocto is going to start requiring both branch names and
`protocol=https` in GitHub SRC_URI variables. Ran the upstream tool to
automatically fix our URIs.
1. https://git.yoctoproject.org/poky/tree/documentation/migration-guides/migration-3.5.rst#n10
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I5f9619f7ff707050b469ae74c8aa16846568c5e4
|
|
This sets u-boot to the package containing the v2016.07-aspeed-openbmc
branch. This ensures there will be no change if the default changes in
the future.
Change-Id: Iea12e1691dbdda34a3f95d6d0862add0b15bcf37
Signed-off-by: Joel Stanley <joel@jms.id.au>
|
|
`BBLAYERS_NON_REMOVABLE` is obsolete and no longer required.
As it said by Yocto documentation it can be used by `Hob`
https://www.yoctoproject.org/docs/1.5.2/ref-manual/ref-manual.html#var-BBLAYERS_NON_REMOVABLE
that already removed since Yocto-2.1
https://www.yoctoproject.org/tools-resources/projects/hob
Change-Id: Ibc2d8268a9d837a81e9cf6b0131dba8d0a030a3f
Signed-off-by: Alexander Filippov <a.filippov@yadro.com>
|
|
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: Ia05891430c6c97a89b7bc8ba4558ae496866bf7d
|
|
bitbake offers a choice of DEB, RPM or IPK packaging. To a degree the
choice is functionally arbitrary for image generation but control over
the package format becomes important if we want to:
1. Include runtime package management functionality in the firmware image
2. Mess about with the packages on the build system
With respect to 1 the IPK format and opkg (an ipk package manager) are
designed for embedded systems[1] - by contrast to RPMs have heavier
dependencies and a greater impact on the size and complexity of the
firmware image.
Regarding 2, the embedded nature and the need for opkg to work without
much fuss leads to a lower configuration barrier by comparison to RPMs.
With ipk it becomes possible to reuse the packages built during image
preparation for core analysis without needing to generate an SDK:
```
$ export LD_LIBRARY_PATH=./tmp/work/x86_64-linux/opkg-native/*/recipe-sysroot-native/usr/lib
$ MY_DEBUG_ROOT=tmp/rootfs-debug
$ ./tmp/sysroots-components/x86_64/opkg-native/usr/bin/opkg \
-f ./tmp/work/p10bmc-openbmc-linux-gnueabi/obmc-phosphor-image/*/opkg.conf \
-o $MY_DEBUG_ROOT \
update
$ fakeroot ./tmp/sysroots-components/x86_64/opkg-native/usr/bin/opkg \
-f ./tmp/work/p10bmc-openbmc-linux-gnueabi/obmc-phosphor-image/1.0-r0/opkg.conf \
-o $MY_DEBUG_ROOT \
install dbus-sensors dbus-sensors-dbg
$ gdb-multiarch
(gdb) set solib-absolute-prefix .../tmp/rootfs-debug
(gdb) add-auto-load-safe-path .../tmp/rootfs-debug
(gdb) file tmp/rootfs-debug/usr/bin/nvmesensor
(gdb) core-file obmcdump_17_9597/core.nvmesensor.0.aae91b519d0e4e0e8bbe746e3f6cd25f.2779.9594000000
Core was generated by `/usr/bin/nvmesensor'.
Program terminated with signal SIGABRT, Aborted.
pthread_kill.c:45
45 pthread_kill.c: No such file or directory.
(gdb) bt
pthread_kill.c:45
../sysdeps/posix/raise.c:26
/home/andrew/src/openbmc/openbmc/build/p10bmc/tmp/rootfs-debug/usr/lib/libstdc++.so.6
/home/andrew/src/openbmc/openbmc/build/p10bmc/tmp/rootfs-debug/usr/lib/libstdc++.so.6
/home/andrew/src/openbmc/openbmc/build/p10bmc/tmp/rootfs-debug/usr/lib/libstdc++.so.6
/home/andrew/src/openbmc/openbmc/build/p10bmc/tmp/rootfs-debug/usr/lib/libstdc++.so.6
"xyz.openbmc_project.NVMeSensor", this=0x488f04) at
/usr/include/sdbusplus/bus.hpp:234
../../../../../../workspace/sources/dbus-sensors/src/NVMeSensorMain.cpp:159
(gdb)
```
This approach documented in the Poky Reference Manual:
https://www.yoctoproject.org/docs/1.0/poky-ref-manual/poky-ref-manual.html#platdev-gdb-remotedebug-launch-gdb-inferiorbins
Switch all machines to IPK to align the debugging experience with
upstream's documentation and to facilitate efficient use of packaged
software at runtime.
[1] https://openwrt.org/docs/guide-user/additional-software/opkg
Change-Id: I8ef526add2d7a6790de1b3eb3fb85cd39b864f23
Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
|
|
Upstream bbclasses changed to typically use the `:${PN}` override
syntax, including the SYSTEMD_ variables. Change our systemd.bbclass
to do the same for consistency and perform a tree-wide variable
replacement.
Spot checked by building bletchley and witherspoon and checking
some of the SYSTEMD_LINK directives on installed packages under qemu.
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I20a9dd809bff8af8759488734f80486c7228c6eb
|
|
We've typically kept these LAYERSERIES_COMPAT to 2 releases: the current
and the upcoming. Remove 'gatesgarth' is it is now 2 releases back.
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I5e812a94fed1738898af75c0fdee81996a5bbf20
|
|
Update with the latest version from upstream.
Change-Id: I1a7da37b0457dab873afaf6445aca360d54b47ca
Signed-off-by: Adriana Kobylak <anoo@us.ibm.com>
|
|
Change-Id: I9116ed7260e369136acb39eec15075db2d4dbeba
Signed-off-by: Adriana Kobylak <anoo@us.ibm.com>
|
|
Change Ie1a7c389edb6b7a048836a49283ceb62de51bba5 will be transitioning
the 'Type' property in Inventory.Item.Chassis to an enumeration.
In order to avoid crashing in PIM we need these default values in the
starting YAML to be fully-qualified enumeration names that will match
the values which will start in Ie1a7c.
This code is safe to merge as is without any PDI or PIM changes. Prior
to the PDI change, we must make a change to PIM that allows conversion
automatic from string->enum, leveraging library interfaces available in
sdbusplus. These will be submitted independently.
I checked the codebase for usage of this string. It appears that the
value is currently, effectively, write-only. There is code in bmcweb
that fills in the equivalent Redfish value but currently just hard-codes
the string 'RackMount'.
Tested: Booted Witherspoon in a QEMU model with this change and proposed
changes to sdbusplus + PIM. PIM no longer coredumps with the PDI change
and yields an expected persistence file:
```
$ pwd
/var/lib/phosphor-inventory-manager/xyz/openbmc_project/inventory/system/chassis
$ cat xyz.openbmc_project.Inventory.Item.Chassis
{
"value0": {
"cereal_class_version": 2,
"Type": 3
}
}
```
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: Icaf9447f31ccdd945cdf74b3e017682e4aed686f
|
|
This is apparently not actually working anymore and is removed in the
next poky update.
Change-Id: Ia1c6a258d124a4a30a14fc42e8e0bba95e64faeb
Signed-off-by: William A. Kennington III <wak@google.com>
|
|
The repository supports it, so switch it over in bitbake.
Signed-off-by: Matt Spinler <spinler@us.ibm.com>
Change-Id: I8d96c79b395ee2614ddd869091569f245426c5c7
|
|
Latest upstream yocto has moved on to the 3.3 hardknott release
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
Change-Id: Ieae36798d66d21c2c642931f06407d3bb2acf163
|
|
This layer provides libseccomp.
Signed-off-by: Anton D. Kachalov <gmouse@google.com>
Change-Id: I84513d56f2ed75fab49043196b98ef8b858e394f
|
|
Most openpower machines use DISTRO="openbmc-openpower" but
Romulus has a "openbmc-romulus" distro config file. This
config file has nothing except an inclusion of the openpower
one, so remove this useless indirection.
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I371ce4137369e4964eed1529f8a8190e31721ecc
|
|
The next release of Yocto is soon and most of the upstream
layers have switched support strings for it. Support layer
compat for gatesgarth (current) and dunfell (previous).
(From meta-ibm rev: 284e2ad06544f567582b2732a22e052190b5b95a)
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: Ic1cb74ebd387715b08feccd8b94bfd0be7d35205
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
As we add support for a new series, remove support for the oldest.
Little to no testing is done with older ones.
(From meta-ibm rev: 6255d23fd68023fdc838d0a2262b43883b86be58)
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
Change-Id: Id63754bdc8ec82628cad23427bfdadafca380c15
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
Dbus sensor interface 'xyz.openbmc_project.Sensor.Value' migrated
from int to double. Therefore all old 'int64_t' types in
"*-ipmi-sensor*.yaml" files for this interface should be changed to
'double'.
(From meta-ibm rev: c769f790e14b9caa4fc842dc1e2035378de6ae60)
Signed-off-by: Konstantin Aladyshev <aladyshev22@gmail.com>
Change-Id: Id8e164203541550eb19301ca429702cbf7d4b2f8
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
Similar to commit 567a8d for witherspoon, this issue has now popped
up on Romulus hardware.
The concept of warm reboot (reboot the host without removing chassis
power) was added a while back as a part of clarifying support for
the Redfish ComputerSystem.Reset Action. It has since been found that
some IBM hardware, including Romulus, do not support warm reboots.
The issue is intermittent and depends on the reboot being requested
while the host is in the middle of some i2c transactions. If a warm
reboot occurs during this window, then on the subsequent boot the i2c
hardware will be in a bad state and the host firmware is unable to
recover it.
Given this issue, it has been requested that BMC software go back to
doing cold reboots (cycling power to the chassis during any reboot
request) on this hardware.
(From meta-ibm rev: 718f2b5405c2c1f000cc0ccbf2860a283997c08a)
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
Change-Id: I31810e89977532d4368050871aba03d78deb9c59
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
This was a P9 DD1 workaround. These systems are no longer supported and
are not found in the wild, so the workaround has been retired.
All of the p9 systems in the tree exclude the package so this change
should be a noop.
(From meta-ibm rev: 38c5c9539fb3f01fc2b96d483701938bffdf48a0)
Change-Id: I1c7786526dee4d9bb0d5db916890c1cbe3d51577
Signed-off-by: Joel Stanley <joel@jms.id.au>
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
(From meta-ibm rev: b2f780f527e156adff88c3c8e05fe87762266db6)
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
Change-Id: I523e93b55dd1333640ef7e84aa596b5efb06c4ca
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
|
|
The yaml add the "xyz.openbmc_project.Inventory.Item.Chassis" interface
to "/xyz/openbmc_project/inventory/system/chassis" object.
This is needed for bmcweb to correctly get the chassis id in
/redfish/v1/Chassis/.
Tested: Verify that /redfish/v1/Chassis/chassis exists on Romulus with
this change. It was 404 without this change.
(From meta-ibm rev: 41b829feead473e5541f7d4c3c075e2b1a525c45)
Signed-off-by: Lei YU <mine260309@gmail.com>
Change-Id: Ie091ad32315fb4b4e45d8b35966a7a238dd1d539
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
|
|
Base hardware enablement support for meta-romulus is provided by
meta-romulus and meta-openpower. meta-ibm adds advanced serviceability
functions for IBM branded products backed by the IBM service structure.
Romulus is a reference platform for ODMs and/or OEMS building POWER9
based systems. End users cannot purchase support contracts for Romulus
systems. As such remove meta-ibm from the build configuration in the
spirit of keeping the reference implementation mini and simplified.
At a functional level this removes the ibm-logging application from
images.
Note that meta-ibm can always be re-integrated by simply adding it to
bblayers.conf
(From meta-ibm rev: c8cb279e0a8e9d941d8a73f04496a81e0efd5501)
Change-Id: Ie657468f2566042c5faad9ae5a7923a2f9222564
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
|
|
OE-core master will be dropping warrior soon; zeus is the next release.
(From meta-ibm rev: 9c8d1b961e1185ab2e7eabf6f6b42754fcb57db0)
Change-Id: I791f06ddb596584d1a566657a38ad6eb13692521
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
|
|
IBMBASE is only used for pointing at licenses...point at the
licenses in oe-core in meta/files/common-licenses instead. to match the
defacto convention used in other oe layers like meta-openembedded.
(From meta-ibm rev: cbbb0e5b4e44d831fce5daafd13272d401440f40)
Change-Id: Ib214f92b2c384e3eb2a1f53ecf4b21034438d001
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
|
|
The cooling-type in phosphor-fan is optional, and Romulus is a
air-cooled system and does not require cooling-type application at all.
Remove the cooling-type app and the related configuration files.
Tested: Verify the generated phosphor-fan related hpp and cpp files are
identical as before, and the system is able to power on and do
fan monitor/control without problem.
(From meta-ibm rev: 3168f5e5fb59bd37eed1600d42a8d6fc02875fea)
Change-Id: I30f47ef63f4ff5bea89f31966273930baaf070ea
Signed-off-by: Lei YU <mine260309@gmail.com>
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
|
|
(From meta-ibm rev: 8f7ff601395419a933eb0506d5cc07fd805f7708)
Change-Id: Iffb2c550511b4024b721c2ba81aec1f909ce7703
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
|
|
With updated from openbmc/openbmc#2911, event actions are now able to be
configured with group subsets and event timers are now included in the
available event triggers. Actions with group subsets allow a single
event's set of triggers to run actions against a given group subset
instead of the entire event groups. Also, with timers being included as
a trigger, events can be configured with or without a timer instead of a
disabled timer being created for events that previously did not require
a timer.
Tested:
Generated code is functionally equivalent to previous yaml
(From meta-ibm rev: 03eacc4ef87b3b89cf36aab977ff4eab68f8b4fc)
Change-Id: Ib7eadf5b7c2cb27440e5944348460295334b9f13
Signed-off-by: Matthew Barth <msbarth@us.ibm.com>
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
|
|
This allows upstream to drop some backward-incompatible changes while
the yaml in meta-ibm gets updated to the new format coming in from
upstream.
(From meta-ibm rev: 81ca4572471629b5114a52afb50a84d9e2cc92d1)
Change-Id: Ia554f14c624d236e02d7898d2f95e7a90384dadf
Signed-off-by: Matthew Barth <msbarth@us.ibm.com>
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
|
|
A prior patch added a new recipe romulus-yaml-config to provide all the
YAML configuration for Romulus systems. This patch switches the layer
to that recipe by setting EXTRA_OECONF and DEPENDS as appropriate in the
required bbappends.
With this switch, a number of recipes and their provided YAML are now
dead metadata, so remove those.
(From meta-ibm rev: 66ca8269544784908b0abf78715620adf04d69cc)
Change-Id: I02305ec4c431122ae6ab440c5dcb3def509b7ff9
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
|
|
YAML configuration files exist scattered throughout the OpenBMC tree and
how they are used is controlled with layers dependencies of virtuals and
preferred providers.
Most of the time the above scheme is very difficult to comprehend. This
patch continues a re-thinking of that approach towards a more
centralized scheme.
Specifically this patch implements a single YAML config recipe for the
Romulus systems. The logic contained in the recipe was pulled from all
over the OpenBMC tree - the ability to comprehend how the different YAML
files are generated and consumed should be greatly eased.
One notable detail - unlike the upstream recipes, romulus-yaml-config
is a target recipe and as such enables MACHINE based overrides.
YAML files were copied from different locations in the tree, and run
through a styling application (pyyaml dump(load(yaml))):
romulus-ipmi-fru-properties-native:extra-properties.yaml ->
romulus-yaml-config:romulus-ipmi-fru-properties.yaml
romulus-ipmi-fru-read-inventory-native:config.yaml ->
romulus-yaml-config:romulus-ipmi-fru.yaml
romulus-ipmi-fru-read-bmc-inventory-native:bmc-fru-config.yaml ->
romulus-yaml-config:romulus-ipmi-fru-properties.yaml
romulus-ipmi-inventory-sel-native:sel-config.yaml ->
romulus-yaml-config:romulus-ipmi-inventory-sensors.yaml
romulus-ipmi-sensor-inventory-native:config.yaml ->
romulus-yaml-config:romulus-ipmi-sensors.yaml
(From meta-ibm rev: 67b17f7aa658b71908de47aa0b0814530da36259)
Change-Id: I3953274db23f9ccf8cb74d3962e797bf935d61a8
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
|
|
These files aren't used by ipmid anymore, so drop. See host-ipmid
change 41ac50530a for details.
(From meta-ibm rev: dfd8a072958aa531a41d7fbb190675c66b75e15c)
Change-Id: I3aaab61c24fa0a4167cb04b60e577c1761605887
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
|
|
The service phosphor-cooling-type is required by multi-user.target and
thus will be started by default. In case multi-user.target is started
for multiple times, this service will be startd for multiple times as
well, and eventually get start-limit-hit failure.
This commit adds RemainAfterExit=true so the service will be treated as
started and thus will not be started for multiple times.
Resolves openbmc/openbmc#3544
Tested: Reboot Romulus BMC when the host is on, and verify the BMC gets
READY state after reboot.
(From meta-ibm rev: fdd8f7bb1cca6b505d6b92c741d04a9c6bbfafa7)
Change-Id: Ie3817296265c860d770333c59b591da9d083139c
Signed-off-by: Lei YU <mine260309@gmail.com>
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
|