health_metric: add path for tmp storage

Add the path for temporary storage to be used in HealthMonitor.

Change-Id: I58792286c872846a7b8d71fed11190de852025d6
Signed-off-by: Jagpal Singh Gill <paligi

health_metric: add path for tmp storage

Add the path for temporary storage to be used in HealthMonitor.

Change-Id: I58792286c872846a7b8d71fed11190de852025d6
Signed-off-by: Jagpal Singh Gill <paligill@gmail.com>

show more ...

Update RestrictionMode enum and config file terms

Update the terminology of the RestrictionMode interface enumeration as
well as other language used in configuration file. Also update
terminology u

Update RestrictionMode enum and config file terms

Update the terminology of the RestrictionMode interface enumeration as
well as other language used in configuration file. Also update
terminology used in Bond interface configuration file.

Change-Id: I8a1e299c5644220c5190948b0621df26179c98c4
Signed-off-by: Matt Simmering <matthew.simmering@intel.com>

show more ...

Modify StaticRoute interfaces to static gateway

StaticRoute interfaces being used implementing IPv6 static gateways
https://gerrit.openbmc.org/c/openbmc/bmcweb/+/62255
Network StaticRoute feature wa

Modify StaticRoute interfaces to static gateway

StaticRoute interfaces being used implementing IPv6 static gateways
https://gerrit.openbmc.org/c/openbmc/bmcweb/+/62255
Network StaticRoute feature was dropped for OpenBMC, so these static
route interfaces are no longer needed.

This commit modifies exiting static route interfaces to support static
gateway with required properties

Change-Id: Icd94b35098b50376d701ea2e3b03ece9f2f71be7
Signed-off-by: Ravi Teja <raviteja28031990@gmail.com>

show more ...

health_metric: add bmc metric path

Add the BMC health metric path to be used in phosphor-health-monitor.

Change-Id: I6a06fe12f7c89847a035a0e6b87dbdbad36cb708
Signed-off-by: Jagpal Singh Gill <palig

health_metric: add bmc metric path

Add the BMC health metric path to be used in phosphor-health-monitor.

Change-Id: I6a06fe12f7c89847a035a0e6b87dbdbad36cb708
Signed-off-by: Jagpal Singh Gill <paligill@gmail.com>

show more ...

Control.Power.Mode: Add new power management modes

New modes:
- Balanced Performance
- Efficiency Favor Performance
- Efficiency Favor Power
Also reordered the enums (alphabetical)

Modes were added

Control.Power.Mode: Add new power management modes

New modes:
- Balanced Performance
- Efficiency Favor Performance
- Efficiency Favor Power
Also reordered the enums (alphabetical)

Modes were added to the Redfish ComputerSystem v1.22 schema

Change-Id: I70ba36a6afb13896d812eeb822752020b03d9673
Signed-off-by: Chris Cain <cjcain@us.ibm.com>

show more ...

Throttle, Item: Add throttling association

Create association documentation between Throttle and Item.

Signed-off-by: George Liu <liuxiwei@ieisystem.com>
Change-Id: Iae6b1d370077f91e2e6d25ef82f2dfc

Throttle, Item: Add throttling association

Create association documentation between Throttle and Item.

Signed-off-by: George Liu <liuxiwei@ieisystem.com>
Change-Id: Iae6b1d370077f91e2e6d25ef82f2dfcf8bf0051b

show more ...

chassis: associations: Sort by name

Signed-off-by: George Liu <liuxiwei@ieisystem.com>
Change-Id: I378083d760d51a874bb7baa7566a1af470bb5158

IBM: Api to collect single FRU VPD

An api to collect VPD of a given FRU by launching the
parser exe asynchronously.

It can be used to collect VPD of any given FRU in case of
concurrent maintenance.

IBM: Api to collect single FRU VPD

An api to collect VPD of a given FRU by launching the
parser exe asynchronously.

It can be used to collect VPD of any given FRU in case of
concurrent maintenance.

As a pre-requisite for this api, deleteFRUVPD api needs to
be called for that particular FRU.

As this api makes async call to the parser, caller needs to
register for Present property change signal for that FRU on
DBus to ensure successful execution of VPD parser for that
FRU.
The caller is also suggested to have a timer of around two
mins at their end and in case the present property is not
set to true in that timeline, can mark the call as failed.

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

show more ...

Inventory.Item: add namespace path constant

Add path constant that allow code to avoid hard-coding strings,
matching the requirements outlined in the description section of the
YAML.

Signed-off-by:

Inventory.Item: add namespace path constant

Add path constant that allow code to avoid hard-coding strings,
matching the requirements outlined in the description section of the
YAML.

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

show more ...

Sensor.Value: add path constants

Add path constants that allow code to avoid hard-coding strings.

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

Sensor.Value: add path constants

Add path constants that allow code to avoid hard-coding strings.

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

show more ...

host-state: define new allowed host transition prop

IBM has a system where the host firmware does not want to allow
a ForcedWarmReboot due to potential error paths with the chassis power
remaining o

host-state: define new allowed host transition prop

IBM has a system where the host firmware does not want to allow
a ForcedWarmReboot due to potential error paths with the chassis power
remaining on and their software being forcefully rebooted.

Define a new property which allows the BMC software to optionally define
which Transition actions they support.

bmcweb currently hard codes the AllowableValues in
redfish/v1/Systems/system/ResetActionInfo. That code will be changed
to first look for this new property on D-Bus. If it's not found or
empty, stick with the hard coded defaults. If it is found and it's
non-empty then use it to fill in the AllowableValues.

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

show more ...

Item Associations: Fix capitalization and clarify

This capitalization was mixed, move to the exact capitalization of the
interfaces. E.g. Cpu from [1].

Made clear it was Led.Group and Connector.Por

Item Associations: Fix capitalization and clarify

This capitalization was mixed, move to the exact capitalization of the
interfaces. E.g. Cpu from [1].

Made clear it was Led.Group and Connector.Port being associated.

[1] https://github.com/openbmc/phosphor-dbus-interfaces/blob/2117deabad40f9ac63815f3338e7ac3e1e3fe397/yaml/xyz/openbmc_project/Inventory/Item/Cpu.interface.yaml

Change-Id: I2a4bc4142b5c93335259c98538e0e1754c920bff
Signed-off-by: Gunnar Mills <gmills@us.ibm.com>

show more ...

Cpu/PCIeSlot: Add {connecting, connected_to}

https://gerrit.openbmc.org/c/openbmc/phosphor-dbus-interfaces/+/64951
removes the current associated_pcie_slots, upstream_processor
association.

This co

Cpu/PCIeSlot: Add {connecting, connected_to}

https://gerrit.openbmc.org/c/openbmc/phosphor-dbus-interfaces/+/64951
removes the current associated_pcie_slots, upstream_processor
association.

This commit adds that association back but with an existing association.
Redfish is adding this link to the PCIeDevice so will have to do
something like Cpu -> Connected_to -> PCIeSlot -> Containing ->
PCIeDevice to fill this out in bmcweb. If instead we put this on
PCIeDevice wouldn't be able to have this link for empty PCIeSlots.

[1] was the original commit for Cpu, PCIeDevice association.

https://gerrit.openbmc.org/c/openbmc/bmcweb/+/61509 is the commit adding
this Link to Redfish, it should be refreshed with this newly named
association.

Redfish describes this as "An array of links to the processors that are
directly connected or directly bridged to this PCIe slot."

Took descriptions from [2].

[1] https://github.com/openbmc/phosphor-dbus-interfaces/commit/df55704398ec8fc3cc608206f05a2226f134c786
[2] https://github.com/openbmc/phosphor-dbus-interfaces/commit/df32eb57476e86044a996c5f2b6765a772c10de4

Change-Id: I7a5be817b565de035e6166b07743f0ec8e919950
Signed-off-by: Gunnar Mills <gmills@us.ibm.com>

show more ...

cpu/pcie: remove deprecated association name

There are currently no users of this association in the OpenBMC
organization and the name chosen here does not match the association
naming design. Remo

cpu/pcie: remove deprecated association name

There are currently no users of this association in the OpenBMC
organization and the name chosen here does not match the association
naming design. Remove these associations.

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

show more ...

Dump.Create: add default path and service names

Add default service name for the phosphor-debug-collector
implementation and paths for the dump namespace.

Signed-off-by: Patrick Williams <patrick@s

Dump.Create: add default path and service names

Add default service name for the phosphor-debug-collector
implementation and paths for the dump namespace.

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

show more ...

Logging: add default path and service names

Add default paths and service names from the reference implementation
in phosphor-logging. The service for logging is expected to be a
singleton and the

Logging: add default path and service names

Add default paths and service names from the reference implementation
in phosphor-logging. The service for logging is expected to be a
singleton and the manager object resides at a fixed path.

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

show more ...

Logging.Entry: fix old documentation

`org.openbmc` interfaces were removed years and years ago but the
description still referenced it.

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

Logging.Entry: fix old documentation

`org.openbmc` interfaces were removed years and years ago but the
description still referenced it.

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

show more ...

State: add default paths for BMC,Host,Chassis

Add namespace and path constants matching those in the README for
programmatic use.

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

