Work around docker race condition when running build post commit hooks.

[jim-minter]

https://bugzilla.redhat.com/show_bug.cgi?id=1425824

[csrwng]

👍 I didn't realize we were doing a logs call instead of attach. Makes sense.

[jim-minter]

Tested with fork AMI and seems to solve the problem.

[jim-minter]

@bparees this should go into release-1.5 as well - does that require a second PR? Once this has merged or concurrently?

[bparees]

@bparees this should go into release-1.5 as well - does that require a second PR? Once this has merged or concurrently?

yes it needs a second pr against the release-1.5 branch, concurrent is fine.

[bparees]

[testextended][extended:core(builds)]

[bparees]

[test]

[bparees]

@jim-minter failure looks legitimate.

[bparees]

(or the merge queue is broken)

[bparees]

merge tag removed, this breaks an extended test.

[jim-minter]

Yes, sorry, repushed and hopefully fixed. Let's see the test results. Ditto on #13196.

[bparees]

@jim-minter I think the extended tests are still going to fail.

[openshift-bot]

Evaluated for origin testextended up to e97d96b

[openshift-bot]

Evaluated for origin test up to e97d96b

[openshift-bot]

continuous-integration/openshift-jenkins/test FAILURE (https://ci.openshift.redhat.com/jenkins/job/test_pull_requests_origin_future/795/) (Base Commit: ad8ea70)

[jim-minter]

flake #13068

[openshift-bot]

continuous-integration/openshift-jenkins/testextended SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin_extended/1151/) (Base Commit: ad8ea70) (Extended Tests: core(builds))

[bparees]

[merge]

[bparees]

flake #13234
[merge]

[openshift-bot]

Evaluated for origin merge up to e97d96b

[openshift-bot]

continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pull_requests_origin_future/828/) (Base Commit: 8b1ddd5) (Image: devenv-rhel7_6035)