----- Original Message -----
From: "Dan Kenigsberg" <danken@redhat.com> To: "Francesco Romani" <fromani@redhat.com> Cc: "David Caro" <dcaro@redhat.com>, "Adam Litke" <alitke@redhat.com>, = devel@ovirt.org, "Sandro Bonazzola" <sbonazzo@redhat.com>, "David Caro Estevez" <dcaroest@redhat.com> Sent: Monday, August 24, 2015 2:36:55 PM Subject: Re: Merging patches into vdsm without CI =20 On Mon, Aug 24, 2015 at 03:32:53AM -0400, Francesco Romani wrote:
----- Original Message -----
From: "Dan Kenigsberg" <danken@redhat.com> To: "David Caro" <dcaro@redhat.com> Cc: "Adam Litke" <alitke@redhat.com>, fromani@redhat.com, devel@ovirt.org, "Sandro Bonazzola" <sbonazzo@redhat.com>, "David Caro Estevez" <dcaroest@redhat.com> Sent: Monday, August 24, 2015 8:53:07 AM Subject: Re: Merging patches into vdsm without CI =20 On Mon, Aug 24, 2015 at 12:01:58AM +0200, David Caro wrote:
On 08/23, Dan Kenigsberg wrote:
On Fri, Aug 21, 2015 at 05:04:03PM -0400, Adam Litke wrote: > Hi all, >=20 > Recent breakage in the vdsm CI flows have caused the change u=
> trigger to be disabled. This means that CI scores are no lon= ger > being > automatically applied to uploaded changes. This means that p= atches > cannot be merged into vdsm. I have a queue of patches which = are > otherwise ready for merge (which have passed CI in the past b= ut > needed > rebasing). These patches have been stalled for almost a week= now. >=20 > What can we to to "unfreeze" the vdsm development process in =
--UugvWAfsgieZRqgk Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable The patch for the jenkins job is here: https://gerrit.ovirt.org/#/c/42477/ let me know if that's ok and I'll merge it (you might want for example run = it on more distros) On 08/24, Francesco Romani wrote: pload the
> short > and long term? Earlier today I worked with Sandro and David = on > manually running CI on my dev machine but am getting 100s of > failures > (so it looks like this wont even be a good short-term solutio= n). =20 Francesco and David were supposed to revive the broken job on F= riday, but I see that it is still disabled. =20 What is the reason of the delay? =20 Francesco wanted the approval of the network guys, as the job has= to create the tun char device itself (because mock does not do it). =20 =20 If that's allowable, then it can be enabled right away (you'll ha= ve to rebase to pull the scripts in). =20 I'm not sure how this can be done. but in any case, I'd rather NOSE_EXCLUDE tcTests until we solve this issue. =20 Hi all, =20 this is true. I didn't want to rush in a possible half-baked "solutio= n" without any ack from the owners of the tests. =20 With this patch: https://gerrit.ovirt.org/#/c/45134/3 the CI jobs WORKSFORME. =20 I used these instructions =20 http://www.ovirt.org/index.php?title=3DCI/Build_and_test_standards#Te= sting_the_scripts_locally =20 and tested on el7 and fc22 succesfully. =20 I've merged these patches (despite little comments that can wait for a follow-up patch). =20 David, can you now enable CI on the master branch? Later, Francesco, would you backport these two patches to 3.6? =20 Sure thing! I'll just wait for Jenkins second check before to backport pa= tches. =20 Bests, =20 --=20 Francesco Romani RedHat Engineering Virtualization R & D Phone: 8261328 IRC: fromani
--=20 David Caro Red Hat S.L. Continuous Integration Engineer - EMEA ENG Virtualization R&D Tel.: +420 532 294 605 Email: dcaro@redhat.com Web: www.redhat.com RHT Global #: 82-62605 --UugvWAfsgieZRqgk Content-Type: application/pgp-signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v1 iQEcBAEBAgAGBQJV2x82AAoJEEBxx+HSYmnDr/kIAJR4uCySprd2d5LWcDrKmsnd hFJ6gbbMO0eo6Y23MNU6qF24B6i+CuVBRAdiN5nnZe7Ny/n+mgyjGea1DpSYuMyV IX7Gb0ADDXz/sUr4KrulyH6F3PQDo5sMRp4FimdC+Z5mOhdgUbtNzOvqFii2Fuxg 5N1WtMphH1qNuPKMBtTYE9JcE5fdsuCRP6/IclQEeAJN9zOze6C9TA+SzW3R2/oN SfN71cQ/i2DhhLL6L0HDS+pfJmUG3HX+A8lSAa/cTagwnqQOuyfMbVXxOzDWygc5 XJDrtbcXfeDOXmMYAgckMYuzcDseMUluHdqpVMWaj+M8vvkFzdAPRflAc/2HqLk= =kcFr -----END PGP SIGNATURE----- --UugvWAfsgieZRqgk--