Skip to content
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

review units for metadata.yaml telemetry #10556

Closed
7 tasks done
codeboten opened this issue Jul 8, 2024 · 3 comments
Closed
7 tasks done

review units for metadata.yaml telemetry #10556

codeboten opened this issue Jul 8, 2024 · 3 comments

Comments

@codeboten
Copy link
Contributor

codeboten commented Jul 8, 2024

Many metrics generated by the collector (via mdatagen) and configured via metadata.yaml have units set to dimensionless (or "1"). We should review these to ensure they adhere to semantic conventions

List of metrics to review in this repo:

An example PR for one: #10647

@crobert-1
Copy link
Member

For reference, here's the current full list of references to unit: "1" in core:

List
cmd/mdatagen/testdata/no_type_attr.yaml:    unit: "1"
cmd/mdatagen/testdata/unused_attribute.yaml:    unit: "1"
cmd/mdatagen/testdata/invalid_type_attr.yaml:    unit: "1"
cmd/mdatagen/internal/samplereceiver/metadata.yaml:    unit: "1"
cmd/mdatagen/internal/samplereceiver/metadata.yaml:      unit: "1"
cmd/mdatagen/internal/samplereceiver/metadata.yaml:      unit: "1"
processor/processorhelper/metadata.yaml:      unit: "1"
processor/processorhelper/metadata.yaml:      unit: "1"
processor/processorhelper/metadata.yaml:      unit: "1"
processor/processorhelper/metadata.yaml:      unit: "1"
processor/processorhelper/metadata.yaml:      unit: "1"
processor/processorhelper/metadata.yaml:      unit: "1"
processor/processorhelper/metadata.yaml:      unit: "1"
processor/processorhelper/metadata.yaml:      unit: "1"
processor/processorhelper/metadata.yaml:      unit: "1"
processor/processorhelper/metadata.yaml:      unit: "1"
processor/processorhelper/metadata.yaml:      unit: "1"
processor/processorhelper/metadata.yaml:      unit: "1"
processor/batchprocessor/metadata.yaml:      unit: "1"
processor/batchprocessor/metadata.yaml:      unit: "1"
processor/batchprocessor/metadata.yaml:      unit: "1"
processor/batchprocessor/metadata.yaml:      unit: "1"
receiver/receiverhelper/metadata.yaml:      unit: "1"
receiver/receiverhelper/metadata.yaml:      unit: "1"
receiver/receiverhelper/metadata.yaml:      unit: "1"
receiver/receiverhelper/metadata.yaml:      unit: "1"
receiver/receiverhelper/metadata.yaml:      unit: "1"
receiver/receiverhelper/metadata.yaml:      unit: "1"
receiver/scraperhelper/metadata.yaml:      unit: "1"
receiver/scraperhelper/metadata.yaml:      unit: "1"
exporter/exporterhelper/metadata.yaml:      unit: "1"
exporter/exporterhelper/metadata.yaml:      unit: "1"
exporter/exporterhelper/metadata.yaml:      unit: "1"
exporter/exporterhelper/metadata.yaml:      unit: "1"
exporter/exporterhelper/metadata.yaml:      unit: "1"
exporter/exporterhelper/metadata.yaml:      unit: "1"
exporter/exporterhelper/metadata.yaml:      unit: "1"
exporter/exporterhelper/metadata.yaml:      unit: "1"
exporter/exporterhelper/metadata.yaml:      unit: "1"
exporter/exporterhelper/metadata.yaml:      unit: "1"
exporter/exporterhelper/metadata.yaml:      unit: "1"

In terms of reviewing each metric, here's the component list that we can review to track this issue's completion:

  • cmd/mdatagen/testdata
  • cmd/mdatagen/internal/samplereceiver
  • processor/processorhelper
  • processor/batchprocessor
  • receiver/receiverhelper
  • receiver/scraperhelper
  • exporter/exporterhelper

@codeboten
Copy link
Contributor Author

Thanks @crobert-1, i updated the description to include the checklist and opened an issue for each one. This should make it each to parallelize the work

codeboten added a commit that referenced this issue Jul 22, 2024
This updates units for processhelper's internal telemetry.

Related #10556

Fixes
#10651

For this PR i updated `metadata.yaml` and ran `make gogenerate`, then
`make -C processor test`

---------

Signed-off-by: Alex Boten <223565+codeboten@users.noreply.github.com>
@codeboten codeboten moved this from Todo to In Progress in Collector: v1 Jul 30, 2024
@codeboten
Copy link
Contributor Author

This is done now

@github-project-automation github-project-automation bot moved this from In Progress to Done in Collector: v1 Aug 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

No branches or pull requests

3 participants