State: add default paths for BMC,Host,Chassis

Add namespace and path constants matching those in the README for
programmatic use.

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

show more ...

ObjectMapper: Add default paths

The ObjectMapper service is expected to be at a specific location per
the description. Document this with code constants as well.

Signed-off-by: Patrick Williams <p

ObjectMapper: Add default paths

The ObjectMapper service is expected to be at a specific location per
the description. Document this with code constants as well.

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

show more ...

ObjectMapper: match interfaces with reality

The documented ObjectMapper interfaces do not match the reference
implementation in phosphor-objmgr. Lots of code currently uses
hand-written dbus calls

ObjectMapper: match interfaces with reality

The documented ObjectMapper interfaces do not match the reference
implementation in phosphor-objmgr. Lots of code currently uses
hand-written dbus calls to the ObjectMapper, using the real interfaces
and not the documented ones. Refactoring all of these implementations
is quite a bit of effort, which could be staged at a later time (or
leverage a versioned interface). Update the interface to match reality
so that generated bindings could be leveraged.

Tested:

Ran the reference implementation and compared the signatures with the
YAML and updated as necessary.
```
$ busctl --user introspect xyz.openbmc_project.ObjectMapper /xyz/openbmc_project/object_mapper xyz.openbmc_project.ObjectMapper
NAME TYPE SIGNATURE RESULT/VALUE FLAGS
.GetAncestors method sas a{sa{sas}} -
.GetAssociatedSubTree method ooias a{sa{sas}} -
.GetAssociatedSubTreePaths method ooias as -
.GetObject method sas a{sas} -
.GetSubTree method sias a{sa{sas}} -
.GetSubTreePaths method sias as -
```

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

show more ...

Add Hostname property errors

Hostname does not allow special characters while setting hostname
this commit adds InvalidArgument error to handle invalid hostnames

Change-Id: I2ecc92637364baebe9e845b

Add Hostname property errors

Hostname does not allow special characters while setting hostname
this commit adds InvalidArgument error to handle invalid hostnames

Change-Id: I2ecc92637364baebe9e845bd22099fcb9744c2f4
Signed-off-by: Ravi Teja <raviteja28031990@gmail.com>

show more ...

Ipmi.SOL: add default values

In the absence of explicit defaults, the implicit defaults applied to
some of these parameters were somewhat less than ideal, such as
AccumulateIntervalMS getting set to

Ipmi.SOL: add default values

In the absence of explicit defaults, the implicit defaults applied to
some of these parameters were somewhat less than ideal, such as
AccumulateIntervalMS getting set to zero, leading to netipmid going wild
consuming all available CPU cycles whenever an SOL session was active.

This should also obviate the need for most of the currently-existing
overrides of these settings in the main openbmc repo, which are nearly
all identical.

Signed-off-by: Zev Weiss <zev@bewilderbeest.net>
Change-Id: I745b53bae968976729e2c088c8821bec5f4803f0

show more ...

VPD Error interface UnknownSystemType supported

DevTree gets selected based on HW and IM of the system VPD.
If we get anything unexpected from these keywords, it couldn't
determine which systemType

VPD Error interface UnknownSystemType supported

DevTree gets selected based on HW and IM of the system VPD.
If we get anything unexpected from these keywords, it couldn't
determine which systemType it is and so shouldn't load DevTree.
A PEL with this error interface UnknownSystemType will be logged
and break the booting process there itself.

Change-Id: I0437f4db87b4ab556d7c5433a861689c682fafe4
Signed-off-by: Giridhari Krishna <giridharikrishnan@gmail.com>

show more ...

health-metric: add dbus path for total cpu

Add the DBus path for total CPU to be used in phosphor-health-monitor.

Change-Id: Id50d84c7cdc96e723575db591a3d97d052d016bb
Signed-off-by: Jagpal Singh Gi

health-metric: add dbus path for total cpu

Add the DBus path for total CPU to be used in phosphor-health-monitor.

Change-Id: Id50d84c7cdc96e723575db591a3d97d052d016bb
Signed-off-by: Jagpal Singh Gill <paligill@gmail.com>

show more ...

health-metric: add dbus path for read-write storage

Add the DBus path for read write storage to be used in phosphor-health-monitor.

Change-Id: Ic162468dc272a6707cd39476d8e9f128de83e874
Signed-off-b

health-metric: add dbus path for read-write storage

Add the DBus path for read write storage to be used in phosphor-health-monitor.

Change-Id: Ic162468dc272a6707cd39476d8e9f128de83e874
Signed-off-by: Jagpal Singh Gill <paligill@gmail.com>

show more ...