Dump: add file open error to Create interface

Add "xyz.openbmc_project.Common.File.Error.Open" to the list of errors
for the case where a dump file couldn't be opened in the course of dump
creation.

Dump: add file open error to Create interface

Add "xyz.openbmc_project.Common.File.Error.Open" to the list of errors
for the case where a dump file couldn't be opened in the course of dump
creation.

Initial usage in phosphor-debug-collector repo:
https://github.com/openbmc/phosphor-debug-collector/commit/919f71c652728da24e3e78e3ef14662265c5f812#diff-917a58c38d39e3d9353bec42ba512927c880aa5df49a3aa1700498fb17b25926R80

Signed-off-by: Claire Weinan <cweinan@google.com>
Change-Id: Ief61e89e0b5976a500447a445d90a3520ea20164

show more ...

Fixed a mispelling
Saw a typo in the README. Simply fixed.

Signed-off-by: Corey Hardesty <corey.hardesty@icloud.com>
Change-Id: Id0ac0e4e14811c5009376992d6ff2e92e1a3ef17

inventory: drive: Add resettable property to indicate resettable drive

Enable the option to let people know if a drive support reset action.
This allow us to differentiate between the resettable dri

inventory: drive: Add resettable property to indicate resettable drive

Enable the option to let people know if a drive support reset action.
This allow us to differentiate between the resettable drives and the
non-resettable ones. Not all drives are resettable and is wrong to
assume they are ressetable.

Change-Id: Iac5acba82530163a52b35b9d18de40f8748f4d00
Signed-off-by: Willy Tu <wltu@google.com>

show more ...

Added RetryAttempts property

This property allows the number of reboot attempts left on the BMC to
differ from the total number of allowed host reboots. New functionality added to bmcweb will allow

Added RetryAttempts property

This property allows the number of reboot attempts left on the BMC to
differ from the total number of allowed host reboots. New functionality added to bmcweb will allow an external user to specify the amount
of "RetryAttempts" desired(if AutoReboot is enabled). The original
property "AttemptsLeft"still contains the number of allowed reboot
attempts remaining while this new property "RetryAttempts" contains
the maximum number of reboot attempts that are allowed.

Signed-off-by: Corey Hardesty <corey.hardesty@icloud.com>
Change-Id: I527fba6fea0bbc9ab3b731def3405a2d10a58055

show more ...

Add safe mode to user interface

Add the ability to see when system is in safe mode.
Tested: caused safe mode in system.
Signed-off-by: Sheldon Bailey <baileysh@us.ibm.com>
Change-Id: Ibcf14178f4243d

Add safe mode to user interface

Add the ability to see when system is in safe mode.
Tested: caused safe mode in system.
Signed-off-by: Sheldon Bailey <baileysh@us.ibm.com>
Change-Id: Ibcf14178f4243d3c00a270b5811e446b656131f5

show more ...

markdownlint: minor formatting changes to Software/README

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

prettier: reformat YAML

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

EthernetInterface: Mark InterfaceName and VLANId as const

InterfaceName and VLANId are properties that never change once
initialized, mark them as const.

Change-Id: I0a43c732d8793d3e98666733778f10d

EthernetInterface: Mark InterfaceName and VLANId as const

InterfaceName and VLANId are properties that never change once
initialized, mark them as const.

Change-Id: I0a43c732d8793d3e98666733778f10de23b08640
Signed-off-by: Jiaqing Zhao <jiaqing.zhao@intel.com>

show more ...

EthernetInterface: Mark Speed and AutoNeg as readonly

These values are read from ethtool ioctl and cannot be modified in
networkd, mark them as readonly.

Change-Id: I5df264c64dc82ca61f53b05a36762b3

EthernetInterface: Mark Speed and AutoNeg as readonly

These values are read from ethtool ioctl and cannot be modified in
networkd, mark them as readonly.

Change-Id: I5df264c64dc82ca61f53b05a36762b30b4f4810f
Signed-off-by: Jiaqing Zhao <jiaqing.zhao@intel.com>

show more ...

EthernetInterface: Change LinkUp property to readonly

A const property never changes during the lifetime of the object it
belongs to. But in phosphor-networkd, linkUp will be changed by the
service

EthernetInterface: Change LinkUp property to readonly

A const property never changes during the lifetime of the object it
belongs to. But in phosphor-networkd, linkUp will be changed by the
service itself. It should be readonly instead of const.

