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
Fix gitserver bc search #10576
Fix gitserver bc search #10576
Conversation
@bparees ptal |
if err != nil { | ||
return err | ||
} | ||
|
||
matchingBuildConfigs := []*buildapi.BuildConfig{} | ||
|
||
for _, bc := range buildConfigList.Items { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
go's horrible iteration behavior strikes again, eh?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes :)
lgtm [merge] |
[Test]ing while waiting on the merge queue |
405009e
to
8bcc7c8
Compare
Evaluated for origin test up to 8bcc7c8 |
continuous-integration/openshift-jenkins/test FAILURE (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/8325/) |
#8221 |
[test] |
The Origin test job could not be run again for this pull request.
|
@stevekuznetsov why isn't the bot respecting @csrwng's flake comment? #10576 (comment) [merge] |
@stevekuznetsov nm, i guess the issue @csrwng linked didn't have the flake label. |
Evaluated for origin merge up to 8bcc7c8 |
continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/8355/) |
Fixes bug in gitserver search for matching build config (pointer to loop var).
Adds unit test to catch the bug.