Motivation

When a report is generated, we replace the existing app_telemetry_meter_impl with a new instance of a default_app_telemetry_meter_impl. This means that the existing node labels cache will be empty, until a new config is seen, and any subsequent reports will possibly have empty node and alt_node attributes.

Changes

• When creating the new default_app_telemetry_meter_impl instance, set its node labels cache to the value it had in the old app_telemetry_meter_impl instance.
• Add a shared lock to protect the impl_ pointer, as it can be modified and accessed from different threads at the same time.

Results

ApplicationTelemetrySocketTest in FIT now passes, with the exception of some assertions failing due to CXXCBC-650.