<div dir="ltr">Hi all,<div><br></div><div>Main discussion: <a href="https://lore.kernel.org/openbmc/CAHwn2XknKWyv6Bsq48tkvA52rHKZRFoEJgUuZUmMj6uZb584JQ@mail.gmail.com/" target="_blank">https://lore.kernel.org/openbmc/CAHwn2XknKWyv6Bsq48tkvA52rHKZRFoEJgUuZUmMj6uZb584JQ@mail.gmail.com/</a> (a bit outdated)</div><div><br></div><div>This will be a sub-thread about Storage/Drive resources and new properties that I want to add to bmcweb.</div><div><br></div><div>Related thread: <a href="https://lore.kernel.org/openbmc/CAHwn2XnqABuyc3uMQDs+YgwYJL+yKLwCYn1myOVdZ=gkhw3L-A@mail.gmail.com/">device level logging service</a></div><div><br></div><div>I am thinking of adding a number of support for Storage/Drive in bmcweb and just want some feedback from the community,</div><div><br></div><div>The potential changes includes</div><div>- Multi-Storage Resource Support</div><div>- Drive Software</div><div>- New Drive Property Support</div><div>  - State Conditions</div> - PhysicalLocation<br> - Reset Action <div><br><div><div>-----</div><div><br></div><div>Storage:</div><div><br></div><div>Adding support for multiple Storages instead of just hardcoded to `1`. This allows us to represent one Storage per plugin and organize the resources differently. </div><div><br></div><div>We could potentially use `xyz.openbmc_project.Inventory.Item.StorageController` to create the storages and everything Drive under it will be linked.</div><div><br></div><div>----</div><div><br></div><div>Drive Software:</div><div><br></div><div>It will use SoftwareInventory and will just need to be able to add RelatedItem back to a Drive based on the D-bue Association interface.</div><div><br></div><div>----</div><div><br></div><div>Drive Properties:</div><div><br></div><div> - State Conditions:<br></div><div>Add log services related to the drive as discussed in <a href="https://lore.kernel.org/openbmc/CAHwn2XnqABuyc3uMQDs+YgwYJL+yKLwCYn1myOVdZ=gkhw3L-A@mail.gmail.com/">device level logging</a> to add debug information.</div><div><br></div><div>- PhysicalLocation</div><div>Use ` Inventory.Decorator.LocationCode` directly</div><div><br></div><div>-  Reset Action </div><div>Add reset/powercycle support to Drive resource. Based on <a href="https://gerrit.openbmc-project.xyz/c/openbmc/phosphor-dbus-interfaces/+/43155">https://gerrit.openbmc-project.xyz/c/openbmc/phosphor-dbus-interfaces/+/43155</a>.</div><div><br></div><div>---</div><div><br></div><div>Please let me know if there are any questions and suggestions on the proposed changes.</div><div><br></div><div>Best,</div><div><br></div><div>Willy Tu</div><div><br></div><div><br></div><div><br></div><div><br></div><div><br></div></div></div></div>