Skip to content

Conversation

paulojmdias
Copy link
Member

@paulojmdias paulojmdias commented Sep 25, 2025

Description

This PR ports the check-codeowners workflow from the opentelemetry-collector-contrib repository.

On this side, the command referenced in the workflow is make generate-codeowners instead of make gencodeowners as is in the contrib version. In the Makefile, I added the -skipgithub to avoid giving an error like below when the members run it.

2025/09/25 22:52:36 set the environment variable `GITHUB_TOKEN` to a PAT token to authenticate

I ran gengithub, which also produced some changes in issue templates.

Like the PR #13898, I also removed the make install-tools from this workflow.

The idea to port this workflow here came from the PR open-telemetry/opentelemetry-collector-contrib#42999 discussion in Slack.

@paulojmdias paulojmdias requested a review from a team as a code owner September 25, 2025 22:03
@github-actions github-actions bot added the release:risky-change This change may affect the release label Sep 25, 2025
@codecov
Copy link

codecov bot commented Sep 25, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.68%. Comparing base (0694086) to head (ee74366).
⚠️ Report is 16 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #13908      +/-   ##
==========================================
+ Coverage   91.66%   91.68%   +0.01%     
==========================================
  Files         652      652              
  Lines       42506    42506              
==========================================
+ Hits        38964    38971       +7     
+ Misses       2734     2729       -5     
+ Partials      808      806       -2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@dmitryax dmitryax added this pull request to the merge queue Oct 7, 2025
Merged via the queue into open-telemetry:main with commit f680426 Oct 7, 2025
62 checks passed
@github-actions github-actions bot added this to the next release milestone Oct 7, 2025
@paulojmdias paulojmdias deleted the chore/ci_codeowners_condition branch October 9, 2025 13:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release:risky-change This change may affect the release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants