euca-describe-volumes fails in lucid

[Scott Moser]

Binary package hint: euca2ools

euca-describe-volumes fails in lucid right now, this is due to python-boto being at version 1.9 rather than 1.8.

[Robert Collins (lifeless)]

hasattr can swallow exceptions. I suggest

=== modified file 'bin/euca-describe-volumes'
--- bin/euca-describe-volumes 2009-10-12 09:11:57 +0000
+++ bin/euca-describe-volumes 2010-02-06 00:15:08 +0000
@@ -73,8 +73,11 @@
      volume_string += '\t%s' % (volume.snapshot_id)
  else:
      volume_string += '\t'
- if volume.availabilityZone:
- volume_string += '\t%s' % volume.availabilityZone
+
+ az = getattr(volume, 'availabilityZone', object)
+ if az is object: az=volume.zone
+ if az: volume_string += '\t%s' % az
+
  volume_string += '\t%s\t%s' % (volume.status, volume.create_time)
  print 'VOLUME\t%s' % (volume_string)
  if volume.status == 'in-use':

[Dustin Kirkland  (kirkland)]

Neil-

This bug is high priority, and targeted at Alpha3.

Any chance you could take a look at the suggested patch(es) and perhaps mangle one upstream?

[Neil Soman (neilsoman)]

Thanks for the patch. The main consideration was whether this breaks upstream compatibility with boto-1.8d, which is the version the upstream will stick with for euca2ools-1.2 and it does not seem to. I have applied it.

thanks
neil

------------------------------------------------------------
revno: 248
committer: Neil
branch nick: euca2ools-1.0
timestamp: Thu 2010-02-11 13:45:50 -0800
message:
  fixes LP: #517871
------------------------------------------------------------

[Launchpad Janitor (janitor)]

This bug was fixed in the package euca2ools - 1.2~bzr248-0ubuntu1

---------------
euca2ools (1.2~bzr248-0ubuntu1) lucid; urgency=low

  * Merge from upstream bzr revision; should fix:
    - LP: #517871 - fix euca-describe-volumes
 -- Dustin Kirkland <email address hidden> Thu, 11 Feb 2010 16:17:10 -0600

[Scott Moser (smoser)]

Is this at all valid against Eucalyptus ? Or is it just euca2ools and we should close the Eucalyptus track.

[Thierry Carrez (ttx)]

@Scott: "Eucalyptus" is eucalyptus upstream, not the eucalyptus package. At this point, it also contains euca2ools upstream, so I think it's valid to keep it open ?

[Scott Moser (smoser)]

@Thierry,
 You're right. I forgot that. I think its valid to keep it open, but my guess is its fixed-released in current released version of euca2ools, but leave that up to Eucalyptus people.