Skip to content

[IA] Update Assisted and Automatic Instrumentation #7157

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

Open
wants to merge 42 commits into
base: main
Choose a base branch
from

Conversation

svrnm
Copy link
Member

@svrnm svrnm commented Jun 20, 2025

This PR is for #6932:

  • It renames "zero-code" to "Automatic & Assisted Instrumentation"
  • It introduces "zero-code" to be synonym with "automatic"
  • It introduces the term "assisted"
  • It adds a note to the concept page that the different "ways" of instrumentation are not having clear boundaries
  • This would add a place to document injector, profiler & OBI.

What's missing:

  • fix links that lead to /zero-code/ URLs
  • review some other pages to verify if "zero-code" or "automatic" is the right word

@svrnm svrnm requested a review from a team as a code owner June 20, 2025 12:34
@svrnm svrnm added the IA Information architecture rework label Jun 20, 2025
@opentelemetrybot opentelemetrybot requested review from a team June 20, 2025 12:34
Copy link
Member

@theletterf theletterf left a comment

Choose a reason for hiding this comment

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

Initial review! Thanks!

Copy link
Contributor

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

I left some copy edits, but it looks good to me!

svrnm and others added 16 commits June 27, 2025 11:12
Co-authored-by: opentelemetrybot <[email protected]>
Co-authored-by: Fabrizio Ferri-Benedetti <[email protected]>
Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: opentelemetrybot <[email protected]>
Co-authored-by: Fabrizio Ferri-Benedetti <[email protected]>
Co-authored-by: opentelemetrybot <[email protected]>
Co-authored-by: Patrice Chalin <[email protected]>
Co-authored-by: Severin Neumann <[email protected]>
@opentelemetrybot opentelemetrybot requested review from a team June 27, 2025 09:19
@svrnm
Copy link
Member Author

svrnm commented Jun 27, 2025

ok... I butchered this PR myself 😅 trying to fix it

@svrnm
Copy link
Member Author

svrnm commented Jun 27, 2025

hm ... we have a "bigger" problem:

ERROR Param "vers.otel" not found: "/Users/severin/Projects/opentelemetry.io/content/pt/docs/zero-code/java/agent/api.md:30:14"
ERROR Param "vers.otel" not found: "/Users/severin/Projects/opentelemetry.io/content/pt/docs/zero-code/java/agent/api.md:39:56"

With refactoring we break localizations that depend on the versions.

Maybe it is time to move those versions out from the doc files into some dedicated location, i think we discussed this in the past.

I'd like to get @chalin's perspective on that, until then this PR is blocked unfortunately.

@svrnm svrnm added the blocked label Jun 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.