HostDeserializer doesn't support multiple line xml
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
OpenStack Compute (nova) |
Fix Released
|
Medium
|
Mauro S M Rodrigues |
Bug Description
If you do a request to Hosts extension to put a host into maintenance mode or update its status, as it's in the unit tests, with this string:
("<?xml version='1.0' encoding=
'<updates>
it works fine, but if you change the string to:
"""<?xml version='1.0' encoding='UTF-8'?>
<updates>
</updates>"""
it doesn't work, if you change the test string to this one it also doesn't work. The trace is:
run_tests.sh nova.tests.
=======
ERROR: nova.tests.
-------
_StringException: Traceback (most recent call last):
File "/opt/stack/
result = self.deserializ
File "/opt/stack/
return self.dispatch(
File "/opt/stack/
return action_
File "/opt/stack/
updates[
AttributeError: Text instance has no attribute 'tagName'
I'm about to submit a fix to this where a I change the HostDeserializer to HostUpdateDeser
[API V2 AUDIT] Also I think that put the tags updates doesn't make so much sense since the json version doesn't have it, so I'm also marking this with api v2 audit tag: nova-v2-
Changed in nova: | |
importance: | Undecided → Low |
importance: | Low → Medium |
Changed in nova: | |
milestone: | none → grizzly-2 |
status: | Fix Committed → Fix Released |
Changed in nova: | |
milestone: | grizzly-2 → 2013.1 |
Fix proposed to branch: master /review. openstack. org/16774
Review: https:/