Grant Henke has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/16885 )

Change subject: [docker] Add support for openSUSE in the Docker build
......................................................................


Patch Set 4:

(3 comments)

http://gerrit.cloudera.org:8080/#/c/16885/2/docker/bootstrap-dev-env.sh
File docker/bootstrap-dev-env.sh:

http://gerrit.cloudera.org:8080/#/c/16885/2/docker/bootstrap-dev-env.sh@185
PS2, Line 185: zypper
> OK, but then I'm curious: why do we use DEBIAN_FRONTEND for  Ubuntu/Debian
Ubuntu/Debian build hang on the kerberos and tzdata package installs if 
DEBIAN_FRONTEND isn't used. I don't see that problem here.


http://gerrit.cloudera.org:8080/#/c/16885/2/docker/bootstrap-java-env.sh
File docker/bootstrap-java-env.sh:

http://gerrit.cloudera.org:8080/#/c/16885/2/docker/bootstrap-java-env.sh@82
PS2, Line 82: clean
> To remove repo metadata as well, I guess.  As per https://en.opensuse.org/S
Done


http://gerrit.cloudera.org:8080/#/c/16885/2/docker/bootstrap-runtime-env.sh
File docker/bootstrap-runtime-env.sh:

http://gerrit.cloudera.org:8080/#/c/16885/2/docker/bootstrap-runtime-env.sh@95
PS2, Line 95: openssl
> Ah, I see -- it seems I confused run-time and development images.  Does krb
krb5-devel might be able to be simplified to a smaller runtime only dependency. 
I would like to do that in a follow up if you are okay with that.



--
To view, visit http://gerrit.cloudera.org:8080/16885
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I9c1ce282fda67db5043445a139c888bc74c44680
Gerrit-Change-Number: 16885
Gerrit-PatchSet: 4
Gerrit-Owner: Grant Henke <granthe...@apache.org>
Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Grant Henke <granthe...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Wed, 06 Jan 2021 15:14:10 +0000
Gerrit-HasComments: Yes

Reply via email to