impala-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Laszlo Gaal (Code Review)" <>
Subject [Impala-ASF-CR] Add CentOS 8.1 support to bootstrap
Date Wed, 08 Apr 2020 17:08:06 GMT
Hello Zoltan Borok-Nagy, David Knupp, Tim Armstrong, Joe McDonnell, Impala Public Jenkins,

I'd like you to reexamine a change. Please visit

to look at the new patch set (#7).

Change subject: Add CentOS 8.1 support to

Add CentOS 8.1 support to

CentOS 8.1 is a new major version of the CentOS family.
It is now stable and popular enough to start supporting it for Impala

Prepare a raw CentOS 8.1 system to support Impala development and testing.
This should work on a standalone computer, on a vrtual machine,
or inside a Docker container.

- curl is added to the list of required packages, required by
- snappy-devel moved to the PowerTools repo, so it needs to be installed
  from there
- CentOS 8 has no default Python version. The bootstrap script installs
  (or configures) Python2 with pip2, then makes them the default via the
  "alternatives" mechanism. The installer is adaptive, it performs only
  the necessry steps, so it works in various environments.
  The installer logic is also shared between bin/ and
- The toolchain package tag "ec2-centos-8" is added to
- For some unknown reason, when the downloaded Maven tarball is extracted
  in a Docker-based test, the "bin" and "boot" directories are created
  with owner-only permissions. The 'impdev' users has no access to the
  maven executable, which then breaks the build.
  This patch forcibly restores the correct permissions on these
  directories; this is a no-op when the extraction happens correctly.
- TOOLCHAIN_ID is bumped to a build that already has CentOS 8 binaries.
- Add Centos8-specific bootstrap code to the Docker-based tests.

Change-Id: I3df5d48eca7a10219264e3604a4f05f072188e6e
M bin/
M bin/
M bin/
M docker/
4 files changed, 115 insertions(+), 13 deletions(-)

  git pull ssh:// refs/changes/23/15623/7
To view, visit
To unsubscribe, visit

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I3df5d48eca7a10219264e3604a4f05f072188e6e
Gerrit-Change-Number: 15623
Gerrit-PatchSet: 7
Gerrit-Owner: Laszlo Gaal <>
Gerrit-Reviewer: David Knupp <>
Gerrit-Reviewer: Impala Public Jenkins <>
Gerrit-Reviewer: Joe McDonnell <>
Gerrit-Reviewer: Laszlo Gaal <>
Gerrit-Reviewer: Tim Armstrong <>
Gerrit-Reviewer: Zoltan Borok-Nagy <>

  • Unnamed multipart/alternative (inline, 8-Bit, 0 bytes)
View raw message