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 (#4).
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.
* It seems the behavior of finding a suitable (a.k.a. selectable)
sources among configured NTP servers has changed, and chronyd 4.6.1
is less picky than chronyd 3.5. To address that, an extra
'minsources' directive was added when running chronyd in client-only
mode as necessary.
* Apparently, chronyd's behavior w.r.t. locking into time sources
if they are far apart from each other or have unsynchronized clock
has changed between 3.5 and 4.6.1, and one test scenario that was
relying on chronyd's 3.5 behavior was failing. That was fixed.
* chronyd 4.6.1 correctly reports an error in its configuration when
seeing multiple servers that would occupy the same configuration
slot. Since it's not possible to bind and listen at arbitrary
loopback IP addresses (127.0.0.0/8) on macOS, various test scenarios
involving multiple chronyd servers aren't run on macOS anymore.
* 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, 68 insertions(+), 70 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/87/22187/4
--
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: 4
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Abhishek Chennaka <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Ashwani Raina <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
