snapshot merge failing on Experimental

Hi everyone, CI is failing on snapshot merge tests: http://jenkins.ovirt.org/job/test-repo_ovirt_experimental_master/3618/testRe... 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

On 24 Nov 2016, at 10:32, Evgheni Dereveanchin <ederevea@redhat.com> wrote:
Hi everyone,
CI is failing on snapshot merge tests: http://jenkins.ovirt.org/job/test-repo_ovirt_experimental_master/3618/testRe...
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.
I have to say I’m completely lost in what is being run, why, and with what data for both patches in gerrit all check-patch/check-merged are green and working…
Regards, Evgheni Dereveanchin _______________________________________________ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel

--Apple-Mail=_2ACE1794-FA87-4DEA-B70E-C7B15EA833BE Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset=utf-8 And to me this looks actually like the rest api returned a 500 Status = not VDSM=20
On Nov 24, 2016, at 10:37 AM, Michal Skrivanek = <michal.skrivanek@redhat.com> wrote: =20
=20 On 24 Nov 2016, at 10:32, Evgheni Dereveanchin <ederevea@redhat.com> = wrote: =20 Hi everyone, =20 CI is failing on snapshot merge tests: = http://jenkins.ovirt.org/job/test-repo_ovirt_experimental_master/3618/test= Report/junit/(root)/004_basic_sanity/snapshot_merge/ =20 Not sure what is causing this but I see several patches=20 that may be related that were recently merged: https://gerrit.ovirt.org/#/c/67064 https://gerrit.ovirt.org/#/c/67184 =20 Could someone please check why it's happening as we need to fix/revert the problematic patch to make tests work again. =20 I have to say I=E2=80=99m completely lost in what is being run, why, = and with what data for both patches in gerrit all check-patch/check-merged are green and = working=E2=80=A6 =20 =20 Regards,=20 Evgheni Dereveanchin=20 _______________________________________________ Devel mailing list Devel@ovirt.org <mailto:Devel@ovirt.org> http://lists.ovirt.org/mailman/listinfo/devel = <http://lists.ovirt.org/mailman/listinfo/devel> =20 =20 =20
Devel mailing list Devel@ovirt.org <mailto:Devel@ovirt.org> http://lists.ovirt.org/mailman/listinfo/devel = <http://lists.ovirt.org/mailman/listinfo/devel>
--Apple-Mail=_2ACE1794-FA87-4DEA-B70E-C7B15EA833BE Content-Transfer-Encoding: quoted-printable Content-Type: text/html; charset=utf-8 <html><head><meta http-equiv=3D"Content-Type" content=3D"text/html = charset=3Dutf-8"></head><body style=3D"word-wrap: break-word; = -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" = class=3D""><div class=3D""><br class=3D""></div><div class=3D"">And to = me this looks actually like the rest api returned a 500 Status not = VDSM </div><br class=3D""><div><blockquote type=3D"cite" = class=3D""><div class=3D"">On Nov 24, 2016, at 10:37 AM, Michal = Skrivanek <<a href=3D"mailto:michal.skrivanek@redhat.com" = class=3D"">michal.skrivanek@redhat.com</a>> wrote:</div><br = class=3D"Apple-interchange-newline"><div class=3D""><blockquote = type=3D"cite" style=3D"font-family: Helvetica; font-size: 12px; = font-style: normal; font-variant-caps: normal; font-weight: normal; = letter-spacing: normal; orphans: auto; text-align: start; text-indent: = 0px; text-transform: none; white-space: normal; widows: auto; = word-spacing: 0px; -webkit-text-size-adjust: auto; = -webkit-text-stroke-width: 0px;" class=3D""><br = class=3D"Apple-interchange-newline">On 24 Nov 2016, at 10:32, Evgheni = Dereveanchin <<a href=3D"mailto:ederevea@redhat.com" = class=3D"">ederevea@redhat.com</a>> wrote:<br class=3D""><br = class=3D"">Hi everyone,<br class=3D""><br class=3D"">CI is failing on = snapshot merge tests:<br class=3D""><a = href=3D"http://jenkins.ovirt.org/job/test-repo_ovirt_experimental_master/3= 618/testReport/junit/(root)/004_basic_sanity/snapshot_merge/" = class=3D"">http://jenkins.ovirt.org/job/test-repo_ovirt_experimental_maste= r/3618/testReport/junit/(root)/004_basic_sanity/snapshot_merge/</a><br = class=3D""><br class=3D"">Not sure what is causing this but I see = several patches<span class=3D"Apple-converted-space"> </span><br = class=3D"">that may be related that were recently merged:<br = class=3D"">https://gerrit.ovirt.org/#/c/67064<br = class=3D"">https://gerrit.ovirt.org/#/c/67184<br class=3D""><br = class=3D"">Could someone please check why it's happening as we<br = class=3D"">need to fix/revert the problematic patch to make tests<br = class=3D"">work again.<br class=3D""></blockquote><br = style=3D"font-family: Helvetica; font-size: 12px; font-style: normal; = font-variant-caps: normal; font-weight: normal; letter-spacing: normal; = orphans: auto; text-align: start; text-indent: 0px; text-transform: = none; white-space: normal; widows: auto; word-spacing: 0px; = -webkit-text-stroke-width: 0px;" class=3D""><span style=3D"font-family: = Helvetica; font-size: 12px; font-style: normal; font-variant-caps: = normal; font-weight: normal; letter-spacing: normal; orphans: auto; = text-align: start; text-indent: 0px; text-transform: none; white-space: = normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px; = float: none; display: inline !important;" class=3D"">I have to say I=E2=80= =99m completely lost in what is being run, why, and with what = data</span><br style=3D"font-family: Helvetica; font-size: 12px; = font-style: normal; font-variant-caps: normal; font-weight: normal; = letter-spacing: normal; orphans: auto; text-align: start; text-indent: = 0px; text-transform: none; white-space: normal; widows: auto; = word-spacing: 0px; -webkit-text-stroke-width: 0px;" class=3D""><span = style=3D"font-family: Helvetica; font-size: 12px; font-style: normal; = font-variant-caps: normal; font-weight: normal; letter-spacing: normal; = orphans: auto; text-align: start; text-indent: 0px; text-transform: = none; white-space: normal; widows: auto; word-spacing: 0px; = -webkit-text-stroke-width: 0px; float: none; display: inline = !important;" class=3D"">for both patches in gerrit all = check-patch/check-merged are green and working=E2=80=A6</span><br = style=3D"font-family: Helvetica; font-size: 12px; font-style: normal; = font-variant-caps: normal; font-weight: normal; letter-spacing: normal; = orphans: auto; text-align: start; text-indent: 0px; text-transform: = none; white-space: normal; widows: auto; word-spacing: 0px; = -webkit-text-stroke-width: 0px;" class=3D""><br style=3D"font-family: = Helvetica; font-size: 12px; font-style: normal; font-variant-caps: = normal; font-weight: normal; letter-spacing: normal; orphans: auto; = text-align: start; text-indent: 0px; text-transform: none; white-space: = normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: = 0px;" class=3D""><blockquote type=3D"cite" style=3D"font-family: = Helvetica; font-size: 12px; font-style: normal; font-variant-caps: = normal; font-weight: normal; letter-spacing: normal; orphans: auto; = text-align: start; text-indent: 0px; text-transform: none; white-space: = normal; widows: auto; word-spacing: 0px; -webkit-text-size-adjust: auto; = -webkit-text-stroke-width: 0px;" class=3D""><br class=3D"">Regards,<span = class=3D"Apple-converted-space"> </span><br class=3D"">Evgheni = Dereveanchin<span class=3D"Apple-converted-space"> </span><br = class=3D"">_______________________________________________<br = class=3D"">Devel mailing list<br class=3D""><a = href=3D"mailto:Devel@ovirt.org" class=3D"">Devel@ovirt.org</a><br = class=3D""><a href=3D"http://lists.ovirt.org/mailman/listinfo/devel" = class=3D"">http://lists.ovirt.org/mailman/listinfo/devel</a><br = class=3D""><br class=3D""><br class=3D""></blockquote><br = style=3D"font-family: Helvetica; font-size: 12px; font-style: normal; = font-variant-caps: normal; font-weight: normal; letter-spacing: normal; = orphans: auto; text-align: start; text-indent: 0px; text-transform: = none; white-space: normal; widows: auto; word-spacing: 0px; = -webkit-text-stroke-width: 0px;" class=3D""><span style=3D"font-family: = Helvetica; font-size: 12px; font-style: normal; font-variant-caps: = normal; font-weight: normal; letter-spacing: normal; orphans: auto; = text-align: start; text-indent: 0px; text-transform: none; white-space: = normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px; = float: none; display: inline !important;" = class=3D"">_______________________________________________</span><br = style=3D"font-family: Helvetica; font-size: 12px; font-style: normal; = font-variant-caps: normal; font-weight: normal; letter-spacing: normal; = orphans: auto; text-align: start; text-indent: 0px; text-transform: = none; white-space: normal; widows: auto; word-spacing: 0px; = -webkit-text-stroke-width: 0px;" class=3D""><span style=3D"font-family: = Helvetica; font-size: 12px; font-style: normal; font-variant-caps: = normal; font-weight: normal; letter-spacing: normal; orphans: auto; = text-align: start; text-indent: 0px; text-transform: none; white-space: = normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px; = float: none; display: inline !important;" class=3D"">Devel mailing = list</span><br style=3D"font-family: Helvetica; font-size: 12px; = font-style: normal; font-variant-caps: normal; font-weight: normal; = letter-spacing: normal; orphans: auto; text-align: start; text-indent: = 0px; text-transform: none; white-space: normal; widows: auto; = word-spacing: 0px; -webkit-text-stroke-width: 0px;" class=3D""><a = href=3D"mailto:Devel@ovirt.org" style=3D"font-family: Helvetica; = font-size: 12px; font-style: normal; font-variant-caps: normal; = font-weight: normal; letter-spacing: normal; orphans: auto; text-align: = start; text-indent: 0px; text-transform: none; white-space: normal; = widows: auto; word-spacing: 0px; -webkit-text-size-adjust: auto; = -webkit-text-stroke-width: 0px;" class=3D"">Devel@ovirt.org</a><br = style=3D"font-family: Helvetica; font-size: 12px; font-style: normal; = font-variant-caps: normal; font-weight: normal; letter-spacing: normal; = orphans: auto; text-align: start; text-indent: 0px; text-transform: = none; white-space: normal; widows: auto; word-spacing: 0px; = -webkit-text-stroke-width: 0px;" class=3D""><a = href=3D"http://lists.ovirt.org/mailman/listinfo/devel" = style=3D"font-family: Helvetica; font-size: 12px; font-style: normal; = font-variant-caps: normal; font-weight: normal; letter-spacing: normal; = orphans: auto; text-align: start; text-indent: 0px; text-transform: = none; white-space: normal; widows: auto; word-spacing: 0px; = -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px;" = class=3D"">http://lists.ovirt.org/mailman/listinfo/devel</a></div></blockq= uote></div><br class=3D""></body></html>= --Apple-Mail=_2ACE1794-FA87-4DEA-B70E-C7B15EA833BE--

