Il giorno mar 6 nov 2018, 14:56 Ondra Machacek <[email protected]> ha scritto:
> You are using Python SDK from master, but 4.2 engine, there are > currently changes in master Python SDK, which are not understand by 4.2 > engine. I would suggest you to use both same versions. > we shouldn't break api in sdk4. isn't there any way to preserve <4.3 compatibility in sdk4-4.3? > Also I would suggest you to send just needed parameters for specific > actions. For example, when using preview_snapshot action you should send > Snaphost object with just ID, not full object. It's not really needed > and you wouldn't be hitting this issue. > > On 11/6/18 1:25 PM, Eitan Raviv wrote: > > Hi, > > > > We have a failure on OST/network-suite-master for several days now. > > My suspicions of the cause are detailed below. > > Can you have a look? > > > > Thanks > > > > ---------- Forwarded message --------- > > From: Eitan Raviv <[email protected]> > > Date: Mon, Nov 5, 2018 at 8:39 AM > > Subject: Re: [oVirt Jenkins] ovirt-system-tests_network-suite-4.2 - > Build # > > 719 - Still Failing! > > To: Ori Liel <[email protected]> > > > > > > Hi, > > Can you have a look? > > Thanks > > > > On Thu, Nov 1, 2018 at 10:08 AM Eitan Raviv <[email protected]> wrote: > > > >> Hi Ondra, > >> > >> We have above failure on OST network-suite-4.2 [1] due to: > >> Fault reason is "Request syntactically incorrect.". Fault detail is "For > >> correct usage, see: > >> > https://192.168.201.4/ovirt-engine/apidoc#services/vm/methods/preview_snapshot > >> " > >> > >> and I see in engine.log [2]: > >> > >> javax.xml.bind.UnmarshalException: unexpected element (uri:"", > local:"disks") > >> > >> at the same time I see in model_4.2 [3] that a change was done to add > the disks, but if i am not mistaken the ovirt-engine-sdk pom.xml [4] was > not bumped to reflect this change. > >> > >> Can you please check? > >> > >> Thanks > >> > >> [1] > >> > https://jenkins.ovirt.org/job/ovirt-system-tests_network-suite-4.2/719/artifact/exported-artifacts/pytest.junit.xml > >> [2] > >> > https://jenkins.ovirt.org/job/ovirt-system-tests_network-suite-4.2/717/artifact/exported-artifacts/tests.test_mac_pools/lago-network-suite-4-2-engine/_var_log/ovirt-engine/engine.log > >> [3] > >> > https://gerrit.ovirt.org/#/q/project:ovirt-engine-api-model+branch:model_4.2 > >> [4] > https://gerrit.ovirt.org/#/q/project:ovirt-engine-sdk+branch:sdk_4.2 > >> > >> > >> > >> ---------- Forwarded message --------- > >> From: Edward Haas <[email protected]> > >> Date: Thu, Nov 1, 2018 at 8:35 AM > >> Subject: Fwd: [oVirt Jenkins] ovirt-system-tests_network-suite-4.2 - > Build > >> # 719 - Still Failing! > >> To: <[email protected]> > >> > >> > >> FYI > >> > >> ---------- Forwarded message --------- > >> From: <[email protected]> > >> Date: Thu, Nov 1, 2018 at 6:02 AM > >> Subject: [oVirt Jenkins] ovirt-system-tests_network-suite-4.2 - Build # > >> 719 - Still Failing! > >> To: <[email protected]>, <[email protected]>, <[email protected]> > >> > >> > >> Project: > >> http://jenkins.ovirt.org/job/ovirt-system-tests_network-suite-4.2/ > >> Build: > >> http://jenkins.ovirt.org/job/ovirt-system-tests_network-suite-4.2/719/ > >> Build Number: 719 > >> Build Status: Still Failing > >> Triggered By: Started by timer > >> > >> ------------------------------------- > >> Changes Since Last Success: > >> ------------------------------------- > >> Changes for Build #717 > >> [Marcin Sobczyk] Change logging level for 'schema.inconsistency' > >> > >> > >> Changes for Build #718 > >> [Ales Musil] master: Update master suite compatibility version > >> > >> > >> Changes for Build #719 > >> [Ales Musil] master: Update master suite compatibility version > >> > >> > >> > >> > >> ----------------- > >> Failed Tests: > >> ----------------- > >> 1 tests failed. > >> FAILED: > >> > network-suite-4.2.tests.test_mac_pools.test_undo_preview_snapshot_when_mac_used_reassigns_a_new_mac > >> > >> Error Message: > >> Error: Fault reason is "Request syntactically incorrect.". Fault detail > is > >> "For correct usage, see: > >> > https://192.168.201.4/ovirt-engine/apidoc#services/vm/methods/preview_snapshot > ". > >> HTTP response code is 400. > >> > >> Stack Trace: > >> system = <lib.system.SDKSystemRoot object at 0x7fc7f91f48d0> > >> default_cluster = <lib.clusterlib.Cluster object at 0x7fc7f91f4bd0> > >> ovirtmgmt_vnic_profile = <lib.netlib.VnicProfile object at > 0x7fc7f81b3710> > >> cirros_template = 'Cirros_0_4_0' > >> > >> def test_undo_preview_snapshot_when_mac_used_reassigns_a_new_mac( > >> system, default_cluster, ovirtmgmt_vnic_profile, > >> cirros_template): > >> with virtlib.vm_pool(system, size=2) as (vm_0, vm_1): > >> vm_0.create(vm_name=VM0, > >> cluster=default_cluster, > >> template=cirros_template) > >> vm_0.wait_for_down_status() > >> > >> vm_0.run() > >> vm_0.wait_for_up_status() > >> > >> nicless_snapshot = _create_snapshot(vm_0) > >> > >> vm_0.create_vnic(NIC_NAME_1, ovirtmgmt_vnic_profile, > >> MAC_ADDR_1) > >> vm_0.stop() > >> vm_0.wait_for_down_status() > >> > >>> nicless_snapshot.preview() > >> > >> network-suite-4.2/tests/test_mac_pools.py:124: > >> _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ > _ > >> _ _ _ > >> network-suite-4.2/lib/virtlib.py:243: in preview > >> snapshot=self.get_sdk_type() > >> /usr/lib64/python2.7/site-packages/ovirtsdk4/services.py:31091: in > >> preview_snapshot > >> return self._internal_action(action, 'previewsnapshot', None, > headers, > >> query, wait) > >> /usr/lib64/python2.7/site-packages/ovirtsdk4/service.py:299: in > >> _internal_action > >> return future.wait() if wait else future > >> /usr/lib64/python2.7/site-packages/ovirtsdk4/service.py:55: in wait > >> return self._code(response) > >> /usr/lib64/python2.7/site-packages/ovirtsdk4/service.py:296: in callback > >> self._check_fault(response) > >> /usr/lib64/python2.7/site-packages/ovirtsdk4/service.py:132: in > >> _check_fault > >> self._raise_error(response, body) > >> _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ > _ > >> _ _ _ > >> > >> response = <ovirtsdk4.http.Response object at 0x7fc7f81b3350> > >> detail = <ovirtsdk4.types.Fault object at 0x7fc7f81b39d0> > >> > >> @staticmethod > >> def _raise_error(response, detail=None): > >> """ > >> Creates and raises an error containing the details of the > >> given HTTP > >> response and fault. > >> > >> This method is intended for internal use by other > components > >> of the > >> SDK. Refrain from using it directly, as backwards > >> compatibility isn't > >> guaranteed. > >> """ > >> fault = detail if isinstance(detail, types.Fault) else None > >> > >> msg = '' > >> if fault: > >> if fault.reason: > >> if msg: > >> msg += ' ' > >> msg = msg + 'Fault reason is "%s".' % fault.reason > >> if fault.detail: > >> if msg: > >> msg += ' ' > >> msg = msg + 'Fault detail is "%s".' % fault.detail > >> if response: > >> if response.code: > >> if msg: > >> msg += ' ' > >> msg = msg + 'HTTP response code is %s.' % response.code > >> if response.message: > >> if msg: > >> msg += ' ' > >> msg = msg + 'HTTP response message is "%s".' % > >> response.message > >> > >> if isinstance(detail, six.string_types): > >> if msg: > >> msg += ' ' > >> msg = msg + detail + '.' > >> > >> class_ = Error > >> if response is not None: > >> if response.code in [401, 403]: > >> class_ = AuthError > >> elif response.code == 404: > >> class_ = NotFoundError > >> > >> error = class_(msg) > >> error.code = response.code if response else None > >> error.fault = fault > >>> raise error > >> E Error: Fault reason is "Request syntactically incorrect.". Fault > >> detail is "For correct usage, see: > >> > https://192.168.201.4/ovirt-engine/apidoc#services/vm/methods/preview_snapshot > ". > >> HTTP response code is 400. > >> > >> /usr/lib64/python2.7/site-packages/ovirtsdk4/service.py:118: Error > >> > > > _______________________________________________ > Infra mailing list -- [email protected] > To unsubscribe send an email to [email protected] > Privacy Statement: https://www.ovirt.org/site/privacy-policy/ > oVirt Code of Conduct: > https://www.ovirt.org/community/about/community-guidelines/ > List Archives: > https://lists.ovirt.org/archives/list/[email protected]/message/T5FWQGOS7TEJUTSVTUP2BD3ZXLUPRCWF/ >
_______________________________________________ Infra mailing list -- [email protected] To unsubscribe send an email to [email protected] Privacy Statement: https://www.ovirt.org/site/privacy-policy/ oVirt Code of Conduct: https://www.ovirt.org/community/about/community-guidelines/ List Archives: https://lists.ovirt.org/archives/list/[email protected]/message/5VGUA7EL7IFSVCVL2WVU6NABDS4LFKIV/
