Canonical Juju

Race in juju/controller/destroy and TestDestroyCommandConfirmation

Bug #1579051 reported by Curtis Hovey
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Canonical Juju
Fix Released
Critical
Tim Penhey
Canonical Juju 2.0-beta7

Bug Description

As seen in
    http://reports.vapour.ws/releases/issue/572c8dd4749a5679b8e5b394

Race seen involving
    github.com/juju/juju/cmd/juju/controller/destroy.go
and
     /home/ubuntu/juju-core_2.0-beta7/src/github.com/juju/juju/cmd/testing/testing.go:83 +0x197
  github.com/juju/juju/cmd/juju/controller_test.(*DestroySuite).TestDestroyCommandConfirmation()

This issue was first seen in the lp-1321442-ec2-non-default-vpc-support-take-2 branch

See original description
Curtis Hovey (sinzui)
tags: added: ci race-condition regression
description: updated
Changed in juju-core:
milestone: none → 2.0-beta7
assignee: nobody → Dimiter Naydenov (dimitern)
status: New → Triaged
importance: Undecided → High
Curtis Hovey (sinzui)
tags: added: blocker
Changed in juju-core:
importance: High → Critical
Alexis Bruemmer (alexis-bruemmer)
Changed in juju-core:
status: Triaged → In Progress
Revision history for this message
Dimiter Naydenov (dimitern) wrote :

I cannot reproduce this on lp-1321442-ec2-not-default-vpc-support-take-2-branch with -race.

Dimiter Naydenov (dimitern)
Changed in juju-core:
assignee: Dimiter Naydenov (dimitern) → nobody
status: In Progress → Invalid
Revision history for this message
Curtis Hovey (sinzui) wrote :

As seen in the issue, this race is still happening and it originates from master.

Changed in juju-core:
status: Invalid → Triaged
tags: added: intermittent-failure
Revision history for this message
Dimiter Naydenov (dimitern) wrote :

FWIW I strongly suspect this is related to the recent changes to cmd.Main and testing juju-1 vs juju-2 commands: bug #1579059 and bug #1580184

Reed O'Brien (reedobrien)
Changed in juju-core:
assignee: nobody → Reed O'Brien (reedobrien)
status: Triaged → In Progress
Reed O'Brien (reedobrien)
Changed in juju-core:
assignee: Reed O'Brien (reedobrien) → nobody
Cheryl Jennings (cherylj)
Changed in juju-core:
status: In Progress → Triaged
Reed O'Brien (reedobrien)
Changed in juju-core:
assignee: nobody → Reed O'Brien (reedobrien)
status: Triaged → In Progress
Tim Penhey (thumper)
Changed in juju-core:
assignee: Reed O'Brien (reedobrien) → Tim Penhey (thumper)
Tim Penhey (thumper)
Changed in juju-core:
status: In Progress → Fix Committed
Curtis Hovey (sinzui)
Changed in juju-core:
status: Fix Committed → Fix Released
Canonical Juju QA Bot (juju-qa-bot)
affects: juju-core → juju
Changed in juju:
milestone: 2.0-beta7 → none
milestone: none → 2.0-beta7
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

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