On Thu, Nov 24, 2016 at 11:37 AM, Michal Skrivanek < michal.skrivanek@redhat.com> wrote:
On 24 Nov 2016, at 10:32, Evgheni Dereveanchin <ederevea@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.
I have to say I’m completely lost in what is being run, why, and with what data for both patches in gerrit all check-patch/check-merged are green and working…
These tests are not running in check-patch or merged, they run after each project is built (i.e commit merged), so even if you see all green in CI, you need to understand there might be failures post merge which are REAL and needs to be fixed the soon they are found.
Regards, Evgheni Dereveanchin _______________________________________________ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
_______________________________________________ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
-- Eyal Edri Associate Manager RHV DevOps EMEA ENG Virtualization R&D Red Hat Israel phone: +972-9-7692018 irc: eedri (on #tlv #rhev-dev #rhev-integ)

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@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

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@redhat.com> To: "Evgheni Dereveanchin" <ederevea@redhat.com> Cc: "devel" <devel@ovirt.org>, "Nir Soffer" <nsoffer@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@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

Sent a fix patch: https://gerrit.ovirt.org/#/c/67260/ On Thu, Nov 24, 2016 at 11:44 AM, Evgheni Dereveanchin <ederevea@redhat.com> wrote:
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@redhat.com> To: "Evgheni Dereveanchin" <ederevea@redhat.com> Cc: "devel" <devel@ovirt.org>, "Nir Soffer" <nsoffer@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@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
_______________________________________________ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
participants (6)
-
Ala Hino
-
Evgheni Dereveanchin
-
Eyal Edri
-
Maor Lipchuk
-
Michal Skrivanek
-
Vinzenz Feenstra