Sandro Bonazzola has posted comments on this change.

Change subject: Update engine's mgmt net to have the correct vlan id
......................................................................


Patch Set 6: Verified-1

Looks like it's failing on SDK.

 2014-09-03 09:39:27 DEBUG otopi.context context._executeMethod:152 method 
exception
 Traceback (most recent call last):
  File "/usr/lib/python2.6/site-packages/otopi/context.py", line 142, in 
_executeMethod
    method['method']()
  File 
"/usr/share/ovirt-hosted-engine-setup/scripts/../plugins/ovirt-hosted-engine-setup/engine/add_host.py",
 line 440, in _closeup
    mgmt_network.update()
  File "/usr/lib/python2.6/site-packages/ovirtsdk/infrastructure/brokers.py", 
line 14403, in update
    body=ParseHelper.toXml(self.superclass),
  File "/usr/lib/python2.6/site-packages/ovirtsdk/utils/parsehelper.py", line 
36, in toXml
    entity.export(output, 0, name_=ParseHelper.getXmlTypeInstance(type_name))
  File "/usr/lib/python2.6/site-packages/ovirtsdk/xml/params.py", line 14331, 
in export
    self.exportChildren(outfile, level + 1, namespace_, name_, 
pretty_print=pretty_print)
  File "/usr/lib/python2.6/site-packages/ovirtsdk/xml/params.py", line 14351, 
in exportChildren
    self.vlan.export(outfile, level, namespace_, name_='vlan', 
pretty_print=pretty_print)
 AttributeError: 'int' object has no attribute 'export'

-- 
To view, visit http://gerrit.ovirt.org/32269
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ica4c5804395d364438f1e142387376cc7adc7172
Gerrit-PatchSet: 6
Gerrit-Project: ovirt-hosted-engine-setup
Gerrit-Branch: master
Gerrit-Owner: Antoni Segura Puimedon <asegu...@redhat.com>
Gerrit-Reviewer: Antoni Segura Puimedon <asegu...@redhat.com>
Gerrit-Reviewer: Juan Hernandez <juan.hernan...@redhat.com>
Gerrit-Reviewer: Lior Vernia <lver...@redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbona...@redhat.com>
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to