euca-run-instances does not correctly validate parameters

Bug #546420 reported by C de-Avillez
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
euca2ools
Fix Released
Undecided
Unassigned
euca2ools (Ubuntu)
Fix Released
Wishlist
Dustin Kirkland 

Bug Description

cempedak:~$ euca-run-instances -k mykey emi-DFE41074 -t c1.medium -n 8--group more8
Traceback (most recent call last):
  File "/usr/bin/euca-run-instances", line 198, in <module>
    main()
  File "/usr/bin/euca-run-instances", line 139, in main
    max_count = int(counts[1])
ValueError: invalid literal for int() with base 10: ''
cempedak:~$

This is a clear user error: '-n 8--group more8'. Code is expecting an integer, and got '8--group'. Nevertheless, we should not fail with an exception.

Probably WishList, at most Low.

affects: eucalyptus (Ubuntu) → euca2ools (Ubuntu)
Changed in euca2ools (Ubuntu):
status: New → Confirmed
importance: Undecided → Wishlist
Scott Moser (smoser)
Changed in euca2ools (Ubuntu):
assignee: nobody → Dustin Kirkland (kirkland)
status: Confirmed → In Progress
affects: eucalyptus → euca2ools
Revision history for this message
Launchpad Janitor (janitor) wrote :

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

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

  [ Scott Moser ]
  * euca-run-instances:
    - --keypair => --key in usage and man page, LP: #530816
    - print error rather than trace on invalid instance-count, LP: #546420
    - euca-describe-instances: output "running", not "running ", LP: #531453
  * euca-revoke: only show usage once with --help
  * euca-download-bundle: fix usage, LP: #546567
  * euca-bundle-image:
    - fix failure on --block-device-mapping, LP: #546548
    - print usage on --help if userid has '-', LP: #546526
    - fix failure when image name string is in dest string, LP: #522060
  * euca-describe-image-attribute: fix --kernel or --ramdisk when
    image did not have emi or eri, LP: #546551

  [ Dustin Kirkland ]
  * Cherry-pick up to upstream r266, fixes:
    - LP: #536876 - document euca-describe-availability-zones verbose
    - LP: #526591 - enhance parsing of config file
    - LP: #531076 - fix euca-describe-images against specific image
 -- Dustin Kirkland <email address hidden> Thu, 25 Mar 2010 15:36:43 -0700

Changed in euca2ools (Ubuntu):
status: In Progress → Fix Released
Revision history for this message
Scott Moser (smoser) wrote :
Changed in euca2ools:
status: New → Fix Committed
Changed in euca2ools:
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.