Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Error in the auto-request-review workflow #1381

Open
brandtkeller opened this issue Oct 1, 2024 · 3 comments
Open

Error in the auto-request-review workflow #1381

brandtkeller opened this issue Oct 1, 2024 · 3 comments

Comments

@brandtkeller
Copy link
Collaborator

Description

Noted that there is some permutation of the auto-request-review workflow that returns an error due to required collaborator definition be met in order to add a reviewer. There must be an inconsistency somewhere.

Image

@brandtkeller
Copy link
Collaborator Author

@eddie-knight @mnm678 This may be an admin activity for mapping what exists in the auto_request_review.yml file to collaborators in the repository settings perhaps?

@mnm678
Copy link
Collaborator

mnm678 commented Oct 7, 2024

It seems like auto_request_review.yml used to be able to include non-collaborators (according to the comment at the top). I'm following up with the non-collaborators on this list to either get them removed from this list or added as collaborators for now, but we should look into what changed there.

@mrcdb
Copy link
Member

mrcdb commented Oct 9, 2024

I may be wrong, but there is an organization-level setting that allows restricting code reviews in org repos to members with explicit access only (therefore, excluding any user that is neither an org member or an external collaborator): https://docs.github.com/en/organizations/managing-organization-settings/managing-pull-request-reviews-in-your-organization .

This setting may have been enforced over time on the CNCF org level, is there any way we can get feedback on this?

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants