On Thu, Jul 10, 2014 at 8:19 PM, James Solomon <[email protected]> wrote:
> Hi all, > > I've recently added the Actions bit mentioned by Andrew to FilterSuite. > I'll defintely have a look over that. We're chatting about why that might > be happening. Have you seen this more than once? > Thanks. I have seen it a bunch of times, but today was the first time I noticed it. I think I pulled master earlier today, but I don't recall when I had done so before. > Bodie > > > >> From: Andrew Wilkins <[email protected]> >> Date: Thu, Jul 10, 2014 at 3:19 PM >> Subject: Re: New flakey juju tests >> To: Martin Packman <[email protected]> >> Cc: "[email protected]" <[email protected]> >> >> >> >> Here's another, that I've started seeing today: >> >> FAIL: filter_test.go:508: FilterSuite.TestActionEvents >> ... >> filter_test.go:531: >> assertChange(testIds) >> filter_test.go:490: >> c.Assert(event.ActionId, gc.Equals, id) >> ... obtained string = "wordpress/0_a_5" >> ... expected string = "wordpress/0_a_1" >> >> >> On Wed, Jul 9, 2014 at 5:26 PM, Martin Packman < >> [email protected]> wrote: >> >>> We've done a lot of work recently improving the reliability of our >>> test suite. Unfortunately, feature work has been introducing new tests >>> that are intermittently failing. >>> >>> <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> >>> >>> >>> Can we have another look at these tests and fix them up to be properly >>> robust? I don't want to back out changes that have been in trunk for a >>> while, but we can't leave unreliable tests on trunk. Thanks, >>> >>> Martin >>> >>> -- >>> Juju-dev mailing list >>> [email protected] >>> Modify settings or unsubscribe at: >>> https://lists.ubuntu.com/mailman/listinfo/juju-dev >>> >> >> >> -- >> Juju-dev mailing list >> [email protected] >> Modify settings or unsubscribe at: >> https://lists.ubuntu.com/mailman/listinfo/juju-dev >> >> >> >
-- Juju-dev mailing list [email protected] Modify settings or unsubscribe at: https://lists.ubuntu.com/mailman/listinfo/juju-dev
