ibm-openbmc / dev

Product Development Project Mgmt and Tracking
16 stars 2 forks source link

PLDM: Consume availability interface for cores #3481

Closed manojkiraneda closed 2 years ago

manojkiraneda commented 2 years ago

In the current state, the logic is bmcweb for displaying cores is that :

It checks the Present property on core dbus object to update the [Status][State] as Enabled/Absent.

In addition to to Present property, we should also use the Available property , if the Available is false, then set the [Status][State] should be set as "UnavailableOffline".

lxwinspur commented 2 years ago

review by: https://github.com/ibm-openbmc/bmcweb/pull/254

mzipse commented 2 years ago

@manojkiraneda, I see the PR 254 is merged. Can we mark this issue "Code Review in Driver"? Also, do we need this upstreamed as well?

manojkiraneda commented 2 years ago

yes, this is needed upstream , but it depends on a lot of base commits - for now..i think we can close this as its in the driver.