any in-progress Redfish TelemetryService enhancements?

[Wludzik, Jozef]

Hi, "Append" is on the list of to dos and should be ready till summer
(might be ready). Other feature that are described by Redfish Telemetry
Model should be ready till the end of summer too -> triggers and
collection functions.

About the issue, the metric reading is not updated when there is no
update from the sensor dbus. To debug it best way to monitor the
PropertiesChanged signal from the service that implement the sensor. If
there is no signal from service than metric won't be updated. If there
is a signal, then Telemetry service has a regression.

I would be very glad for any feedback on telemetry, even filing a bug on
Telemetry sounds great. Thanks in advance

Thanks,
Jozef

On 2/19/2021 20:33 PM, Brad Bishop wrote:
> On Thu, Feb 18, 2021 at 12:04:57PM -0800, Ed Tanous wrote:
>> On Thu, Feb 18, 2021 at 11:16 AM Brad Bishop
>> <bradleyb at fuzziesquirrel.com> wrote:
>>>
>>> Has anyone thought about how to implement the Append behaviors for the
>>> ReportUpdates property in a MetricReportDefinition?
>>>
>>
>> As an aside, if anyone else is interested in Telemetry Service,
>> there's a lot of code there that's not having a lot of community
>> feedback or testing on the reviews, so if anyone else is interested
>> please pull down the patches and provide constructive feedback.
>>
>> https://gerrit.openbmc-project.xyz/q/topic:%2522telemetry%2522+(status:open+OR+status:merged)+status:open
>>
> 
> I pulled these in and did some quick sanity checking.  I was able to
> create a report definition and then see the report get generated and
> read it.
> 
> I have to say - it was really nice being able to pull a feature in and
> having it work out of the box.  So really great job to the Intel team in
> that respect.
> 
> I did find a bug with the telemetry service itself - when reports with
> multiple metrics are created, the last (or maybe all but the first?)
> metric in the report isn't updated after the initial reading.  I was too
> lazy to file a bug and didn't save any information.  I'll try to do that
> soon.