launchpad.net/juju-core/juju tests sometimes fail mgo tip

Bug #1252859 reported by Michael Hudson-Doyle
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
juju-core
Fix Released
High
Unassigned

Bug Description

I've seen the

Waiting for sockets to die: 0 in use, 1 alive
/home/mwhudson/goplay/src/launchpad.net/juju-core/testing/mgo.go:333:
    c.Fatal("Test left sockets in a dirty state")

thing in other failures.

Revision history for this message
Michael Hudson-Doyle (mwhudson) wrote :
Revision history for this message
Andrew Wilkins (axwalk) wrote :

This is not particular to gccgo - there's a race condition somewhere, causing this even with gc.

Revision history for this message
John A Meinel (jameinel) wrote :

Last time we ran into a failure like this there was a race in the mgo driver.
I also seem to remember that we had to use a fixed version of mgo (241?) because of other incompatibilities. I wonder if we got a newer version and it brought in another race.

Changed in juju-core:
importance: Undecided → High
status: New → Triaged
tags: removed: gccgo
Revision history for this message
John A Meinel (jameinel) wrote :

As noted, this isn't gccgo specific.

Revision history for this message
Michael Hudson-Doyle (mwhudson) wrote : Re: [Bug 1252859] Re: launchpad.net/juju-core/juju tests fail with gccgo

Ah well I just ran "go get juju-core/..." so I probably did get mgo tip?
And in fact you need a very recent fix for it to work at all with gccgo.
On 21 Nov 2013 01:50, "John A Meinel" <email address hidden> wrote:

> As noted, this isn't gccgo specific.
>
> --
> You received this bug notification because you are subscribed to the bug
> report.
> https://bugs.launchpad.net/bugs/1252859
>
> Title:
> launchpad.net/juju-core/juju tests fail with gccgo
>
> To manage notifications about this bug go to:
> https://bugs.launchpad.net/juju-core/+bug/1252859/+subscriptions
>

Revision history for this message
Dave Cheney (dave-cheney) wrote :

oop, I think you are right, somewhere along the way we did bump the
version of the mongo driver.

On Thu, Nov 21, 2013 at 6:32 AM, Michael Hudson-Doyle
<email address hidden> wrote:
> Ah well I just ran "go get juju-core/..." so I probably did get mgo tip?
> And in fact you need a very recent fix for it to work at all with gccgo.
> On 21 Nov 2013 01:50, "John A Meinel" <email address hidden> wrote:
>
>> As noted, this isn't gccgo specific.
>>
>> --
>> You received this bug notification because you are subscribed to the bug
>> report.
>> https://bugs.launchpad.net/bugs/1252859
>>
>> Title:
>> launchpad.net/juju-core/juju tests fail with gccgo
>>
>> To manage notifications about this bug go to:
>> https://bugs.launchpad.net/juju-core/+bug/1252859/+subscriptions
>>
>
> --
> You received this bug notification because you are subscribed to juju-
> core.
> Matching subscriptions: MOAR JUJU SPAM!
> https://bugs.launchpad.net/bugs/1252859
>
> Title:
> launchpad.net/juju-core/juju tests fail with gccgo
>
> Status in juju-core:
> Triaged
>
> Bug description:
> I've seen the
>
> Waiting for sockets to die: 0 in use, 1 alive
> /home/mwhudson/goplay/src/launchpad.net/juju-core/testing/mgo.go:333:
> c.Fatal("Test left sockets in a dirty state")
>
> thing in other failures.
>
> To manage notifications about this bug go to:
> https://bugs.launchpad.net/juju-core/+bug/1252859/+subscriptions

Revision history for this message
Michael Hudson-Doyle (mwhudson) wrote : Re: launchpad.net/juju-core/juju tests fail with gccgo

And this week, it seems, the tests pass with gccgo. So yeah, this isn't a real gccgo failure...

summary: - launchpad.net/juju-core/juju tests fail with gccgo
+ launchpad.net/juju-core/juju tests fail mgo tip
summary: - launchpad.net/juju-core/juju tests fail mgo tip
+ launchpad.net/juju-core/juju tests sometimes fail mgo tip
Curtis Hovey (sinzui)
tags: added: amd64 gccgo
Revision history for this message
Ian Booth (wallyworld) wrote :

We bunch of mgo related test fixes have recently landed in trunk. Closing this, but re-open if issue still occurs.

Changed in juju-core:
status: Triaged → Fix Committed
Curtis Hovey (sinzui)
Changed in juju-core:
milestone: none → 1.21-alpha2
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.

Other bug subscribers

Remote bug watches

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