Is your feature request related to a problem? Please describe.
Right now, users need to do this themselves example.
That is annoying and error prone. We should do it for people.
One open question is whether to replace certain characters like / with _ as we do for External Metrics labels. Is there a way for users to configure that (in the SDK or at the metrics ingestion layer) depending on the needs of their metrics provider?
Describe the solution you'd like
SDK should label worker metrics with standardized temporal_worker_deployment_name and temporal_worker_build_id labels, similar to how it automatically labels with temporal_namespace.
If the worker is unversioned, these values could be __unversioned__ (as we do in server), or just omitted or just empty string, idc.
Additional context
This will help people manage per-version metrics with more success and less setup.
Per-SDK Tickets
Is your feature request related to a problem? Please describe.
Right now, users need to do this themselves example.
That is annoying and error prone. We should do it for people.
One open question is whether to replace certain characters like
/with_as we do for External Metrics labels. Is there a way for users to configure that (in the SDK or at the metrics ingestion layer) depending on the needs of their metrics provider?Describe the solution you'd like
SDK should label worker metrics with standardized
temporal_worker_deployment_nameandtemporal_worker_build_idlabels, similar to how it automatically labels withtemporal_namespace.If the worker is unversioned, these values could be
__unversioned__(as we do in server), or just omitted or just empty string, idc.Additional context
This will help people manage per-version metrics with more success and less setup.
Per-SDK Tickets