-
Notifications
You must be signed in to change notification settings - Fork 44
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
permission issues #8
Comments
/kind oncall-hotlist |
Do you mean the tide components running in the OSS prow cluster cannot set contexts on -this- repo, guest-test-infra ? The only permissions we have granted on all of our prow-managed repos is to grant the google-oss-robot merge and comment permissions. If there's something else we should configure, can you explain? |
Correct, @google-oss-robot needs write access to the repo in order to update status contexts. We are seeing the following error: jsonPayload: {
component: "tide"
controller: "status-update"
error: "status code 404 not one of [201], body: {"message":"Not Found","documentation_url":"https://developer.github.com/v3/repos/statuses/#create-a-status"}"
file: "prow/tide/status.go:328"
func: "k8s.io/test-infra/prow/tide.(*statusController).setStatuses.func1"
level: "error"
msg: "Failed to set status context from "" to "pending"."
org: "GoogleCloudPlatform"
pr: 7
repo: "guest-test-infra"
sha: "1892aaa2fc554339b68fed4cecf02583eb5c1467"
} |
@google-oss-robot is an admin on this repo, so I'm not sure the issue here |
Just took over oncall today from Ben, let me investigate more and report back |
it appears tide (The merge robot) from does not have permission to set contexts on this repo https://github.com/GoogleCloudPlatform/oss-test-infra
cc @fejta
The text was updated successfully, but these errors were encountered: