Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/14355 )
Change subject: [mini_chronyd] introduce MiniChronyd::address() ...................................................................... [mini_chronyd] introduce MiniChronyd::address() Introduced MiniChronyd::address() method to find IP address and port at which the underlying NTP server is listening for incoming requests. In case if the NTP server is bound to the wildcard address, use the loopback address for NTP communication with the server. Also, updated corresponding call sites to use the new method instead of accessing MiniChronyd's options. Prior to this patch, the ToolTest.TestRemoteReplicaCopy test scenario would fail if running with built-in NTP client enabled. Change-Id: I62a326a47160aefe9a5806162e6df7c4d6060ba5 Reviewed-on: http://gerrit.cloudera.org:8080/14355 Tested-by: Kudu Jenkins Reviewed-by: Alexey Serbin <[email protected]> --- M src/kudu/clock/ntp-test.cc M src/kudu/clock/test/mini_chronyd-test.cc M src/kudu/clock/test/mini_chronyd.cc M src/kudu/clock/test/mini_chronyd.h M src/kudu/mini-cluster/external_mini_cluster.cc 5 files changed, 27 insertions(+), 19 deletions(-) Approvals: Kudu Jenkins: Verified Alexey Serbin: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/14355 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I62a326a47160aefe9a5806162e6df7c4d6060ba5 Gerrit-Change-Number: 14355 Gerrit-PatchSet: 4 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120)
