Vivid unit tests need to pass
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| | juju-core |
High
|
Eric Snow | ||
| | 1.23 |
High
|
Eric Snow | ||
Bug Description
We have seen the vivid unit tests pass, but that id only 1/10. We need to get this suite reliable for 1.23.0. The common test failures are:
FAIL: pinger_test.go:97: pingerSuite.
FAIL: root_test.go:40: rootSuite.
FAIL: replicaset_
FAIL: tracker_
The pingerSuite.
| Curtis Hovey (sinzui) wrote : | #1 |
| Eric Snow (ericsnowcurrently) wrote : | #2 |
This appears to be related to https:/
| description: | updated |
| Changed in juju-core: | |
| milestone: | 1.23 → 1.24-alpha1 |
| Martin Packman (gz) wrote : | #3 |
Another test run:
<http://
logs_test.go:109: LogsSuite.
worker_test.go:89: suite.TestPrune
worker_test.go:54: suite.TestPrune
(all bug 1434741 and fixed on trunk)
resumer_test.go:37: ResumerSuite.
resumer_test.go:61:
c.Assert(diff <= 4*testInterval, jc.IsTrue)
... obtained bool = false
No current bug filed? Also intermittent?
| Eric Snow (ericsnowcurrently) wrote : | #4 |
I've addressed lp:1408459 but there are other failures that need to be addressed. I expect we will bump into various transient failures as time goes by, not unlike what happens with merge-gating and with the other unit tests. Regardless, I'm aiming to at least get the test runs consistently passing ASAP, and certainly for the 1.23.0 release.
| Changed in juju-core: | |
| assignee: | nobody → Eric Snow (ericsnowcurrently) |
| Eric Snow (ericsnowcurrently) wrote : | #5 |
I've opened lp:1436397 for one of the failures. Note that it does not affect 1.23 (only master).
| Eric Snow (ericsnowcurrently) wrote : | #6 |
I've opened lp:1436407 for the (currently) only remaining test failure on 1.23.
| Changed in juju-core: | |
| status: | Triaged → In Progress |
| Eric Snow (ericsnowcurrently) wrote : | #7 |
intermittent on 1.23: lp:1398415 (old bug)
| Eric Snow (ericsnowcurrently) wrote : | #8 |
a possibly intermittent failure on 1.23: lp:1436495 (new bug)
| Eric Snow (ericsnowcurrently) wrote : | #9 |
I'd meant to say *1.24* on that last one.
| Eric Snow (ericsnowcurrently) wrote : | #10 |
Another possibly intermittent failure on 1.24: lp:1436507 (new bug)
| Eric Snow (ericsnowcurrently) wrote : | #11 |
Both 1.23 and master are now passing intermittently. Presumably we still need to get them passing much more consistenly.
| Eric Snow (ericsnowcurrently) wrote : | #12 |
On master the tests should consistently pass once the proposed fix for lp:1436397 lands (probably today).
| Eric Snow (ericsnowcurrently) wrote : | #13 |
I've logged another intermittent one for master: lp:1438274.
| Eric Snow (ericsnowcurrently) wrote : | #14 |
The tests are passing frequently enough that I'm closing this issue. An further specific failures can be addressed in new issues.
| Changed in juju-core: | |
| status: | In Progress → Fix Committed |
| Changed in juju-core: | |
| status: | Fix Committed → Fix Released |


Attached is a log of the run.