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

[CI] Fix external links into opentelemetry-collector-contrib #5970

Conversation

chalin
Copy link
Contributor

@chalin chalin commented Jan 17, 2025

  • Contributes to [CI] Check links into GitHub #5951
  • Fixes external links into the opentelemetry-collector-contrib and opentelemetry-collector repos
  • In a blog post, pins the URLs to v0.117.0 for most links to keep them valid over time

@chalin chalin added the CI/infra CI & infrastructure label Jan 17, 2025
@chalin chalin requested a review from a team as a code owner January 17, 2025 21:02
@opentelemetrybot opentelemetrybot requested review from a team and krol3 and removed request for a team January 17, 2025 21:03
@chalin chalin force-pushed the chalin-im-rw-otel-collector-contrib-link-fixes-2025-01-17 branch from 6bbe348 to 5947766 Compare January 17, 2025 21:04
@opentelemetrybot opentelemetrybot requested a review from a team January 17, 2025 21:04
[Métricas predeterminadas](https://github.com/open-telemetry/opentelemetry-collector-contrib/blob/main/receiver/kubeletsstatsreceiver/documentation.md).
[Métricas predeterminadas](https://github.com/open-telemetry/opentelemetry-collector-contrib/blob/main/receiver/kubeletstatsreceiver/documentation.md).
Copy link
Contributor Author

@chalin chalin Jan 17, 2025

Choose a reason for hiding this comment

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

Drops the excess s for the URL to fix it.

Comment on lines -116 to +119
1. [spanmetricsconnector](https://github.com/open-telemetry/opentelemetry-collector-contrib/blob/main/extension/spanmetricsprocessor/README.md)
2. [servicegraphconnector](https://github.com/open-telemetry/opentelemetry-collector-contrib/blob/main/extension/servicegraphprocessor/README.md)
3. [routingconnector](https://github.com/open-telemetry/opentelemetry-collector-contrib/blob/main/processor/routingprocessor/README.md)
4. [countconnector](https://github.com/open-telemetry/opentelemetry-collector-contrib/blob/main/extension/countprocessor/README.md)
5. [datadogconnector](https://github.com/open-telemetry/opentelemetry-collector-contrib/blob/main/exporter/datadogexporter/README.md)
1. [spanmetricsconnector](https://github.com/open-telemetry/opentelemetry-collector-contrib/blob/v0.117.0/connector/spanmetricsconnector/README.md)
2. [servicegraphconnector](https://github.com/open-telemetry/opentelemetry-collector-contrib/blob/v0.117.0/connector/servicegraphconnector/README.md)
3. [routingconnector](https://github.com/open-telemetry/opentelemetry-collector-contrib/blob/v0.117.0/processor/routingprocessor/README.md)
4. [countconnector](https://github.com/open-telemetry/opentelemetry-collector-contrib/blob/v0.117.0/connector/countconnector/README.md)
Copy link
Contributor Author

@chalin chalin Jan 17, 2025

Choose a reason for hiding this comment

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

Fixes links of *connector components to actually refer to the connector README, not the now obsolete *processor READMEs.

@chalin chalin mentioned this pull request Jan 17, 2025
43 tasks
[metrics for the purposes of its own monitoring](https://github.com/open-telemetry/opentelemetry-collector/blob/main/docs/monitoring.md).
[metrics for the purposes of its own monitoring](/docs/collector/internal-telemetry/#use-internal-telemetry-to-monitor-the-collector).
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@chalin chalin merged commit 0a1b0ba into open-telemetry:main Jan 17, 2025
17 checks passed
@chalin chalin deleted the chalin-im-rw-otel-collector-contrib-link-fixes-2025-01-17 branch January 17, 2025 21:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blog CI/infra CI & infrastructure lang:es
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants