On Wed, May 19, 2021 at 2:40 PM Ed Tanous <edtanous@google.com> wrote:

On Wed, May 19, 2021 at 2:27 PM Nan Zh= ou <nanzhou@goog= le.com> wrote:
>
> Thanks for your reply, Ed!
>
>>
>> You've also omitted what properties you're trying to targe= t.=C2=A0 Could
>> you walk through a little bit about what you're trying to acco= mplish
>> with this?=C2=A0 (beyond just "implement the redfish schemas)= .=C2=A0 Are you
>> trying to get smart data?=C2=A0 just inventory information?=C2=A0 = Do you plan to
>> represent connections?
>
>
> 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.

I'm assuming these will be the dbus properties names, as redfish
doesn't really have a presence property or a powerstatus property.
With that said, that seems reasonable to me to want to do, but we need
to make sure we're clear on how this differs from the existing Drive interfaces.

>
> On Wed, May 19, 2021 at 1:43 PM Ed Tanous <ed@tanous.net> wrote:
>>
>> On Wed, May 19, 2021 at 12:57 AM Jun-Lin Chen (=E9=99=B3=E4=BF=8A= =E9=9C=96)
>> <Jun-Lin.Chen@quantatw.com> wrote:
>> >
>> > Hi all,
>> >
>> >
>> >
>> > In order to implement the feature to view simple storage devi= ce status, such SATA HDD presence status.
>> >
>> > We propose to implement it on the Redfish SimpleStorage URI.<= br> >> >
>> >
>> >
>> > As our proposal for Redfish SimpleStorage, https://gerrit.openbmc-project.xyz/c/openbmc/docs/+/43352=
>> >
>> > And an new daemon, https://ge= rrit.openbmc-project.xyz/c/openbmc/docs/+/43351
>> >
>> >
>> >
>> > There are two new dbus-interfaces to be proposed,
>> >
>> > xyz.openbmc_project.Inventory.Item.StorageDevice: here is an = empty interface just mark the dbus path that SimpleStorage Manager should c= ollect.
>> >
>> > xyz.openbmc_project.Inventory.Item.SimpleStorage: it should h= ave the properties redfish needs, and can be parser by bmcweb.
>>
>>
>> How does this differ from xyz.openbmc_project.Inventory.Item.Drive= ?
>> They look fairly similar;=C2=A0 We generally don't name dbus i= nterfaces
>> after Redfish directly.=C2=A0 We should reuse what already exists = where
>> possible, and only invent new things where needed.
>>
>> You've also omitted what properties you're trying to targe= t.=C2=A0 Could
>> you walk through a little bit about what you're trying to acco= mplish
>> with this?=C2=A0 (beyond just "implement the redfish schemas)= .=C2=A0 Are you
>> trying to get smart data?=C2=A0 just inventory information?=C2=A0 = Do you plan to
>> represent connections?
>>
>> >
>> >
>> >
>> > The details are in the proposal.
>> >
>> >
>> >
>> > Welcome and thank you for comments and suggestions
>> >
>> >
>> >
>> > Best regards,
>> >
>> > Jun-Lin Chen
>> >
>> >