-
Notifications
You must be signed in to change notification settings - Fork 831
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
Migrate to use prow GitHub app instead of bot account #2347
Comments
I think we are using the prow bot to automatically add labels to PRs and to allow folks that don't have access to the github UI to add labels to issues as well. We are not currently using it to run CI or to merge PRs though. |
I see, then we will need to migrate, please install https://github.com/apps/google-oss-prow for this repo |
Any update yet? |
We'd been fighting some flaky tests so that got put onto the back burner. I plan to look at it next week. |
I've installed the prow app into the That took to me https://github.com/GoogleCloudPlatform/oss-test-infra/blob/master/prow/oss/onboarding.md?installation_id=20615503&setup_action=install -- but this looks like it might be the old instructions? Can you advise on next steps and also if there is cleanup that I should do for the old prow permissions (e.g. can I delete the PAT now)? |
Thank you so much! I have confirmed that it was installed. I'll take care of the rest, and there are no more actions from your end. The PAT is still used in other places of prow, I'll handle the migration in the background, and let you know when it's safe to remove the PAT |
Sounds good. Feel free to reopen this ticket or create a new one when the time comes. |
Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Part of GoogleCloudPlatform/oss-test-infra#1171
OSS prow currently uses bot personal access token(PAT) for authentication with GitHub APIs, which has a global rate limit of 5000 per hour. This is not very well scalable with many tenants. Switching over to GitHub app will get a rate limit per installation, which would greatly increase the rate limit exhaustion problem we have seen lately
Describe the solution you'd like
A clear and concise description of what you want to happen.
Install Google OSS Prow app on corresponding org/repo https://github.com/apps/google-oss-prow
OR
I'm not seeing much prow activity going on in this repo, so if it's not used any more, might be a good opportunity to remove the footprints? Please comment on this issue to let me know if this is the case, I will take care of the cleanup.
The text was updated successfully, but these errors were encountered: