Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/14691 )
Change subject: KUDU-2994: add NO_CHRONY flag ...................................................................... Patch Set 1: (4 comments) http://gerrit.cloudera.org:8080/#/c/14691/1//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/14691/1//COMMIT_MSG@9 PS1, Line 9: NO_CHONY > NO_CHRONY Done http://gerrit.cloudera.org:8080/#/c/14691/1//COMMIT_MSG@20 PS1, Line 20: WIP: should we remove all the MiniChronyd as well if NO_CHRONY : is specified? I tried to do so, and it works but requires : added several #ifdefs into external_mini_cluster.{cc,h}. > It's less important because MiniChrony is only loosely coupled with chrony Thank you for the feedback. Done. http://gerrit.cloudera.org:8080/#/c/14691/1/build-support/mini-cluster/build_mini_cluster_binaries.sh File build-support/mini-cluster/build_mini_cluster_binaries.sh: http://gerrit.cloudera.org:8080/#/c/14691/1/build-support/mini-cluster/build_mini_cluster_binaries.sh@110 PS1, Line 110: -DNO_TESTS=1 -DNO_CHRONY=1 \ > Might want to expand on the comment on L104 to rationalize this. Done http://gerrit.cloudera.org:8080/#/c/14691/1/src/kudu/mini-cluster/external_mini_cluster.cc File src/kudu/mini-cluster/external_mini_cluster.cc: http://gerrit.cloudera.org:8080/#/c/14691/1/src/kudu/mini-cluster/external_mini_cluster.cc@120 PS1, Line 120: num_ntp_servers(0) > Do we want to allow using the built-in NTP client _without_ MiniChronyd? I Yep, we could allow for an extra option to specify some external NTP servers to run the built-in NTP client of external mini-cluster tests. It might be a viable option, but it's slower compared with MiniChronyd and a dedicated chronyd NTP server as a part of the mini-cluster. I think that behavior might be controlled by an environment variable, where the variable would contain the set of external NTP servers to point the built-in NTP client to. -- To view, visit http://gerrit.cloudera.org:8080/14691 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I95495f8a2418546048e2ea28c4f44cb399f08fd3 Gerrit-Change-Number: 14691 Gerrit-PatchSet: 1 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Tue, 12 Nov 2019 08:23:48 +0000 Gerrit-HasComments: Yes
