Attila Bukor has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/21466 )
Change subject: Update build pattern for fetching flaky tests list ...................................................................... Update build pattern for fetching flaky tests list During the infra changes which happened in the last months, the build_id prefix has been changed [note 1]. As a result flaky tests are not retried during testing in the pre-commit runs. This patch updates the build pattern in the build-and-test.sh script, such that the list of flaky tests is fetched correctly. [note 1]: The build_id of an isolated test is constructed with Jenkins job's name and build number, prefixed with "jenkins-". To illustrate this let me give some examples, for build_ids prior and after the infra upgrade. Before: jenkins-kudu-gerrit-BUILD_TYPE=ASAN-29232 jenkins-kudu-gerrit-BUILD_TYPE=DEBUG-29227 After: jenkins-build_and_test-64 jenkins-build_and_test-63 >From the above we can see that the new Jenkins job name caused the issue. The new job is parametrised, does not contain the build type in the job's name. Therefore, a change in the naming is justified. In case something like this happens in the future, the build_ids can be observed in the test_results MySQL database on the isolateserver test infra host. Change-Id: I317a3a32c06c06306b97566f954e0ffd508ce01f Reviewed-on: http://gerrit.cloudera.org:8080/21327 Reviewed-by: Ashwani Raina <[email protected]> Tested-by: Kudu Jenkins Reviewed-by: Alexey Serbin <[email protected]> (cherry picked from commit 7b5fc9f2f0ad538242656431bf76e253cc537772) Reviewed-on: http://gerrit.cloudera.org:8080/21466 Reviewed-by: Marton Greber <[email protected]> Reviewed-by: Zoltan Chovan <[email protected]> Tested-by: Attila Bukor <[email protected]> --- M build-support/jenkins/build-and-test.sh 1 file changed, 6 insertions(+), 3 deletions(-) Approvals: Marton Greber: Looks good to me, approved Ashwani Raina: Looks good to me, but someone else must approve Zoltan Chovan: Looks good to me, but someone else must approve Attila Bukor: Verified -- To view, visit http://gerrit.cloudera.org:8080/21466 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: branch-1.17.x Gerrit-MessageType: merged Gerrit-Change-Id: I317a3a32c06c06306b97566f954e0ffd508ce01f Gerrit-Change-Number: 21466 Gerrit-PatchSet: 2 Gerrit-Owner: Attila Bukor <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Ashwani Raina <[email protected]> Gerrit-Reviewer: Attila Bukor <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Marton Greber <[email protected]> Gerrit-Reviewer: Zoltan Chovan <[email protected]>
