Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/17292 )
Change subject: IMPALA-10646: Add non-server RHEL 8 signature to toolchain bootstrap ...................................................................... IMPALA-10646: Add non-server RHEL 8 signature to toolchain bootstrap Recent Red Hat Enterprise Linux 8.x version return a shorter release name string from `lsb_release -si` than earlier versions. This shorter string was not recognized in the OS mapper logic in bin/bootstrap_toolchain.py, makig it -- and the build process -- break on Red Hat 8.2 The patch adds the shorter signature as a point fix. Rename a local variable to fix an unrelated name conflict (shadowing) found by flake8. Tests: run bin/bootstrap_toolchain.py manually on Red Hat 8.2, then run a complete build on the same OS. Regression-tested (build and dataload only) on the following versions: - Centos 8.2 (as opposed to Red Hat 8.2) - Centos 7.9 - Ubuntu 18.04 Change-Id: Icb1a6c215b1b5a65691042bb7d94fb034392d135 Reviewed-on: http://gerrit.cloudera.org:8080/17292 Reviewed-by: Joe McDonnell <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- M bin/bootstrap_toolchain.py 1 file changed, 4 insertions(+), 3 deletions(-) Approvals: Joe McDonnell: Looks good to me, approved Impala Public Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/17292 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Icb1a6c215b1b5a65691042bb7d94fb034392d135 Gerrit-Change-Number: 17292 Gerrit-PatchSet: 5 Gerrit-Owner: Laszlo Gaal <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Joe McDonnell <[email protected]> Gerrit-Reviewer: Laszlo Gaal <[email protected]>
