Michael Smith has posted comments on this change. ( http://gerrit.cloudera.org:8080/20469 )
Change subject: IMPALA-12436: Create multi-platform builder images ...................................................................... Patch Set 7: (1 comment) http://gerrit.cloudera.org:8080/#/c/20469/6/docker/buildall.py File docker/buildall.py: http://gerrit.cloudera.org:8080/#/c/20469/6/docker/buildall.py@29 PS6, Line 29: ARM_PLATFORMS = ['redhat8', 'redhat9', 'ubuntu2004', 'ubuntu2204'] > Maybe this should be an environment variable? Although we'll need to make c Not going to change it, if we're producing multi-platform containers we should always do it for all platforms we support. -- To view, visit http://gerrit.cloudera.org:8080/20469 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: native-toolchain Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I058deb877f26b48a2d87a65710143bba9a42a5fb Gerrit-Change-Number: 20469 Gerrit-PatchSet: 7 Gerrit-Owner: Michael Smith <[email protected]> Gerrit-Reviewer: Joe McDonnell <[email protected]> Gerrit-Reviewer: Laszlo Gaal <[email protected]> Gerrit-Reviewer: Michael Smith <[email protected]> Gerrit-Comment-Date: Wed, 20 Sep 2023 17:52:20 +0000 Gerrit-HasComments: Yes
