From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-10.2 required=3.0 tests=BAYES_00,DKIMWL_WL_MED, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS, HTML_MESSAGE,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED, USER_IN_DEF_DKIM_WL autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 3FCCDC433ED for ; Wed, 19 May 2021 21:27:30 +0000 (UTC) Received: from lists.ozlabs.org (lists.ozlabs.org [112.213.38.117]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id C23E061074 for ; Wed, 19 May 2021 21:27:29 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org C23E061074 Authentication-Results: mail.kernel.org; dmarc=fail (p=reject dis=none) header.from=google.com Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=openbmc-bounces+openbmc=archiver.kernel.org@lists.ozlabs.org Received: from boromir.ozlabs.org (localhost [IPv6:::1]) by lists.ozlabs.org (Postfix) with ESMTP id 4FlmDJ2J0fz2yyv for ; Thu, 20 May 2021 07:27:28 +1000 (AEST) Authentication-Results: lists.ozlabs.org; dkim=pass (2048-bit key; unprotected) header.d=google.com header.i=@google.com header.a=rsa-sha256 header.s=20161025 header.b=u6Z9TM6Q; dkim-atps=neutral Authentication-Results: lists.ozlabs.org; spf=pass (sender SPF authorized) smtp.mailfrom=google.com (client-ip=2607:f8b0:4864:20::b2b; helo=mail-yb1-xb2b.google.com; envelope-from=nanzhou@google.com; receiver=) Authentication-Results: lists.ozlabs.org; dkim=pass (2048-bit key; unprotected) header.d=google.com header.i=@google.com header.a=rsa-sha256 header.s=20161025 header.b=u6Z9TM6Q; dkim-atps=neutral Received: from mail-yb1-xb2b.google.com (mail-yb1-xb2b.google.com [IPv6:2607:f8b0:4864:20::b2b]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by lists.ozlabs.org (Postfix) with ESMTPS id 4FlmCr1Wmzz2xZp for ; Thu, 20 May 2021 07:27:01 +1000 (AEST) Received: by mail-yb1-xb2b.google.com with SMTP id n83so9129990ybg.0 for ; Wed, 19 May 2021 14:27:01 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20161025; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=2ImZh2ow7Pv3AOqSLp+Os8wR5DfAX51HKmjCNtgKO00=; b=u6Z9TM6QP8m4SM7jOGK4rMMkQ9fPOoENv9IdenPlnbEpfEbhAucpI36t+9WXGeO/gz 9YX0as7qKqDiekz7688weqKDIi4UIK+91gnURs/O7fQ8upEOuuEon55Bncrb15qRTJWK QVTKk0TfnRgeGjkaBGijjl7oVjeZz9HBlzceFArOoCq6oofR5eC0IVCkdIQMoawgwdQx mlT5Bv0cLhxwtqU8tFCnN1VF2K7N70zjW2BlkESP5tA+NghRY3DudzeMVusXGMxhiWdP aZy64gSjwbBkmz6P/KDtMFqsEDF2ogCicg3WB42xLumbPfLGTMc4JzgARSvhVc2eogD4 iLhw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=2ImZh2ow7Pv3AOqSLp+Os8wR5DfAX51HKmjCNtgKO00=; b=F8D1PbgnjmbNDDLGcGzVYOOjvc4Bs1+HyPiZSvwIdHBmCgJWXS6I8z1U4eeJ1+I2cJ pA8sZcZoD7e500BpmRsSn+QKBIKnpytGHy/JPKlHO31XWst46LwMKc6GcIBgVddpCIYl VbsrX5Uhvl3HjZi0rRGEI9VFVLuYqIoYkdUm1V6qRgD3zZoft3nntAJjtSizWx4zuCko m/I72rzP+UonGrBbDUunnCMJUfFjt8+CAlK1QqsgiIknQTbntENgvFn4XNjmzn65m1wv pcsmOfbcRyWth3cKusw0/cPzN82zzQ68Ge2QvT1lyyXuHtKez5FIkUbdAv3kMdFwFlLn EMqA== X-Gm-Message-State: AOAM531KM+HmHG+7AdmbId0laNrR/RIrRcHSNDYl+HEv+AFKpnKkI//A WrX5knyBIrL5c2TXTCEHjjrkPAn6Yk0wZzSBt4RiMNm5qignwQ== X-Google-Smtp-Source: ABdhPJyRnYGHJLuTeC6UfjTRzKDuoxCS7evBVnLTopDw1coSUsYIfHdM3PzzXaAsApNsfcTUm9k6rWalyCz2dWAKa4s= X-Received: by 2002:a25:bad0:: with SMTP id a16mr2333582ybk.441.1621459617909; Wed, 19 May 2021 14:26:57 -0700 (PDT) MIME-Version: 1.0 References: In-Reply-To: From: Nan Zhou Date: Wed, 19 May 2021 14:26:47 -0700 Message-ID: Subject: Re: SimpleStorage proposals To: "openbmc@lists.ozlabs.org" Content-Type: multipart/alternative; boundary="0000000000008b922105c2b57c01" X-BeenThere: openbmc@lists.ozlabs.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Development list for OpenBMC List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: =?UTF-8?B?SnVuLUxpbiBDaGVuICjpmbPkv4rpnJYp?= , "bradleyb@fuzziesquirrel.com" Errors-To: openbmc-bounces+openbmc=archiver.kernel.org@lists.ozlabs.org Sender: "openbmc" --0000000000008b922105c2b57c01 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Thanks for your reply, Ed! > 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? 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. On Wed, May 19, 2021 at 1:43 PM Ed Tanous wrote: > On Wed, May 19, 2021 at 12:57 AM Jun-Lin Chen (=E9=99=B3=E4=BF=8A=E9=9C= =96) > 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 colle= ct. > > > > 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 > > > > > --0000000000008b922105c2b57c01 Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable
Thanks for your reply, Ed!
=C2=A0
You've also omitted = what properties you're trying to target.=C2=A0 Could
you walk throug= h a little bit about what you're trying to accomplish
with this?=C2= =A0 (beyond just "implement the redfish schemas).=C2=A0 Are you
try= ing to get smart data?=C2=A0 just inventory information?=C2=A0 Do you plan = to
represent connections?
=C2=A0
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.
On We= d, May 19, 2021 at 1:43 PM Ed Tanous <e= d@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 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.open= bmc-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 int= erface just mark the dbus path that SimpleStorage Manager should collect. >
> xyz.openbmc_project.Inventory.Item.SimpleStorage: it should have the p= roperties 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 interfaces=
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 target.=C2=A0 = Could
you walk through a little bit about what you're trying to accomplish with this?=C2=A0 (beyond just "implement the redfish schemas).=C2=A0 A= re you
trying to get smart data?=C2=A0 just inventory information?=C2=A0 Do you pl= an to
represent connections?

>
>
>
> The details are in the proposal.
>
>
>
> Welcome and thank you for comments and suggestions
>
>
>
> Best regards,
>
> Jun-Lin Chen
>
>
--0000000000008b922105c2b57c01--