BlobStorage.list_blobs broken

[bmb]

-req = Request("%s/$s?comp=list" % (self.get_base_url(), container_name)
+req = Request("%s/%s?comp=list" % (self.get_base_url(), container_name)

-containers = dom.findall("Blob")
+containers = dom.findall(".//Blob")

Same XPath changes needed in the list_containers method, I think. Note that findall with unqualified sub-element name may work as is with lxml (I don't know, haven't been able to test), but it doesn't with stdlib xml.etree.ElementTree, which I added like so:
-from lxml import etree
+try:

• from lxml import etree
  +except ImportError:
• from xml.etree import ElementTree as etree

[tishon]

The XPath problem you mention is not really a bug - everywhere in the code XML parsing is done with the lxml library because of its superior performance. But if that dependency is a problem for you, I'll make sure the code works without lxml too.

[bmb]

Hi Tihomir,

Yes, I wasn't sure that this was a bug, per se -- just a possible
issue of ElementTree API compatibility.

My preference is to keep the API non-dependent on any external libs -
I will eventually include cloud storage awareness in our remote code,
which is a frozen stand-alone package so C-coded extensions make extra
work in the freeze process. But where lxml is available it seems to
make sense to use it for the performance advantage. And given the
limited functionality the API needs it seems likely we can keep it
lxml.etree and xml.etree.ElementTree compatible with little effort.

Cheers,
~Blair

Blair Bethwaite
Researcher, Developer, SysAdmin, Nimrod and Grid support specialist
Monash eScience and Grid Engineering Lab (http://www.messagelab.monash.edu.au/)
+61 3-9903-2800

[tishon]

Yup, you're right. Hard external dependencies should be kept to a minimum. I'll be able to work on this early next week.