* SimpleStorage proposals @ 2021-05-19 7:57 Jun-Lin Chen (陳俊霖) 2021-05-19 20:43 ` Ed Tanous 0 siblings, 1 reply; 6+ messages in thread From: Jun-Lin Chen (陳俊霖) @ 2021-05-19 7:57 UTC (permalink / raw) To: openbmc; +Cc: bradleyb, Ed Tanous, Nan Zhou [-- Attachment #1: Type: text/plain, Size: 805 bytes --] 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. The details are in the proposal. Welcome and thank you for comments and suggestions Best regards, Jun-Lin Chen [-- Attachment #2: Type: text/html, Size: 4117 bytes --] ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: SimpleStorage proposals 2021-05-19 7:57 SimpleStorage proposals Jun-Lin Chen (陳俊霖) @ 2021-05-19 20:43 ` Ed Tanous 2021-05-19 21:26 ` Nan Zhou 0 siblings, 1 reply; 6+ messages in thread From: Ed Tanous @ 2021-05-19 20:43 UTC (permalink / raw) To: Jun-Lin Chen (陳俊霖); +Cc: openbmc, bradleyb, Nan Zhou 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 > > ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: SimpleStorage proposals 2021-05-19 20:43 ` Ed Tanous @ 2021-05-19 21:26 ` Nan Zhou 2021-05-19 21:40 ` Ed Tanous 0 siblings, 1 reply; 6+ messages in thread From: Nan Zhou @ 2021-05-19 21:26 UTC (permalink / raw) To: openbmc; +Cc: Jun-Lin Chen (陳俊霖), bradleyb [-- Attachment #1: Type: text/plain, Size: 2251 bytes --] 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 <ed@tanous.net> wrote: > 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 > > > > > [-- Attachment #2: Type: text/html, Size: 3225 bytes --] ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: SimpleStorage proposals 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 (陳俊霖) 0 siblings, 2 replies; 6+ messages in thread From: Ed Tanous @ 2021-05-19 21:40 UTC (permalink / raw) To: Nan Zhou; +Cc: openbmc, bradleyb, Jun-Lin Chen (陳俊霖) On Wed, May 19, 2021 at 2:27 PM Nan Zhou <nanzhou@google.com> wrote: > > 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. 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 (陳俊霖) >> <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 >> > >> > ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: SimpleStorage proposals 2021-05-19 21:40 ` Ed Tanous @ 2021-05-19 22:14 ` Nan Zhou 2021-05-21 1:51 ` Jun-Lin Chen (陳俊霖) 1 sibling, 0 replies; 6+ messages in thread From: Nan Zhou @ 2021-05-19 22:14 UTC (permalink / raw) Cc: openbmc, bradleyb, Jun-Lin Chen (陳俊霖) [-- Attachment #1: Type: text/plain, Size: 4292 bytes --] > > > > > 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. > 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. Hi Ed, the presence I am talking about is kind of a logical concept. You can find the dbus and redfish properties in the proposal. I believe we plan to use the following mappings, dbus redifish logical State(enabled) Device#Status#State If state is "enabled", the device is considered present. Health(OK) Device#Status#Heatlth(OK) If health is "OK", the device is considered present and its current status is OK (its power should be OK at least). 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 Zhou <nanzhou@google.com> wrote: > > > > 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. > > 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 (陳俊霖) > >> <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 > >> > > >> > > [-- Attachment #2: Type: text/html, Size: 6762 bytes --] ^ permalink raw reply [flat|nested] 6+ messages in thread
* RE: SimpleStorage proposals 2021-05-19 21:40 ` Ed Tanous 2021-05-19 22:14 ` Nan Zhou @ 2021-05-21 1:51 ` Jun-Lin Chen (陳俊霖) 1 sibling, 0 replies; 6+ messages in thread From: Jun-Lin Chen (陳俊霖) @ 2021-05-21 1:51 UTC (permalink / raw) To: Ed Tanous, Nan Zhou; +Cc: openbmc, bradleyb -----Original Message----- From: Ed Tanous <edtanous@google.com> Sent: Thursday, May 20, 2021 5:41 AM To: Nan Zhou <nanzhou@google.com> Cc: openbmc@lists.ozlabs.org; Jun-Lin Chen (陳俊霖) <Jun-Lin.Chen@quantatw.com>; bradleyb@fuzziesquirrel.com Subject: Re: SimpleStorage proposals On Wed, May 19, 2021 at 2:27 PM Nan Zhou <nanzhou@google.com> wrote: > > 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. 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. After question in redfish forum, https://redfishforum.com/thread/483/different-storage-simplestorage We got SimpleStorage was the early model in Redfish for showing storage devices, and "Devices" in this resource do equate to drives. We would use xyz.openbmc_project.Inventory.Drive instead xyz.openbmc_project.Inventory.SimpleStorage to represent our storage device. > > 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 (陳俊霖) >> <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://apc01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fg >> > errit.openbmc-project.xyz%2Fc%2Fopenbmc%2Fdocs%2F%2B%2F43352&da >> > ta=04%7C01%7CJun-Lin.Chen%40quantatw.com%7Cd419ece6a8d344d93bbb08d9 >> > 1b0ec5c4%7C179b032707fc4973ac738de7313561b2%7C1%7C0%7C6375705731225 >> > 16364%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiL >> > CJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&sdata=BvHa13JAozhhau6qxQqs >> > 8XMe1BX1jMi057h%2BFZ67lJk%3D&reserved=0 >> > >> > And an new daemon, >> > https://apc01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fg >> > errit.openbmc-project.xyz%2Fc%2Fopenbmc%2Fdocs%2F%2B%2F43351&da >> > ta=04%7C01%7CJun-Lin.Chen%40quantatw.com%7Cd419ece6a8d344d93bbb08d9 >> > 1b0ec5c4%7C179b032707fc4973ac738de7313561b2%7C1%7C0%7C6375705731225 >> > 26352%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiL >> > CJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&sdata=c7TSGNJC5BlMxWwmFBQH >> > X3gHH8xbPEu53hsNHun04Jo%3D&reserved=0 >> > >> > >> > >> > 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 >> > >> > ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2021-05-24 1:25 UTC | newest] Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2021-05-19 7:57 SimpleStorage proposals Jun-Lin Chen (陳俊霖) 2021-05-19 20:43 ` Ed Tanous 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 (陳俊霖)
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.