MachineSuite.TestWatchInterfaces intermittently fails

Bug #1339864 reported by Andrew Wilkins
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
juju-core
Fix Released
Medium
Frank Mueller

Bug Description

The following error sometimes occurs in the state package.

The issue appears to be a race between the watcher loop and the assertion. The watcher will see the two changes (two interfaces disabled) individually, but the assertion will *usually* only see them as a single event because it checks after the watcher has seen both events.

----------------------------------------------------------------------
FAIL: machine_test.go:1750: MachineSuite.TestWatchInterfaces

[LOG] 0:00.003 INFO juju.state opening state, mongo addresses: ["localhost:40029"]; entity %!q(<nil>)
[LOG] 0:00.003 DEBUG juju.state dialing mongo
[LOG] 0:00.005 INFO juju.mongo dialled mongo successfully on address "127.0.0.1:40029"
[LOG] 0:00.005 DEBUG juju.state connection established
[LOG] 0:00.023 INFO juju.state adding state server info to legacy environment
[LOG] 0:00.023 INFO juju.state found existing state servers []
[LOG] 0:00.039 INFO juju.state initializing environment
machine_test.go:1808:
    wc.AssertOneChange()
testing/watcher.go:76:
    c.Fatalf("watcher sent unexpected change: (_, %v)", ok)

Revision history for this message
Frank Mueller (themue) wrote :

<http://juju-ci.vapour.ws:8080/job/github-merge-juju/1/console>

FAIL: machine_test.go:1750: MachineSuite.TestWatchInterfaces
...
machine_test.go:1808:
    wc.AssertOneChange()
testing/watcher.go:76:
    c.Fatalf("watcher sent unexpected change: (_, %v)", ok)
... Error: watcher sent unexpected change: (_, true)

...
FAIL github.com/juju/juju/state 123.265s

This test was added in PR 207.

<https://github.com/juju/juju/pull/207>

<http://juju-ci.vapour.ws:8080/job/github-merge-juju/3/console>

FAIL: server_test.go:96: serverSuite.TestAPIServerCanListenOnBothIPv4AndIPv6
...
server_test.go:104:
    c.Assert(err, gc.IsNil)
... value *net.OpError = &net.OpError{Op:"listen", Net:"tcp",
Addr:(*net.TCPAddr)(0xc2107e9f00),
Err:(*os.SyscallError)(0xc21027f560)} ("listen tcp :54321: bind:
address already in use")

...
FAIL github.com/juju/juju/state/apiserver 30.369s

This test was added in PR 224.

<https://github.com/juju/juju/pull/224>

Changed in juju-core:
assignee: nobody → Frank Mueller (themue)
status: Triaged → In Progress
Revision history for this message
Frank Mueller (themue) wrote :
Revision history for this message
Frank Mueller (themue) wrote :

Second failure of serverSuite.TestAPIServerCanListenOnBothIPv4AndIPv6 now moved to extra issue:

https://bugs.launchpad.net/juju-core/+bug/1340777

Frank Mueller (themue)
Changed in juju-core:
status: In Progress → Fix Committed
Curtis Hovey (sinzui)
Changed in juju-core:
milestone: none → 1.21-alpha1
Curtis Hovey (sinzui)
Changed in juju-core:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.