search

GoogleCloudPlatform / oss-test-infra

https://oss-prow.knative.dev
Apache License 2.0
49 stars 134 forks source link

Gerrit adapter fails to query changes for project `linux`. #309

Open cjwagner opened 4 years ago

cjwagner commented 4 years ago

Example:

{
  component: "gerrit"   
  error: "failed to query gerrit changes: API call to https://kunit-review.googlesource.com/a/changes/?n=5&o=CURRENT_REVISION&o=CURRENT_COMMIT&o=CURRENT_FILES&o=MESSAGES&q=project:linux failed: 403 Forbidden"   
  file: "prow/gerrit/client/client.go:276"   
  func: "k8s.io/test-infra/prow/gerrit/client.(*gerritInstanceHandler).queryAllChanges"   
  level: "error"   
  msg: "fail to query changes for project linux"   
}

Interestingly, these errors happen every minute, but only between the 42nd and 49th minutes of each hour. Outside of this window queries appear to succeed. Example:

{
  component: "gerrit"   
  file: "prow/gerrit/client/client.go:315"   
  func: "k8s.io/test-infra/prow/gerrit/client.(*gerritInstanceHandler).queryChangesForProject"   
  level: "info"   
  msg: "Find 5 changes from query [project:linux]"   
}

/kind bug

google-oss-robot commented 4 years ago

@cjwagner: The label(s) kind/bug cannot be applied, because the repository doesn't have them

In response to [this](https://github.com/GoogleCloudPlatform/oss-test-infra/issues/309): >Example: >```yaml >{ > component: "gerrit" > error: "failed to query gerrit changes: API call to https://kunit-review.googlesource.com/a/changes/?n=5&o=CURRENT_REVISION&o=CURRENT_COMMIT&o=CURRENT_FILES&o=MESSAGES&q=project:linux failed: 403 Forbidden" > file: "prow/gerrit/client/client.go:276" > func: "k8s.io/test-infra/prow/gerrit/client.(*gerritInstanceHandler).queryAllChanges" > level: "error" > msg: "fail to query changes for project linux" >} >``` >Interestingly, these errors happen every minute, but only between the 42nd and 49th minutes of each hour. >Outside of this window queries appear to succeed. Example: >```yaml >{ > component: "gerrit" > file: "prow/gerrit/client/client.go:315" > func: "k8s.io/test-infra/prow/gerrit/client.(*gerritInstanceHandler).queryChangesForProject" > level: "info" > msg: "Find 5 changes from query [project:linux]" >} >``` >/kind bug Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md). If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository.