TestAgentConnectionDelaysShutdownWithPing fails

Bug #1455627 reported by Aaron Bentley
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Canonical Juju
Fix Released
High
Tim Penhey
juju-core
Won't Fix
Undecided
Unassigned
1.25
Won't Fix
Undecided
Unassigned
Revision history for this message
Curtis Hovey (sinzui) wrote :

This bug is the most common reason unit tests fail in lxc. We want to run in lxc because it is faster to bring up and parallelise.

Changed in juju-core:
status: New → Triaged
tags: added: lxc
Changed in juju-core:
importance: Undecided → High
milestone: none → 1.25.0
tags: added: unit-tests
Curtis Hovey (sinzui)
no longer affects: juju-core/1.24
Revision history for this message
Curtis Hovey (sinzui) wrote :

We can see this issue affects the large windowss machine as well as lxc
   http://reports.vapour.ws/releases/issue/55564982749a5650f87680a9

tags: added: windows
Curtis Hovey (sinzui)
Changed in juju-core:
milestone: 1.25.0 → none
importance: High → Medium
Revision history for this message
Alexis Bruemmer (alexis-bruemmer) wrote :

Moving this to high and targeting it to 1.25 given the impact on test efforts

Changed in juju-core:
importance: Medium → High
milestone: none → 1.25.0
Revision history for this message
Dimiter Naydenov (dimitern) wrote :

I hope this http://reviews.vapour.ws/r/2454/ might fix it (in net-cli, soon to be merged in master).

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

My "fix" didn't work, but gave me enough information to propose a better one: https://github.com/juju/juju/pull/3094 which should make is pass more reliably as it should be much less sensitive to machine load / power.

Changed in juju-core:
status: In Progress → Fix Committed
Revision history for this message
Aaron Bentley (abentley) wrote :

I'm sorry to report that we are still seeing this in master, even with the second fix landed:
http://reports.vapour.ws/releases/3006/job/run-unit-tests-win2012-amd64/attempt/1126
(3006 was a test of bdc631, which has 3be8e2f as an ancestor)

Changed in juju-core:
status: Fix Committed → Triaged
Curtis Hovey (sinzui)
Changed in juju-core:
milestone: 1.25-alpha1 → 1.25-beta1
Revision history for this message
Dimiter Naydenov (dimitern) wrote :

Well, I've discovered there's a race condition somewhere around suite initialization, which is causing the test to fail intermittently, and not due to heavy machine load (i.e. my fix for calculating the timeout according to time it takes on the actual machine running it *was* correct, but it uncovered the race).

Changed in juju-core:
milestone: 1.25-beta1 → 1.25-beta2
Changed in juju-core:
milestone: 1.25-beta2 → 1.25.1
Changed in juju-core:
milestone: 1.25.1 → 1.25.2
Changed in juju-core:
milestone: 1.25.2 → 1.25.3
Changed in juju-core:
milestone: 1.25.3 → 1.25.4
Changed in juju-core:
milestone: 1.25.4 → 1.25.5
Changed in juju-core:
milestone: 1.25.5 → 1.25.6
Curtis Hovey (sinzui)
Changed in juju-core:
status: Triaged → Fix Released
Revision history for this message
Cheryl Jennings (cherylj) wrote :
Changed in juju-core:
status: Fix Released → Triaged
Revision history for this message
Cheryl Jennings (cherylj) wrote :

Here's a link to the issue to see all the cases where this is being hit:

http://reports.vapour.ws/releases/issue/55564982749a5650f87680a9

Curtis Hovey (sinzui)
Changed in juju-core:
milestone: 1.25.6 → 1.25.7
Revision history for this message
Nicholas Skaggs (nskaggs) wrote :

Just a note that this also affects juju-2.0.

Changed in juju-core:
milestone: 1.25.7 → 2.0-beta13
Curtis Hovey (sinzui)
description: updated
Curtis Hovey (sinzui)
Changed in juju-core:
milestone: 2.0-beta13 → 2.0-beta14
Curtis Hovey (sinzui)
Changed in juju-core:
milestone: 2.0-beta14 → 2.0-beta15
Changed in juju-core:
milestone: 2.0-beta15 → 2.0-beta16
affects: juju-core → juju
Changed in juju:
milestone: 2.0-beta16 → none
milestone: none → 2.0-beta16
Changed in juju-core:
status: New → Won't Fix
Curtis Hovey (sinzui)
Changed in juju:
milestone: 2.0-beta16 → 2.0-beta17
Curtis Hovey (sinzui)
Changed in juju:
milestone: 2.0-beta17 → 2.0-beta18
Changed in juju:
assignee: Dimiter Naydenov (dimitern) → Tim Penhey (thumper)
status: Triaged → In Progress
Curtis Hovey (sinzui)
Changed in juju:
milestone: 2.0-beta18 → 2.0-beta19
Changed in juju:
milestone: 2.0-beta19 → 2.0-rc1
Tim Penhey (thumper)
Changed in juju:
status: In Progress → Fix Committed
Curtis Hovey (sinzui)
Changed in juju:
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.