Change-Id: I42a3c96edd68f4a2dbd81e2ab84583d6b3239c6d
Signed-off-by: Jiaqing Zhao <jiaqing.zhao@intel.com>

show more ...

Change power cap bounds to be writable

The power cap properties are hosted by Settings. The values need to be
written by OCC.Control service. When occ-control attempts to write
these properties, it

Change power cap bounds to be writable

The power cap properties are hosted by Settings. The values need to be
written by OCC.Control service. When occ-control attempts to write
these properties, it gets an error because the values are not writable.

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

show more ...

Rename the reverse_name to reverse_names in associations.

It will allow two different associations with the same name and
different reverse_names.

For example:
associations:
- name: forward_nam

Rename the reverse_name to reverse_names in associations.

It will allow two different associations with the same name and
different reverse_names.

For example:
associations:
- name: forward_name
reverse_names:
- reverse_name_A
- reverse_name_B
required_endpoint_interfaces:
- endpoint
Will have two associations:
- [forward_name, reverse_name_A, endpoint]
- [forward_name, reverse_name_B, endpoint]

Change-Id: Ia01ae59b7dba1556e7c7f8e56ccede47d623c1a1
Signed-off-by: Zhenwei Chen <zhenweichen0207@gmail.com>

show more ...

Fixed incorrect references in Led/Readme.md.

Existing references contained the wrong path to Led/Group.Interface.yaml
and Physical.Interface.yaml.
Also reference in chapter "Dbus interfaces for phys

Fixed incorrect references in Led/Readme.md.

Existing references contained the wrong path to Led/Group.Interface.yaml
and Physical.Interface.yaml.
Also reference in chapter "Dbus interfaces for physical LEDs"
referred to Group.Interface.yaml instead of Physical.Interface.yaml.

This commit fixes file paths and enters two different references
(for Group and for Physical).

Signed-off-by: Vladimir Kuznetsov <v.kuznetsov@yadro.com>
Change-Id: Ib34a470f366db4657fd6d6e49b109a79319f79c3

show more ...

inventory.decorator: Add slot number interface

This interface provides a slot number property for an inventory item.

The initial use case is to hang this on certain
phosphor-inventory-manager paths

inventory.decorator: Add slot number interface

This interface provides a slot number property for an inventory item.

The initial use case is to hang this on certain
phosphor-inventory-manager paths so that when entity-manager probes on
them it can have $SlotNumber available as a variable to give a stable
sensor name, such as 'PCIe Card $SlotNumber Temp'.

Without this, if say the $index variable is used instead, and then there
is no guarantee which device will have which sensor name, especially for
cards that can be added in any order when the BMC is running.

Stable sensor names are important in the
phosphor-inventory-manager/entity-manager hybrid model where PIM is used
for inventory and EM is just used for config.

There is a LocationCode property already that could possibly be used on
some systems, but on IBM systems at least it contains the system serial
number so would vary from system to system which isn't desired either.

Signed-off-by: Matt Spinler <spinler@us.ibm.com>
Change-Id: Iea95d7ad123672909799244b0d80f564cb40d425

show more ...

Add StaticNTPServers propetry to EthernetInterface

Currently there is no way to differentiate between the Static
and DHCP assigned NTP servers at dbus

This commit adds StaticNTPServers peroperty to

Add StaticNTPServers propetry to EthernetInterface

Currently there is no way to differentiate between the Static
and DHCP assigned NTP servers at dbus

This commit adds StaticNTPServers peroperty to hold the static
configuration of NTP servers. This is a read-write property.
The existing NTPServer propery will hold both static and DHCP
assigned NTP servers. This is a read-only property.

Signed-off-by: Sunitha Harish <sunharis@in.ibm.com>
Change-Id: If6ac588c268473802ebb1b8f08144df594ac943a

show more ...

Syslog: introduce interfaces for e-mail sending

Interfaces to configure multiple e-mail destinations, each with an
individual threshold for the priority level of the syslog messages.

The idea is to

Syslog: introduce interfaces for e-mail sending

Interfaces to configure multiple e-mail destinations, each with an
individual threshold for the priority level of the syslog messages.

The idea is to provide facilities to implement a minimally useful subset
of Redfish Event Destinations for e-mail with ability to add additional
properties if the need arises.

D-Bus object path for each destination contains 8 digits of a mailto
hash, so Level can be changed later but for another URI a new object
needs to be created.

Current set of properties is made on assumption that organisation has an
SMTP smarthost (that BMC is allowed to use without authentication) which
takes care of dispatching messages; rsyslog's ommail module is
considered for the initial implementation.

Example session:

root@qemuarm:~# busctl tree xyz.openbmc_project.Syslog.Config
└─/xyz
└─/xyz/openbmc_project
└─/xyz/openbmc_project/logging
└─/xyz/openbmc_project/logging/config
├─/xyz/openbmc_project/logging/config/mail
│ ├─/xyz/openbmc_project/logging/config/mail/2dc1e699
│ ├─/xyz/openbmc_project/logging/config/mail/5d43d672
│ ├─/xyz/openbmc_project/logging/config/mail/b4dc9055
│ ├─/xyz/openbmc_project/logging/config/mail/d5d4db03
│ └─/xyz/openbmc_project/logging/config/mail/e0974603
└─/xyz/openbmc_project/logging/config/remote
root@qemuarm:~# busctl introspect xyz.openbmc_project.Syslog.Config /xyz/openbmc_project/logging/config/mail
NAME TYPE SIGNATURE RESULT/VALUE FLAGS
org.freedesktop.DBus.Introspectable interface - - -
.Introspect method - s -
org.freedesktop.DBus.Peer interface - - -
.GetMachineId method - s -
.Ping method - - -
org.freedesktop.DBus.Properties interface - - -
.Get method ss v -
.GetAll method s a{sv} -
.Set method ssv - -
.PropertiesChanged signal sa{sv}as - -
xyz.openbmc_project.Collection.DeleteAll interface - - -
.DeleteAll method - - -
xyz.openbmc_project.Logging.Syslog.Destination.Mail interface - - -
.From property s "from@some.domain" emits-change writable
xyz.openbmc_project.Logging.Syslog.Destination.Mail.Create interface - - -
.Create method ss o -
xyz.openbmc_project.Network.Client interface - - -
.Address property s "smarthost.lan" emits-change writable
.Port property q 25 emits-change writable
xyz.openbmc_project.Object.Enable interface - - -
.Enabled property b true emits-change writable
root@qemuarm:~# busctl introspect xyz.openbmc_project.Syslog.Config /xyz/openbmc_project/logging/config/mail/d5d4db03
NAME TYPE SIGNATURE RESULT/VALUE FLAGS
org.freedesktop.DBus.Introspectable interface - - -
.Introspect method - s -
org.freedesktop.DBus.Peer interface - - -
.GetMachineId method - s -
.Ping method - - -
org.freedesktop.DBus.Properties interface - - -
.Get method ss v -
.GetAll method s a{sv} -
.Set method ssv - -
.PropertiesChanged signal sa{sv}as - -
xyz.openbmc_project.Logging.Syslog.Destination.Mail.Entry interface - - -
.Level property s "xyz.openbmc_project.Logging.Entry.Leve… emits-change writable
.Mailto property s "mailto://someone2@otherdomain.test" emits-change
xyz.openbmc_project.Object.Delete interface - - -
.Delete method - - -

Signed-off-by: Paul Fertser <fercerpav@gmail.com>
Change-Id: Ic78fc2bf3608c1dd7ec0c962412e69062dde48d4

show more ...

Lanes in use property added

The commit adds property "LanesInUse" property for PICeDevice
interface.
It will denote the number of PCIe lanes in use by this device.

Signed-off-by: Sunny Srivastava <

Lanes in use property added

The commit adds property "LanesInUse" property for PICeDevice
interface.
It will denote the number of PCIe lanes in use by this device.

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

show more ...

Dump: InitiateOffload: Add Unavailable error

The offload may not be available due to various resources
needed for offload is busy or not available in such cases
Unavailable error needs to be returne

Dump: InitiateOffload: Add Unavailable error

The offload may not be available due to various resources
needed for offload is busy or not available in such cases
Unavailable error needs to be returned.

Signed-off-by: Dhruvaraj Subhashchandran <dhruvaraj@in.ibm.com>
Change-Id: Ibaec718c12ed53ca83977aeb41e10d4cac2fb30f

show more ...

Adds new properties to drive

EncryptedStatus, lockedStatus, PredictedMediaLifeLeftPercent, are added
to the drives interface in this change. These properties are planned to be
sent over redfish usin

Adds new properties to drive

EncryptedStatus, lockedStatus, PredictedMediaLifeLeftPercent, are added
to the drives interface in this change. These properties are planned to be
sent over redfish using the following schema properties

