tests/compute/base.py missing import for exceptions

Bug #1049348 reported by Wayne Vestal Weeks
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
tempest
Fix Released
High
Wayne Vestal Weeks

Bug Description

 The class function _get_isolated_creds in BaseComputeTest catches exceptions.Duplicate; however, exceptions is not imported in the module.

Revision history for this message
Jay Pipes (jaypipes) wrote :

Ran into this today while debugging a different bug.

Changed in tempest:
status: New → Confirmed
assignee: nobody → Jay Pipes (jaypipes)
importance: Undecided → High
Revision history for this message
Wayne Vestal Weeks (wayneweeks) wrote :

I'd fix it, but I'm waiting on my membership to be approved. I figured this would be a good one to get familiar with the work flow.

Revision history for this message
David Kranz (david-kranz) wrote :

Wayne, I didn't see your membership request which would normally have been approved immediately. Which group did you apply to?

Revision history for this message
Wayne Vestal Weeks (wayneweeks) wrote :

I applied to openstack-cla. Should i have applied to a different group?

---

Same module is missing a reference to nose as well.

---

Revision history for this message
David Kranz (david-kranz) wrote :

I think you also have to join the group at https://launchpad.net/tempest. Hopefully some one else can add more information if that is not enough.

Revision history for this message
Wayne Vestal Weeks (wayneweeks) wrote : Re: [Bug 1049348] Re: tests/compute/base.py missing import for exceptions

Ahh, ok thanks. I had asked some of the openstack developers in our office,
but I see they told me wrong. Thanks, ill do that first thing when I get in
the office.
On Sep 11, 2012 8:55 PM, "David Kranz" <email address hidden> wrote:
>
> I think you also have to join the group at
> https://launchpad.net/tempest. Hopefully some one else can add more
> information if that is not enough.
>
> --
> You received this bug notification because you are subscribed to the bug
> report.
> https://bugs.launchpad.net/bugs/1049348
>
> Title:
> tests/compute/base.py missing import for exceptions
>
> Status in Tempest:
> Confirmed
>
> Bug description:
> The class function _get_isolated_creds in BaseComputeTest catches
> exceptions.Duplicate; however, exceptions is not imported in the
> module.
>
> To manage notifications about this bug go to:
> https://bugs.launchpad.net/tempest/+bug/1049348/+subscriptions

Changed in tempest:
assignee: Jay Pipes (jaypipes) → Wayne Weeks (wayneweeks)
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to tempest (master)

Fix proposed to branch: master
Review: https://review.openstack.org/12904

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to tempest (master)

Reviewed: https://review.openstack.org/12904
Committed: http://github.com/openstack/tempest/commit/383c71d0b9a6d8169b71185d91e198c2a1ff5a4a
Submitter: Jenkins
Branch: master

commit 383c71d0b9a6d8169b71185d91e198c2a1ff5a4a
Author: Wayne Vestal Weeks <email address hidden>
Date: Wed Sep 12 16:21:17 2012 -0400

    Added missing imports in tempest.tests.compute.base.

    bug 1049348

    Change-Id: I13dad5580558eb400b89ef40d1e7cf8f55cee9e7

Changed in tempest:
status: In Progress → Fix Committed
Changed in tempest:
status: Fix Committed → Fix Released
Sean Dague (sdague)
Changed in tempest:
milestone: none → havana-3
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.