Headless ec2 image always shuts down
Bug #488695 reported by
Jeroen T. Vermeulen
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Launchpad itself |
Fix Released
|
Critical
|
Jeroen T. Vermeulen | ||
Bug Description
Whenever we try to "ec2 land" or "ec2 test" headlessly, the image sets up and then goes "instance shutting-down" instead of running the tests.
mwhudson reports:
I changed how shutting down works in
http://
but the code looks correct to me :/ I guess I'll --postmortem something
today and see what the value of 'really_shutdown' is...
As far as I can tell, really_shutdown isn't set because the "return func(...)" inside the "try" block skips the "else" block attached to the "try."
Related branches
lp://qastaging/~jtv/launchpad/bug-488695
- Michael Hudson-Doyle: Approve
- Graham Binns: Pending (code) requested
-
Diff: 233 lines (+195/-2)2 files modifiedlib/devscripts/ec2test/instance.py (+10/-2)
lib/devscripts/ec2test/tests/test_ec2instance.py (+185/-0)
Revision history for this message
| Michael Hudson-Doyle (mwhudson) wrote | #1 |
Revision history for this message
| Diogo Matsubara (matsubara) wrote Bug fixed by a commit | #2 |
| Changed in launchpad-foundations: | |
| status: | In Progress → Fix Committed |
Revision history for this message
| Ursula Junque (ursinha) wrote A commit mentioned this bug | #3 |
| tags: | added: qa-needstesting |
| Changed in launchpad-foundations: | |
| status: | Fix Committed → Fix Released |
| tags: |
added: qa-ok removed: qa-needstesting |
To post a comment you must log in.
I landed a fix, but jtv's is better, so he's going to land it over mine.