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)
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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Builders x/build issues (builders, bots, dashboards)
Projects
None yet
Development

No branches or pull requests

3 participants