[ovirt-devel] snapshot merge failing on Experimental

[Evgheni Dereveanchin]

Thanks for the input!

These were just suggestions as from first glance 
it looks like some VDSM change causes issues
on Engine when parsing the JSON. It may well be 
something else though and engine-related instead.

Regards, 
Evgheni Dereveanchin 

----- Original Message -----
From: "Ala Hino" <ahino at redhat.com>
To: "Evgheni Dereveanchin" <ederevea at redhat.com>
Cc: "devel" <devel at ovirt.org>, "Nir Soffer" <nsoffer at redhat.com>
Sent: Thursday, 24 November, 2016 10:37:52 AM
Subject: Re: snapshot merge failing on Experimental

The failures are not related to the mentioned patches. The patches
introduce APIs that are not used it.
I see following error following exception causing the failure:

Caused by: java.lang.NullPointerException
	at org.ovirt.engine.core.common.businessentities.storage.DiskImageBase.isQcowFormat(DiskImageBase.java:90)
	at sun.reflect.GeneratedMethodAccessor342.invoke(Unknown Source)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:498)
	at org.codehaus.jackson.map.ser.BeanPropertyWriter.get(BeanPropertyWriter.java:483)
	at org.codehaus.jackson.map.ser.BeanPropertyWriter.serializeAsField(BeanPropertyWriter.java:418)
	at org.codehaus.jackson.map.ser.std.BeanSerializerBase.serializeFields(BeanSerializerBase.java:150)


On Thu, Nov 24, 2016 at 11:32 AM, Evgheni Dereveanchin <ederevea at redhat.com>
wrote:

> Hi everyone,
>
> CI is failing on snapshot merge tests:
> http://jenkins.ovirt.org/job/test-repo_ovirt_experimental_
> master/3618/testReport/junit/(root)/004_basic_sanity/snapshot_merge/
>
> Not sure what is causing this but I see several patches
> that may be related that were recently merged:
> https://gerrit.ovirt.org/#/c/67064
> https://gerrit.ovirt.org/#/c/67184
>
> Could someone please check why it's happening as we
> need to fix/revert the problematic patch to make tests
> work again.
>
> Regards,
> Evgheni Dereveanchin
>