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

Add section on null maps to troubleshooting page #4731

Merged
merged 2 commits into from
Jun 22, 2024

Conversation

tiffany76
Copy link
Contributor

@tiffany76 tiffany76 commented Jun 21, 2024

In the process of reworking the Collector repo docs to finish up this issue, I found that an addition had been made to the Troubleshooting.md file after I copied over the content.

This PR adds the new content to the Troubleshooting page, making it the single source of truth.


Preview: https://deploy-preview-4731--opentelemetry.netlify.app/docs/collector/troubleshooting/#collector-is-experiencing-configuration-issues

@tiffany76 tiffany76 requested review from a team and songy23 and removed request for a team June 21, 2024 23:42
Copy link
Member

@songy23 songy23 left a comment

Choose a reason for hiding this comment

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

LGTM

@theletterf theletterf merged commit 9218cac into open-telemetry:main Jun 22, 2024
16 checks passed
@tiffany76 tiffany76 deleted the null-maps branch June 24, 2024 20:08
mx-psi pushed a commit to open-telemetry/opentelemetry-collector that referenced this pull request Jun 28, 2024
#### Description
Now that
[4246](open-telemetry/opentelemetry.io#4246),
[4322](open-telemetry/opentelemetry.io#4322),
and [4529](open-telemetry/opentelemetry.io#4529)
have been merged, and the new [Internal
telemetry](https://opentelemetry.io/docs/collector/internal-telemetry/)
and
[Troubleshooting](https://opentelemetry.io/docs/collector/troubleshooting/)
pages are live, it's time to clean up the underlying Collector repo docs
so that the website is the single source of truth.

I've deleted any content that was moved to the website, and linked to
the relevant sections where possible. I've consolidated what content
remains in the observability.md file and left troubleshooting.md and
monitoring.md as stubs that point to the website.

I also searched the Collector repo for cross-references to these files
and adjusted links where appropriate.

~~Note that this PR is blocked by
[4731](open-telemetry/opentelemetry.io#4731
EDIT: #4731 is merged and no longer a blocker.

<!-- Issue number if applicable -->
#### Link to tracking issue
Fixes #8886
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.

3 participants