Skip to content

Cleanup collector-sync module#9631

Merged
theletterf merged 6 commits intoopen-telemetry:mainfrom
jaydeluca:fix-collector-scripts
Apr 13, 2026
Merged

Cleanup collector-sync module#9631
theletterf merged 6 commits intoopen-telemetry:mainfrom
jaydeluca:fix-collector-scripts

Conversation

@jaydeluca
Copy link
Copy Markdown
Member

  • I have read and followed the Contributing docs, especially the "First-time contributing?" section.
  • This PR has content that I did not fully write myself.
  • I have the experience and knowledge necessary to understand, review, and validate all content in this PR.1

The script was working but the docs and code needed some cleanup.

  • We were not running the tests in CI, so I added a new github action
  • Fixed up some python type checking errors
  • Updated docs on what the correct commands needed for local development and running tests
  • Deleted the spell check part of the script because it's no longer needed now that we are using the shortcode approach (Make collector component docs more translatable #9402)
  • Fixed a big where we weren't checking the data/collector-versions.yml file for diffs when determining if a PR should be made

Footnotes

  1. Yes, I can answer maintainer questions about the content of this PR, without using AI.

@jaydeluca jaydeluca requested a review from a team as a code owner April 11, 2026 15:32
@otelbot-docs otelbot-docs bot added the missing:docs-approval Co-owning SIG has provided approval, PR needs approval from docs maintainer label Apr 11, 2026
@github-actions github-actions bot added CI/infra Repo CI & infrastructure and removed missing:docs-approval Co-owning SIG has provided approval, PR needs approval from docs maintainer labels Apr 11, 2026
@otelbot-docs otelbot-docs bot added the missing:docs-approval Co-owning SIG has provided approval, PR needs approval from docs maintainer label Apr 11, 2026
@otelbot-docs otelbot-docs bot requested review from a team April 11, 2026 18:50
@otelbot-docs otelbot-docs bot added the missing:sig-approval Co-owning SIG didn't provide an approval label Apr 11, 2026
| {{< component-link name="windowsperfcountersreceiver" type="receiver" repo="contrib" >}} | contrib | - | beta | - |
| {{< component-link name="windowsservicereceiver" type="receiver" repo="contrib" >}} | contrib | - | development | - |
| {{< component-link name="yanggrpcreceiver" type="receiver" repo="contrib" >}} | contrib | - | alpha | - |
| {{< component-link name="zipkinreceiver" type="receiver" repo="contrib" >}} | contrib, core, K8s | beta | - | - |
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

this change was needed due to the link check failing because this receiver has been removed. These manual updates will no longer be needed once #9410 is completed

@jaydeluca
Copy link
Copy Markdown
Member Author

/fix:refcache

@otelbot-docs
Copy link
Copy Markdown
Contributor

otelbot-docs bot commented Apr 11, 2026

fix:refcache applied successfully in run 24289321910.

@theletterf theletterf added this pull request to the merge queue Apr 13, 2026
@otelbot-docs otelbot-docs bot removed the missing:docs-approval Co-owning SIG has provided approval, PR needs approval from docs maintainer label Apr 13, 2026
Merged via the queue into open-telemetry:main with commit 2ad80c2 Apr 13, 2026
26 checks passed
tiffany76 pushed a commit to tiffany76/opentelemetry.io that referenced this pull request Apr 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI/infra Repo CI & infrastructure lang:ja missing:sig-approval Co-owning SIG didn't provide an approval

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants