Skip to content

fix: update semcov dependency and fix minor bugs#9572

Open
rmarra wants to merge 1 commit intoopen-telemetry:mainfrom
rmarra:fix/doc-receiver-builder
Open

fix: update semcov dependency and fix minor bugs#9572
rmarra wants to merge 1 commit intoopen-telemetry:mainfrom
rmarra:fix/doc-receiver-builder

Conversation

@rmarra
Copy link
Copy Markdown

@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: rmarra / name: Raphael Marra (fefd036)

@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
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?

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

Projects

Status: No status

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

6 participants