Hello Ashwani Raina, Kudu Jenkins, Abhishek Chennaka,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/22187
to look at the new patch set (#2).
Change subject: [thirdparty] upgrade chrony up to 4.6.1 version
......................................................................
[thirdparty] upgrade chrony up to 4.6.1 version
With chrony 4.6.1, the following has changed at the Kudu side:
* The chrony-no-superuser.patch is no longer needed since a new '-U'
version is available and used to run chronyd when running tests
under regular (non-super-user) account.
* SO_REUSEPORT option is now utilized by chronyd itself for its own
purposes, but Kudu-specific patching removes the extra limitations
and guardrails to let chronyd run under the test scaffolding.
* Apparently, chronyd's behavior w.r.t. locking into time sources
if they are far apart from each other has changed between 3.5 and
4.6.1, and a few tests that were relying on chronyd's 3.5 behavior
were failing. It's not a big deal as for the overall functionality
for the Kudu built-in NTP client, so I decided to drop them since
I could not justify investing time into investigations just to have
some extra tests that were mostly relying on the quirks and features
found in chronyd 3.5.
* Since we aren't interested in exploring IPv6 support in chronyd at
this particular time, the '-4' option is added to both the chronyd
and the chronyc command lines to work only in IPv4 domain for now.
Change-Id: I3d2dc0145f4af0ff5488bef95d0ab6544038285e
---
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 thirdparty/download-thirdparty.sh
D thirdparty/patches/chrony-no-superuser.patch
M thirdparty/patches/chrony-reuseport.patch
M thirdparty/vars.sh
7 files changed, 40 insertions(+), 153 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/87/22187/2
--
To view, visit http://gerrit.cloudera.org:8080/22187
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I3d2dc0145f4af0ff5488bef95d0ab6544038285e
Gerrit-Change-Number: 22187
Gerrit-PatchSet: 2
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Abhishek Chennaka <[email protected]>
Gerrit-Reviewer: Ashwani Raina <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
