You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@kudu.apache.org by "Alexey Serbin (Code Review)" <ge...@cloudera.org> on 2019/11/20 05:41:51 UTC
[kudu-CR] [clock] run TestNtpDiagnostics against chrony as well
Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/14751
Change subject: [clock] run TestNtpDiagnostics against chrony as well
......................................................................
[clock] run TestNtpDiagnostics against chrony as well
This patch updates the HybridClockTest.TestNtpDiagnostics scenario to
successfully run at machines where 'chronyd' is used as the system
NTP daemon (i.e. rely on the utility suite from the 'chrony' package).
Prior to this patch, the scenario was able to run only against machines
with 'ntpd' as the system daemon (i.e. relied on the utility suite from
the 'ntp' package).
Change-Id: If9c14e939c2722304ce6c5a9f7fc812af183b353
---
M src/kudu/clock/hybrid_clock-test.cc
1 file changed, 2 insertions(+), 2 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/51/14751/1
--
To view, visit http://gerrit.cloudera.org:8080/14751
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: If9c14e939c2722304ce6c5a9f7fc812af183b353
Gerrit-Change-Number: 14751
Gerrit-PatchSet: 1
Gerrit-Owner: Alexey Serbin <as...@cloudera.com>
[kudu-CR] [clock] run TestNtpDiagnostics against chrony as well
Posted by "Adar Dembo (Code Review)" <ge...@cloudera.org>.
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/14751 )
Change subject: [clock] run TestNtpDiagnostics against chrony as well
......................................................................
Patch Set 1: Code-Review+2
--
To view, visit http://gerrit.cloudera.org:8080/14751
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: If9c14e939c2722304ce6c5a9f7fc812af183b353
Gerrit-Change-Number: 14751
Gerrit-PatchSet: 1
Gerrit-Owner: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Wed, 20 Nov 2019 05:51:13 +0000
Gerrit-HasComments: No
[kudu-CR] [clock] run TestNtpDiagnostics against chrony as well
Posted by "Alexey Serbin (Code Review)" <ge...@cloudera.org>.
Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/14751 )
Change subject: [clock] run TestNtpDiagnostics against chrony as well
......................................................................
[clock] run TestNtpDiagnostics against chrony as well
This patch updates the HybridClockTest.TestNtpDiagnostics scenario to
successfully run at machines where 'chronyd' is used as the system
NTP daemon (i.e. rely on the utility suite from the 'chrony' package).
Prior to this patch, the scenario was able to run only against machines
with 'ntpd' as the system daemon (i.e. relied on the utility suite from
the 'ntp' package).
Change-Id: If9c14e939c2722304ce6c5a9f7fc812af183b353
Reviewed-on: http://gerrit.cloudera.org:8080/14751
Reviewed-by: Adar Dembo <ad...@cloudera.com>
Tested-by: Kudu Jenkins
---
M src/kudu/clock/hybrid_clock-test.cc
1 file changed, 2 insertions(+), 2 deletions(-)
Approvals:
Adar Dembo: Looks good to me, approved
Kudu Jenkins: Verified
--
To view, visit http://gerrit.cloudera.org:8080/14751
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: If9c14e939c2722304ce6c5a9f7fc812af183b353
Gerrit-Change-Number: 14751
Gerrit-PatchSet: 2
Gerrit-Owner: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)