<div dir="ltr"><div><div>Thanks for your reply, Ed!</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">You've also omitted what properties you're trying to target.  Could<br>you walk through a little bit about what you're trying to accomplish<br>with this?  (beyond just "implement the redfish schemas).  Are you<br>trying to get smart data?  just inventory information?  Do you plan to<br>represent connections?</blockquote><div> </div></div>As a first step, we are only going to implement three properties, Name, Presence, and PowerStatus. We don't plan to represent connections at the moment.</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, May 19, 2021 at 1:43 PM Ed Tanous <<a href="mailto:ed@tanous.net">ed@tanous.net</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Wed, May 19, 2021 at 12:57 AM Jun-Lin Chen (陳俊霖)<br>
<<a href="mailto:Jun-Lin.Chen@quantatw.com" target="_blank">Jun-Lin.Chen@quantatw.com</a>> wrote:<br>
><br>
> Hi all,<br>
><br>
><br>
><br>
> In order to implement the feature to view simple storage device status, such SATA HDD presence status.<br>
><br>
> We propose to implement it on the Redfish SimpleStorage URI.<br>
><br>
><br>
><br>
> As our proposal for Redfish SimpleStorage, <a href="https://gerrit.openbmc-project.xyz/c/openbmc/docs/+/43352" rel="noreferrer" target="_blank">https://gerrit.openbmc-project.xyz/c/openbmc/docs/+/43352</a><br>
><br>
> And an new daemon, <a href="https://gerrit.openbmc-project.xyz/c/openbmc/docs/+/43351" rel="noreferrer" target="_blank">https://gerrit.openbmc-project.xyz/c/openbmc/docs/+/43351</a><br>
><br>
><br>
><br>
> There are two new dbus-interfaces to be proposed,<br>
><br>
> xyz.openbmc_project.Inventory.Item.StorageDevice: here is an empty interface just mark the dbus path that SimpleStorage Manager should collect.<br>
><br>
> xyz.openbmc_project.Inventory.Item.SimpleStorage: it should have the properties redfish needs, and can be parser by bmcweb.<br>
<br>
<br>
How does this differ from xyz.openbmc_project.Inventory.Item.Drive?<br>
They look fairly similar;  We generally don't name dbus interfaces<br>
after Redfish directly.  We should reuse what already exists where<br>
possible, and only invent new things where needed.<br>
<br>
You've also omitted what properties you're trying to target.  Could<br>
you walk through a little bit about what you're trying to accomplish<br>
with this?  (beyond just "implement the redfish schemas).  Are you<br>
trying to get smart data?  just inventory information?  Do you plan to<br>
represent connections?<br>
<br>
><br>
><br>
><br>
> The details are in the proposal.<br>
><br>
><br>
><br>
> Welcome and thank you for comments and suggestions<br>
><br>
><br>
><br>
> Best regards,<br>
><br>
> Jun-Lin Chen<br>
><br>
><br>
</blockquote></div>