Import problems in ec2instance test
Bug #489068 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
The test I added for bug 488695 is breaking: in credentials.py there's an import of boto, which apparently isn't in the launchpad dependencies.
Luckily credentials.py isn't used except in the EC2Instance factory, which the test doesn't use, so I can move the import of EC2Credentials in there to avoid the whole issue.
Related branches
lp:~jtv/launchpad/bug-489068
- Stuart Bishop (community): Approve
- Michael Hudson-Doyle: Pending (code) requested
-
Diff: 32 lines (+9/-4)1 file modifiedlib/devscripts/ec2test/instance.py (+9/-4)
| description: | updated |
Revision history for this message
| Jeroen T. Vermeulen (jtv) 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.
Filed the deeper problem (dependencies not being moved into launchpad from lp-dev-utils) as bug 489073.