-
Notifications
You must be signed in to change notification settings - Fork 888
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
OTEP 0152 Telemetry Schemas still uses "label" terminology for metrics #1989
Comments
+1 to this (in a few cases, when moving it to the Specification we had to amend the initially accepted changes). The second thing is that although this was published, Metrics still is experimental and as you mentioned, no changes for that portion have been made. I'd rather change it to |
Option 2 makes sense to me. |
I always assumed Option 2 myself, so I vote for that. |
Discussed in spec meeting, decided:
|
Fixes open-telemetry/opentelemetry-specification#1989 OTEP 0152 predates renaming of labels to attributes in the spec. Unfortunately the schema file format still uses the term "label" in the metrics section this way: ```yaml metrics: changes: - rename_labels: label_map: # map of key/values. The keys are the old label name used # in the previous version, the values are the new label name # starting from this version. apply_to_metrics: # Optional. If it is missing the transformation is applied # to all metrics. If it is present the transformation is applied # only to the metrics with the name that is found in the sequence # specified below. ``` We don't want the word `label` in the schema files, instead we want the word `attribute` to be used instead. Discussed in spec meeting, decided: - OTEPs which are not part of the spec repo are not considered to be final, or be an actual part of stable spec (OTEPs are not labeled "stable" in any way). - Modify the OTEP, rename `label` to `attribute` (this commit). - Merge the OTEP with the spec.
Fixes open-telemetry/opentelemetry-specification#1989 OTEP 0152 predates renaming of labels to attributes in the spec. Unfortunately the schema file format still uses the term "label" in the metrics section this way: ```yaml metrics: changes: - rename_labels: label_map: # map of key/values. The keys are the old label name used # in the previous version, the values are the new label name # starting from this version. apply_to_metrics: # Optional. If it is missing the transformation is applied # to all metrics. If it is present the transformation is applied # only to the metrics with the name that is found in the sequence # specified below. ``` We don't want the word `label` in the schema files, instead we want the word `attribute` to be used instead. Discussed in spec meeting, decided: - OTEPs which are not part of the spec repo are not considered to be final, or be an actual part of stable spec (OTEPs are not labeled "stable" in any way). - Modify the OTEP, rename `label` to `attribute` (this commit). - Merge the OTEP with the spec.
Fixes open-telemetry#1989 OTEP 0152 predates renaming of labels to attributes in the spec. Unfortunately the schema file format still uses the term "label" in the metrics section this way: ```yaml metrics: changes: - rename_labels: label_map: # map of key/values. The keys are the old label name used # in the previous version, the values are the new label name # starting from this version. apply_to_metrics: # Optional. If it is missing the transformation is applied # to all metrics. If it is present the transformation is applied # only to the metrics with the name that is found in the sequence # specified below. ``` We don't want the word `label` in the schema files, instead we want the word `attribute` to be used instead. Discussed in spec meeting, decided: - OTEPs which are not part of the spec repo are not considered to be final, or be an actual part of stable spec (OTEPs are not labeled "stable" in any way). - Modify the OTEP, rename `label` to `attribute` (this commit). - Merge the OTEP with the spec.
Fixes open-telemetry/opentelemetry-specification#1989 OTEP 0152 predates renaming of labels to attributes in the spec. Unfortunately the schema file format still uses the term "label" in the metrics section this way: ```yaml metrics: changes: - rename_labels: label_map: # map of key/values. The keys are the old label name used # in the previous version, the values are the new label name # starting from this version. apply_to_metrics: # Optional. If it is missing the transformation is applied # to all metrics. If it is present the transformation is applied # only to the metrics with the name that is found in the sequence # specified below. ``` We don't want the word `label` in the schema files, instead we want the word `attribute` to be used instead. Discussed in spec meeting, decided: - OTEPs which are not part of the spec repo are not considered to be final, or be an actual part of stable spec (OTEPs are not labeled "stable" in any way). - Modify the OTEP, rename `label` to `attribute` (this commit). - Merge the OTEP with the spec.
Fixes open-telemetry/opentelemetry-specification#1989 OTEP 0152 predates renaming of labels to attributes in the spec. Unfortunately the schema file format still uses the term "label" in the metrics section this way: ```yaml metrics: changes: - rename_labels: label_map: # map of key/values. The keys are the old label name used # in the previous version, the values are the new label name # starting from this version. apply_to_metrics: # Optional. If it is missing the transformation is applied # to all metrics. If it is present the transformation is applied # only to the metrics with the name that is found in the sequence # specified below. ``` We don't want the word `label` in the schema files, instead we want the word `attribute` to be used instead. Discussed in spec meeting, decided: - OTEPs which are not part of the spec repo are not considered to be final, or be an actual part of stable spec (OTEPs are not labeled "stable" in any way). - Modify the OTEP, rename `label` to `attribute` (this commit). - Merge the OTEP with the spec.
Fixes open-telemetry/opentelemetry-specification#1989 OTEP 0152 predates renaming of labels to attributes in the spec. Unfortunately the schema file format still uses the term "label" in the metrics section this way: ```yaml metrics: changes: - rename_labels: label_map: # map of key/values. The keys are the old label name used # in the previous version, the values are the new label name # starting from this version. apply_to_metrics: # Optional. If it is missing the transformation is applied # to all metrics. If it is present the transformation is applied # only to the metrics with the name that is found in the sequence # specified below. ``` We don't want the word `label` in the schema files, instead we want the word `attribute` to be used instead. Discussed in spec meeting, decided: - OTEPs which are not part of the spec repo are not considered to be final, or be an actual part of stable spec (OTEPs are not labeled "stable" in any way). - Modify the OTEP, rename `label` to `attribute` (this commit). - Merge the OTEP with the spec.
OTEP 0152 predates renaming of labels to attributes in the spec. Unfortunately the schema file format still uses the term "label" in the metrics section this way:
Ideally we don't want the word
label
in the schema files, instead we want the wordattribute
to be used instead. Now, the problem is that this is an accepted OTEP and we even published several schema files in the spec. Luckily, none of the publishes schema files have metric transformations.The question is can we fix this? A couple choices:
I would like to know what @open-telemetry/specs-approvers think.
The text was updated successfully, but these errors were encountered: