Skip to content

fix: update semcov dependency and fix minor bugs#9572

Merged
tiffany76 merged 3 commits intoopen-telemetry:mainfrom
rmarra:fix/doc-receiver-builder
Apr 13, 2026
Merged

fix: update semcov dependency and fix minor bugs#9572
tiffany76 merged 3 commits intoopen-telemetry:mainfrom
rmarra:fix/doc-receiver-builder

Conversation

@rmarra
Copy link
Copy Markdown
Contributor

@rmarra rmarra commented Apr 6, 2026

Fixes #9571.

This PR fixes the documentation issues reported in #9571 in the Build a receiver tutorial (content/en/docs/collector/extend/custom-component/receiver.md).

What was fixed

  • Updated semconv usage to match go.opentelemetry.io/otel/semconv/v1.38.0:
  • Replaced invalid semconv.Attribute* constants with *Key and enum value patterns.
  • Fixed cloud provider mapping logic in fillResourceWithBackendSystem():
  • Azure/GCP checks now correctly use backend.CloudProvider (instead of backend.OSType).
  • Fixed off-by-one loop in generateTraces():
    i <= numberOfTraces -> i < numberOfTraces.
  • Removed deprecated rand.Seed(time.Now().UnixNano()) from getRandomNumber().
  • Replaced stale semantic conventions reference to Collector semconv source with the correct OTel semconv package docs.

@linux-foundation-easycla
Copy link
Copy Markdown

linux-foundation-easycla bot commented Apr 6, 2026

CLA Signed

The committers listed above are authorized under a signed CLA.

  • ✅ login: otelbot[bot] / name: otelbot (a918e71)
  • ✅ login: rmarra / name: Raphael Marra (fefd036)
  • ✅ login: tiffany76 / name: Tiffany Hrabusa (1fdd730)

@otelbot-docs otelbot-docs bot requested review from a team and bogdandrutu and removed request for a team April 6, 2026 03:04
@otelbot-docs otelbot-docs bot added missing:docs-approval Co-owning SIG has provided approval, PR needs approval from docs maintainer missing:sig-approval Co-owning SIG didn't provide an approval labels Apr 6, 2026
@rmarra rmarra force-pushed the fix/doc-receiver-builder branch from 577b0f0 to b4f25de Compare April 6, 2026 03:07
@otelbot-docs otelbot-docs bot requested a review from a team April 6, 2026 03:07
@theletterf
Copy link
Copy Markdown
Member

@open-telemetry/collector-approvers Please have a look, thanks!

@otelbot-docs otelbot-docs bot removed the missing:docs-approval Co-owning SIG has provided approval, PR needs approval from docs maintainer label Apr 6, 2026
@vitorvasc
Copy link
Copy Markdown
Member

/fix:all

@otelbot-docs
Copy link
Copy Markdown
Contributor

otelbot-docs bot commented Apr 6, 2026

fix:all applied successfully in run 24040173934.

@dmathieu
Copy link
Copy Markdown
Member

dmathieu commented Apr 7, 2026

This PR deletes a blog post file when it shouldn't.

Copy link
Copy Markdown
Contributor

@jade-guiton-dd jade-guiton-dd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Other than the deleted blog post Damien mentioned (which seems to be from the fix:all auto-commit so maybe it's expected?), this looks good to me on the Collector side. Thank you for keeping things up-to-date!

@ManniMansoor

This comment was marked as spam.

@ManniMansoor

This comment was marked as spam.

@otelbot-docs otelbot-docs bot added ready-to-be-merged This PR is ready to be merged by a maintainer and removed missing:sig-approval Co-owning SIG didn't provide an approval labels Apr 7, 2026
@rmarra rmarra force-pushed the fix/doc-receiver-builder branch from 5bb596e to fefd036 Compare April 8, 2026 16:25
@otelbot-docs otelbot-docs bot requested a review from a team April 8, 2026 16:25
@rmarra
Copy link
Copy Markdown
Contributor Author

rmarra commented Apr 8, 2026

Guys, I’ve removed the deletion commit and forced a rebase. As @jade-guiton-dd mentioned, it was introduced by fix:all. Should I have left it in?

Copy link
Copy Markdown
Member

@tiffany76 tiffany76 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks very much @rmarra!

@tiffany76 tiffany76 added the sig:collector:refactor These issues are part of the Collector documentation refactoring project. label Apr 13, 2026
@tiffany76
Copy link
Copy Markdown
Member

/fix

@otelbot-docs
Copy link
Copy Markdown
Contributor

otelbot-docs bot commented Apr 13, 2026

fix applied successfully in run 24357242908.

@tiffany76 tiffany76 added this pull request to the merge queue Apr 13, 2026
Merged via the queue into open-telemetry:main with commit 8b6a868 Apr 13, 2026
24 checks passed
@otelbot
Copy link
Copy Markdown
Contributor

otelbot bot commented Apr 13, 2026

Thank you for your contribution @rmarra! 🎉 We would like to hear from you about your experience contributing to OpenTelemetry by taking a few minutes to fill out this survey.

tiffany76 added a commit to tiffany76/opentelemetry.io that referenced this pull request Apr 13, 2026
Co-authored-by: otelbot <197425009+otelbot@users.noreply.github.com>
Co-authored-by: Tiffany Hrabusa <30397949+tiffany76@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-to-be-merged This PR is ready to be merged by a maintainer sig:collector:refactor These issues are part of the Collector documentation refactoring project. sig:collector

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

[Docs]: receiver.md tutorial: semconv constants don't exist, switch logic bug and deprecated API

7 participants