* Removing /org/openbmc/sensors
@ 2018-08-21 16:53 Adriana Kobylak
2018-08-21 17:13 ` Ed Tanous
0 siblings, 1 reply; 2+ messages in thread
From: Adriana Kobylak @ 2018-08-21 16:53 UTC (permalink / raw)
To: openbmc
Hi all,
I'm finishing up the changes to remove the /org/openbmc/sensors path.
This path just had a handful of sensors left, which with the exception
of 2, were already defined in the
phosphor-ipmi-sensor-inventory%/config.yaml files so they weren't being
used. I moved the remaining 2 to the ipmi config files, so we're ready
to deprecate the /org/openbmc/sensors path.
This change should be transparent since ipmi first looks for the
requested sensor in its generated code from the yaml file, then if not
found it looks for it in the /org/openbmc/sensors path (this 2nd lookup
in the ipmi code can be removed once all changes are merged).
If you have any concerns about this change or questions about querying
sensors outside the /org/openbmc/sensors path, you can reply or comment
in the review where the system configs files in the skeleton repo are
being updated to remove the sensors:
https://gerrit.openbmc-project.xyz/#/q/topic:org_sensors
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: Removing /org/openbmc/sensors
2018-08-21 16:53 Removing /org/openbmc/sensors Adriana Kobylak
@ 2018-08-21 17:13 ` Ed Tanous
0 siblings, 0 replies; 2+ messages in thread
From: Ed Tanous @ 2018-08-21 17:13 UTC (permalink / raw)
To: openbmc
+1
Sounds good to me.
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2018-08-21 17:12 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-08-21 16:53 Removing /org/openbmc/sensors Adriana Kobylak
2018-08-21 17:13 ` Ed Tanous
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.