All of lore.kernel.org
 help / color / mirror / Atom feed
From: Ed Tanous <ed@tanous.net>
To: "Jun-Lin Chen (陳俊霖)" <Jun-Lin.Chen@quantatw.com>
Cc: "openbmc@lists.ozlabs.org" <openbmc@lists.ozlabs.org>,
	"bradleyb@fuzziesquirrel.com" <bradleyb@fuzziesquirrel.com>,
	Nan Zhou <nanzhou@google.com>
Subject: Re: SimpleStorage proposals
Date: Wed, 19 May 2021 13:43:28 -0700	[thread overview]
Message-ID: <CACWQX81Xnpmq7x5S2E=wnRfx=0ob_0zfchA7azks=voVqQ9-Rw@mail.gmail.com> (raw)
In-Reply-To: <HK0PR04MB33460E66D59FF4F2DF64CBDDB12B9@HK0PR04MB3346.apcprd04.prod.outlook.com>

On Wed, May 19, 2021 at 12:57 AM Jun-Lin Chen (陳俊霖)
<Jun-Lin.Chen@quantatw.com> wrote:
>
> Hi all,
>
>
>
> In order to implement the feature to view simple storage device status, such SATA HDD presence status.
>
> We propose to implement it on the Redfish SimpleStorage URI.
>
>
>
> As our proposal for Redfish SimpleStorage, https://gerrit.openbmc-project.xyz/c/openbmc/docs/+/43352
>
> And an new daemon, https://gerrit.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 collect.
>
> xyz.openbmc_project.Inventory.Item.SimpleStorage: it should have 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;  We generally don't name dbus interfaces
after Redfish directly.  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 target.  Could
you walk through a little bit about what you're trying to accomplish
with this?  (beyond just "implement the redfish schemas).  Are you
trying to get smart data?  just inventory information?  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
>
>

  reply	other threads:[~2021-05-19 20:44 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-05-19  7:57 SimpleStorage proposals Jun-Lin Chen (陳俊霖)
2021-05-19 20:43 ` Ed Tanous [this message]
2021-05-19 21:26   ` Nan Zhou
2021-05-19 21:40     ` Ed Tanous
2021-05-19 22:14       ` Nan Zhou
2021-05-21  1:51       ` Jun-Lin Chen (陳俊霖)

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='CACWQX81Xnpmq7x5S2E=wnRfx=0ob_0zfchA7azks=voVqQ9-Rw@mail.gmail.com' \
    --to=ed@tanous.net \
    --cc=Jun-Lin.Chen@quantatw.com \
    --cc=bradleyb@fuzziesquirrel.com \
    --cc=nanzhou@google.com \
    --cc=openbmc@lists.ozlabs.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.