Eucalyptus

euca2ools.Euca2ool.make_connection should not call sys.exit()

Bug #522398 reported by mjpan
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Eucalyptus
Fix Released
Low
Neil Soman
euca2ools (Ubuntu)
Fix Released
Low
Dustin Kirkland 

Bug Description

in the function, it checks for specific system environment variables. If they are not found, the function calls sys.exit. It should instead thrown an exception, so that the caller can handle the exception.

Neil Soman (neilsoman)
Changed in eucalyptus:
assignee: nobody → Neil Soman (neilsoman)
Revision history for this message
Neil Soman (neilsoman) wrote :

------------------------------------------------------------
revno: 250
committer: Neil
branch nick: euca2ools-1.0
timestamp: Tue 2010-02-16 11:56:56 -0800
message:
  fixes LP: #522398
------------------------------------------------------------

Changed in eucalyptus:
status: New → Fix Committed
importance: Undecided → Low
Dustin Kirkland  (kirkland)
Changed in euca2ools (Ubuntu):
status: New → In Progress
importance: Undecided → Low
assignee: nobody → Dustin Kirkland (kirkland)
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package euca2ools - 1.2-0ubuntu5

---------------
euca2ools (1.2-0ubuntu5) lucid; urgency=low

  * Cherry pick fixes from upstream, up to bzr r265, fixes:
    - LP: #522398 - throw exceptions instead of sys.exit(1)
    - LP: #522396 - fix version string
    - LP: #523332 - check rsync return code
    - LP: #516738 - upstream fix for image store
    - LP: #525137 - don't sys.exit(1)
    - LP: #526697 - merge upstream fix for this one
    - LP: #531076 - fix euca-describe-instances against one instance
    - LP: #526591 - fix config file parsing
 -- Dustin Kirkland <email address hidden> Fri, 05 Mar 2010 16:59:25 -0600

Changed in euca2ools (Ubuntu):
status: In Progress → Fix Released
graziano obertelli (graziano.obertelli)
Changed in eucalyptus:
status: Fix Committed → Fix Released
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.