https://github.com/openbmc/bmcweb/blob/c0bd5e4b37f65ce29a392594d1ef240c64c99734/static/redfish/v1/schema/Drive_v1.xml#L371
https://github.com/openbmc/bmcweb/blob/c0bd5e4b37f65ce29a392594d1ef240c64c99734/static/redfish/v1/schema/Drive_v1.xml#L365
https://github.com/openbmc/bmcweb/blob/c0bd5e4b37f65ce29a392594d1ef240c64c99734/static/redfish/v1/schema/Drive_v1.xml#L267

Signed-off-by: John Broadbent <jebr@google.com>
Change-Id: I09347c36ff68140b34a6ec316629aba137b86997

show more ...

Inventory: PCIeDevice: add association to pcie_slot

Create association documentation to pcie slot from pcie devices.

Change-Id: I167aa18864572953bb5f7c7c3b7031d0b6046366
Signed-off-by: Willy Tu <wl

Inventory: PCIeDevice: add association to pcie_slot

Create association documentation to pcie slot from pcie devices.

Change-Id: I167aa18864572953bb5f7c7c3b7031d0b6046366
Signed-off-by: Willy Tu <wltu@google.com>

show more ...

inventory: PCIeDevice: Add MaxLanes information to PCIe Device

The Max Lanes information is maximum number of PCIe lanes supported by
the PCIe Device.

Change-Id: Id90ba3849ee6c31745a2714ac94a01fd4d

inventory: PCIeDevice: Add MaxLanes information to PCIe Device

The Max Lanes information is maximum number of PCIe lanes supported by
the PCIe Device.

Change-Id: Id90ba3849ee6c31745a2714ac94a01fd4d64e657
Signed-off-by: Willy Tu <wltu@google.com>

show more ...

Add Error to Software Version Interface

This commit adds a InvalidSignature error to the
Software Version interface. It will ensure that
if the error is hit during the verificatioon of
an image, a c

Add Error to Software Version Interface

This commit adds a InvalidSignature error to the
Software Version interface. It will ensure that
if the error is hit during the verificatioon of
an image, a corresponding error is available to the
user.

Signed-off-by: Zami Seck <zimzam17@gmail.com>
Change-Id: Iba2eb7689a3e34db15e4a5d31accb69c18f906f5

show more ...

State: New PowerSystemInputs interface

Add a new PowerSystemInputs interface to describe the state of the power
supplied to the chassis via the power supply unit(s), in order to have
state manager h

State: New PowerSystemInputs interface

Add a new PowerSystemInputs interface to describe the state of the power
supplied to the chassis via the power supply unit(s), in order to have
state manager handle Brownout conditions as described in design doc:
https://gerrit.openbmc-project.xyz/c/openbmc/docs/+/48015

This interface would be implemented by the phosphor-power repo and the
Status property set to Fault when a Brownout condition is detected.

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

show more ...

Add additional DHCP configuration options

The existing D-Bus DHCPConf enumearation values do not allow IPv6
Stateless DHCP to be assigned. The IPv4 entries are sufficient, as
DHCP for the v4 stack i

Add additional DHCP configuration options

The existing D-Bus DHCPConf enumearation values do not allow IPv6
Stateless DHCP to be assigned. The IPv4 entries are sufficient, as
DHCP for the v4 stack is truly a boolean.

IPv6 is, at minimum, a ternary control. Adding new enumerations allows
the IPv6 configruation to be done in a more abstract fashion than the
current IPv6AcceptRA D-Bus entry. This D-Bus entry is a direct
reflection of how systemd-networkd manages Routint Advertisement, and
by extension DHCPv6. Only phosphor-network needs this knowledge.

Tested:
The changes compile.
Also, in combination with changes made to phosphor-network, many
busctl set/get property commands were issued to confirm the
enumerations were performing as expected.

Change-Id: I42a46549fed8b2d2187ea6a7eb7278c63b84bbef
Signed-off-by: Johnathan Mantey <johnathanx.mantey@intel.com>

show more ...

Delete: Add InsufficientPermission error

- The object may not be allowed to delete due to insufficient permission
for example, the object might be created by the system, and the user
may not hav

Delete: Add InsufficientPermission error

- The object may not be allowed to delete due to insufficient permission
for example, the object might be created by the system, and the user
may not have permission to delete it.

Signed-off-by: Ramesh Iyyar <rameshi1@in.ibm.com>
Change-Id: Ib662bc275cb091ec41b701faa9aad65b2ae6c6fc

show more ...