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

x/build: LUCI disallow non maintainers to run CI on CLs they do not own or commit when trybots used to #63390

Closed
Jorropo opened this issue Oct 5, 2023 · 2 comments
Labels
Builders x/build issues (builders, bots, dashboards) FrozenDueToAge
Milestone

Comments

@Jorropo
Copy link
Member

Jorropo commented Oct 5, 2023

I was lurking and looked at https://go-review.googlesource.com/c/go/+/532895 and it look good to me so I've tried to start CI so it would already be green (or have chanced to be fixed) before someone with merge powers would take a look however LUCI denied it (when trybots used to allow it).

Even tho this case is different, this is a really useful feature I was using to start trybots on colleagues CLs which are not used to submit patches to go and don't have access to trybots. Now I guess can't do that anymore, have to spend a few days asking for permissions or waiting for someone with merge powers to look at it.

@gopherbot gopherbot added the Builders x/build issues (builders, bots, dashboards) label Oct 5, 2023
@gopherbot gopherbot added this to the Unreleased milestone Oct 5, 2023
@mateusz834
Copy link
Member

#62332

@Jorropo
Copy link
Member Author

Jorropo commented Oct 5, 2023

indeed thx

@Jorropo Jorropo closed this as not planned Won't fix, can't repro, duplicate, stale Oct 5, 2023
@golang golang locked and limited conversation to collaborators Oct 4, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Builders x/build issues (builders, bots, dashboards) FrozenDueToAge
Projects
None yet
Development

No branches or pull requests

3 participants