chrony has migration regressions from autopkgtests (disco/eoan)
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
chrony (Ubuntu) |
Fix Released
|
Medium
|
Unassigned | ||
Xenial |
Won't Fix
|
Medium
|
Unassigned | ||
Bionic |
Won't Fix
|
Medium
|
Unassigned | ||
Cosmic |
Won't Fix
|
Medium
|
Unassigned | ||
Disco |
Won't Fix
|
Medium
|
Unassigned | ||
Eoan |
Fix Released
|
Medium
|
Unassigned |
Bug Description
[Impact]
* Recent changes have caused the chrony autopkgtests to fail.
In this case upstream of chrony and the clk tests changed, which need
to be back under control to match what works reliably for Disco.
* Later versions have this fixed, backport the changes to fix it in Disco
as well
[Test Case]
* Let the autopkgtests run (which is part of the proposed migration
anyway)
Sniffs already show them completing:
https:/
[Regression Potential]
* There is no functional change, only the self-tests as well the
autopkgtest execution are changed.
The one source for a regression could be that the rebuild picks
something up that triggers a behavior change. But the PPA builds have
not shown something (at least not something obvious)
[Other Info]
* This is one of the cases where the actual package as used by the user
has no bug. I'm unsure how to proceed. Do we want to push it just to
disco-proposed but keep it there (to avoid downloads for "nothing")?
I know rbasak wanted to discuss that in the SRU team for the even worse
https:/
To some extend this come under the same banner.
* If this is denied from SRU for this reason I'd ask to add a force-
badtest as a replacement to unblock proposed migration.
---
Checking last eoan merge I realized that some tests were failing for chrony:
https:/
But eoan ran autopkgtests okay when the trigger was chrony/3.5-2ubuntu2 (this last merge):
http://
Despite having failed for the previous 12 times (eoan).
Now, for the first time, we have the same failure for disco:
http://
"""
make: Leaving directory '/tmp/autopkgte
...
110-chronyc .......
111-knownclient xxxxxxxxxxxxxxx
112-port xxxxxxxxxxxxxxx
121-orphan .......
...
SUMMARY:
TOTAL 50
PASSED 48
FAILED 2 (111-knownclient 112-port) (255 256 257 258 259 260 261 262 263 264 265 266 267 268 269 270 271 272 273 274 255 256 257 258 259 260 261 262 263 264 265 266 267 268 269 270 271 272 273 274 265)
"""
And I'm able to reproduce locally:
"""
(c)inaddy@
Testing reply to client configured as server:
network with 1*1 servers and 1 clients:
non-default settings:
client_
server_
starting node 1: OK
starting node 2: OK
running simulation: OK
checking chronyd exit:
node 1: OK
node 2: OK
checking source selection:
node 2: OK
checking port numbers in packet log:
node 1: BAD
node 2: BAD
FAIL
AND
(c)inaddy@
Testing port and acquisitionport directives:
network with 1*1 servers and 1 clients:
non-default settings:
starting node 1: OK
starting node 2: OK
running simulation: OK
checking chronyd exit:
node 1: OK
node 2: OK
checking source selection:
node 2: OK
checking mean/min incoming/outgoing packet interval:
node 1: 2.74e+02 2.74e+02 6.40e+01 6.40e+01 OK
node 2: 2.74e+02 2.74e+02 6.40e+01 6.40e+01 OK
checking clock sync time, max/rms time/freq error:
node 2: 132 9.29e-05 1.21e-06 5.77e-05 1.01e-07 OK
checking port numbers in packet log:
node 1: BAD
node 2: BAD
network with 1*1 servers and 1 clients:
non-default settings:
client_
starting node 1: OK
starting node 2: OK
running simulation: OK
checking chronyd exit:
node 1: OK
node 2: OK
checking port numbers in packet log:
node 1: BAD
node 2: BAD
FAIL
"""
When doing verification for an iproute2 bug (LP: #1831775) we faced the 1st failure in autopkgtests for chrony in disco (at least from the ones I can check from autopkgtests.
Related branches
- Rafael David Tinoco (community): Approve
- Canonical Server packageset reviewers: Pending requested
- git-ubuntu developers: Pending requested
-
Diff: 88 lines (+44/-5)5 files modifieddebian/changelog (+14/-0)
debian/patches/series (+1/-0)
debian/patches/update_processing_of_packet_log.patch (+23/-0)
debian/tests/control (+1/-1)
debian/tests/upstream-simulation-test-suite (+5/-4)
description: | updated |
Changed in chrony (Ubuntu): | |
status: | New → Confirmed |
Changed in chrony (Ubuntu Disco): | |
status: | New → Confirmed |
importance: | Undecided → Medium |
Changed in chrony (Ubuntu Eoan): | |
importance: | Undecided → Medium |
Changed in chrony (Ubuntu Disco): | |
assignee: | nobody → Rafael David Tinoco (rafaeldtinoco) |
Changed in chrony (Ubuntu Eoan): | |
assignee: | nobody → Rafael David Tinoco (rafaeldtinoco) |
Changed in chrony (Ubuntu Eoan): | |
assignee: | Rafael David Tinoco (rafaeldtinoco) → nobody |
Changed in chrony (Ubuntu Disco): | |
status: | Won't Fix → Triaged |
description: | updated |
description: | updated |
Changed in chrony (Ubuntu Disco): | |
status: | In Progress → Fix Released |
Changed in chrony (Ubuntu Disco): | |
status: | Fix Committed → Won't Fix |
Already fixed in Debian.