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

feat(tracing): bold information about changes #2838

Merged
merged 4 commits into from
Jan 27, 2023
Merged

Conversation

mat-rumian
Copy link
Contributor

@mat-rumian mat-rumian commented Jan 27, 2023

Describe better changes for tracing/instrumentation.

Fixes: SumoLogic/sumologic-kubernetes-tools#445

Checklist

  • Changelog updated or skip changelog label added
  • Documentation updated
  • Template tests added for new features
  • Integration tests added or modified for major features

@mat-rumian mat-rumian requested a review from a team as a code owner January 27, 2023 14:06
@github-actions github-actions bot added the documentation documentation label Jan 27, 2023
@@ -298,6 +298,7 @@ The above special configuration values can be replaced either to direct values o

**When?**: If tracing is enabled

**IMPORTANT:** `otelcol` Deployment was moved to `tracesSampler` Deployment.

Choose a reason for hiding this comment

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

How about using

Suggested change
**IMPORTANT:** `otelcol` Deployment was moved to `tracesSampler` Deployment.
> **Warning**
> `otelcol` Deployment was moved to `tracesSampler` Deployment.

as per community/community#16925?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks! :) Applied recommendation

@mat-rumian mat-rumian merged commit 9762ab0 into main Jan 27, 2023
@mat-rumian mat-rumian deleted the more-info-tracing branch January 27, 2023 14:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[update-collection-v3] otelagent, otelcol, otelgateway configuration is lost during migration
2 participants