Re: [ovirt-devel] Merging patches into vdsm without CI
Sent patch and verifyed: https://gerrit.ovirt.org/#/c/45281 On 08/25, Dan Kenigsberg wrote: On Mon, Aug 24, 2015 at 03:43:25PM +0200, David Caro wrote: I've enabled the job that the jenkins patch creates, created manually, but review the patch so I can merge it please: http://jenkins.ovirt.org/job/vdsm_master_check-patch-fc22-x86_64/ Thanks. We've now taken automation/check-patch.sh into ovirt-3.6. Can you issue a similar job for 3.6? It would be nice to run in on el7. Regards, Dan -- David Caro Red Hat S.L. Continuous Integration Engineer - EMEA ENG Virtualization RD Tel.: +420 532 294 605 Email: dc...@redhat.com Web: www.redhat.com RHT Global #: 82-62605 pgp9hGyXvuWLd.pgp Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] Merging patches into vdsm without CI
On Mon, Aug 24, 2015 at 03:43:25PM +0200, David Caro wrote: I've enabled the job that the jenkins patch creates, created manually, but review the patch so I can merge it please: http://jenkins.ovirt.org/job/vdsm_master_check-patch-fc22-x86_64/ Thanks. We've now taken automation/check-patch.sh into ovirt-3.6. Can you issue a similar job for 3.6? It would be nice to run in on el7. Regards, Dan ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] Merging patches into vdsm without CI
On 08/23, Dan Kenigsberg wrote: On Fri, Aug 21, 2015 at 05:04:03PM -0400, Adam Litke wrote: Hi all, Recent breakage in the vdsm CI flows have caused the change upload trigger to be disabled. This means that CI scores are no longer being automatically applied to uploaded changes. This means that patches cannot be merged into vdsm. I have a queue of patches which are otherwise ready for merge (which have passed CI in the past but needed rebasing). These patches have been stalled for almost a week now. What can we to to unfreeze the vdsm development process in 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 solution). Francesco and David were supposed to revive the broken job on Friday, but I see that it is still disabled. What is the reason of the delay? 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). If that's allowable, then it can be enabled right away (you'll have to rebase to pull the scripts in). -- David Caro Red Hat S.L. Continuous Integration Engineer - EMEA ENG Virtualization RD Tel.: +420 532 294 605 Email: dc...@redhat.com Web: www.redhat.com RHT Global #: 82-62605 pgp5EtNuRmZQe.pgp Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] Merging patches into vdsm without CI
I've enabled the job that the jenkins patch creates, created manually, but review the patch so I can merge it please: http://jenkins.ovirt.org/job/vdsm_master_check-patch-fc22-x86_64/ On 08/24, Francesco Romani wrote: - Original Message - From: Dan Kenigsberg dan...@redhat.com To: Francesco Romani from...@redhat.com Cc: David Caro dc...@redhat.com, Adam Litke ali...@redhat.com, devel@ovirt.org, Sandro Bonazzola sbona...@redhat.com, David Caro Estevez dcaro...@redhat.com Sent: Monday, August 24, 2015 2:36:55 PM Subject: Re: Merging patches into vdsm without CI On Mon, Aug 24, 2015 at 03:32:53AM -0400, Francesco Romani wrote: - Original Message - From: Dan Kenigsberg dan...@redhat.com To: David Caro dc...@redhat.com Cc: Adam Litke ali...@redhat.com, from...@redhat.com, devel@ovirt.org, Sandro Bonazzola sbona...@redhat.com, David Caro Estevez dcaro...@redhat.com Sent: Monday, August 24, 2015 8:53:07 AM Subject: Re: Merging patches into vdsm without CI 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, Recent breakage in the vdsm CI flows have caused the change upload trigger to be disabled. This means that CI scores are no longer being automatically applied to uploaded changes. This means that patches cannot be merged into vdsm. I have a queue of patches which are otherwise ready for merge (which have passed CI in the past but needed rebasing). These patches have been stalled for almost a week now. What can we to to unfreeze the vdsm development process in 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 solution). Francesco and David were supposed to revive the broken job on Friday, but I see that it is still disabled. What is the reason of the delay? 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). If that's allowable, then it can be enabled right away (you'll have to rebase to pull the scripts in). I'm not sure how this can be done. but in any case, I'd rather NOSE_EXCLUDE tcTests until we solve this issue. Hi all, this is true. I didn't want to rush in a possible half-baked solution without any ack from the owners of the tests. With this patch: https://gerrit.ovirt.org/#/c/45134/3 the CI jobs WORKSFORME. I used these instructions http://www.ovirt.org/index.php?title=CI/Build_and_test_standards#Testing_the_scripts_locally and tested on el7 and fc22 succesfully. I've merged these patches (despite little comments that can wait for a follow-up patch). David, can you now enable CI on the master branch? Later, Francesco, would you backport these two patches to 3.6? Sure thing! I'll just wait for Jenkins second check before to backport patches. Bests, -- Francesco Romani RedHat Engineering Virtualization R D Phone: 8261328 IRC: fromani -- David Caro Red Hat S.L. Continuous Integration Engineer - EMEA ENG Virtualization RD Tel.: +420 532 294 605 Email: dc...@redhat.com Web: www.redhat.com RHT Global #: 82-62605 pgpFCshD4DIlq.pgp Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] Merging patches into vdsm without CI
- Original Message - From: Dan Kenigsberg dan...@redhat.com To: Francesco Romani from...@redhat.com Cc: David Caro dc...@redhat.com, Adam Litke ali...@redhat.com, devel@ovirt.org, Sandro Bonazzola sbona...@redhat.com, David Caro Estevez dcaro...@redhat.com Sent: Monday, August 24, 2015 2:36:55 PM Subject: Re: Merging patches into vdsm without CI On Mon, Aug 24, 2015 at 03:32:53AM -0400, Francesco Romani wrote: - Original Message - From: Dan Kenigsberg dan...@redhat.com To: David Caro dc...@redhat.com Cc: Adam Litke ali...@redhat.com, from...@redhat.com, devel@ovirt.org, Sandro Bonazzola sbona...@redhat.com, David Caro Estevez dcaro...@redhat.com Sent: Monday, August 24, 2015 8:53:07 AM Subject: Re: Merging patches into vdsm without CI 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, Recent breakage in the vdsm CI flows have caused the change upload trigger to be disabled. This means that CI scores are no longer being automatically applied to uploaded changes. This means that patches cannot be merged into vdsm. I have a queue of patches which are otherwise ready for merge (which have passed CI in the past but needed rebasing). These patches have been stalled for almost a week now. What can we to to unfreeze the vdsm development process in 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 solution). Francesco and David were supposed to revive the broken job on Friday, but I see that it is still disabled. What is the reason of the delay? 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). If that's allowable, then it can be enabled right away (you'll have to rebase to pull the scripts in). I'm not sure how this can be done. but in any case, I'd rather NOSE_EXCLUDE tcTests until we solve this issue. Hi all, this is true. I didn't want to rush in a possible half-baked solution without any ack from the owners of the tests. With this patch: https://gerrit.ovirt.org/#/c/45134/3 the CI jobs WORKSFORME. I used these instructions http://www.ovirt.org/index.php?title=CI/Build_and_test_standards#Testing_the_scripts_locally and tested on el7 and fc22 succesfully. I've merged these patches (despite little comments that can wait for a follow-up patch). David, can you now enable CI on the master branch? Later, Francesco, would you backport these two patches to 3.6? Sure thing! I'll just wait for Jenkins second check before to backport patches. Bests, -- Francesco Romani RedHat Engineering Virtualization R D Phone: 8261328 IRC: fromani ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] Merging patches into vdsm without CI
- Original Message - From: Dan Kenigsberg dan...@redhat.com To: David Caro dc...@redhat.com Cc: Adam Litke ali...@redhat.com, from...@redhat.com, devel@ovirt.org, Sandro Bonazzola sbona...@redhat.com, David Caro Estevez dcaro...@redhat.com Sent: Monday, August 24, 2015 8:53:07 AM Subject: Re: Merging patches into vdsm without CI 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, Recent breakage in the vdsm CI flows have caused the change upload trigger to be disabled. This means that CI scores are no longer being automatically applied to uploaded changes. This means that patches cannot be merged into vdsm. I have a queue of patches which are otherwise ready for merge (which have passed CI in the past but needed rebasing). These patches have been stalled for almost a week now. What can we to to unfreeze the vdsm development process in 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 solution). Francesco and David were supposed to revive the broken job on Friday, but I see that it is still disabled. What is the reason of the delay? 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). If that's allowable, then it can be enabled right away (you'll have to rebase to pull the scripts in). I'm not sure how this can be done. but in any case, I'd rather NOSE_EXCLUDE tcTests until we solve this issue. Hi all, this is true. I didn't want to rush in a possible half-baked solution without any ack from the owners of the tests. With this patch: https://gerrit.ovirt.org/#/c/45134/3 the CI jobs WORKSFORME. I used these instructions http://www.ovirt.org/index.php?title=CI/Build_and_test_standards#Testing_the_scripts_locally and tested on el7 and fc22 succesfully. Bests, -- Francesco Romani RedHat Engineering Virtualization R D Phone: 8261328 IRC: fromani ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] Merging patches into vdsm without CI
On Mon, Aug 24, 2015 at 03:32:53AM -0400, Francesco Romani wrote: - Original Message - From: Dan Kenigsberg dan...@redhat.com To: David Caro dc...@redhat.com Cc: Adam Litke ali...@redhat.com, from...@redhat.com, devel@ovirt.org, Sandro Bonazzola sbona...@redhat.com, David Caro Estevez dcaro...@redhat.com Sent: Monday, August 24, 2015 8:53:07 AM Subject: Re: Merging patches into vdsm without CI 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, Recent breakage in the vdsm CI flows have caused the change upload trigger to be disabled. This means that CI scores are no longer being automatically applied to uploaded changes. This means that patches cannot be merged into vdsm. I have a queue of patches which are otherwise ready for merge (which have passed CI in the past but needed rebasing). These patches have been stalled for almost a week now. What can we to to unfreeze the vdsm development process in 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 solution). Francesco and David were supposed to revive the broken job on Friday, but I see that it is still disabled. What is the reason of the delay? 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). If that's allowable, then it can be enabled right away (you'll have to rebase to pull the scripts in). I'm not sure how this can be done. but in any case, I'd rather NOSE_EXCLUDE tcTests until we solve this issue. Hi all, this is true. I didn't want to rush in a possible half-baked solution without any ack from the owners of the tests. With this patch: https://gerrit.ovirt.org/#/c/45134/3 the CI jobs WORKSFORME. I used these instructions http://www.ovirt.org/index.php?title=CI/Build_and_test_standards#Testing_the_scripts_locally and tested on el7 and fc22 succesfully. I've merged these patches (despite little comments that can wait for a follow-up patch). David, can you now enable CI on the master branch? Later, Francesco, would you backport these two patches to 3.6? Dan. ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] Merging patches into vdsm without CI
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: - Original Message - From: Dan Kenigsberg dan...@redhat.com To: Francesco Romani from...@redhat.com Cc: David Caro dc...@redhat.com, Adam Litke ali...@redhat.com, devel@ovirt.org, Sandro Bonazzola sbona...@redhat.com, David Caro Estevez dcaro...@redhat.com Sent: Monday, August 24, 2015 2:36:55 PM Subject: Re: Merging patches into vdsm without CI On Mon, Aug 24, 2015 at 03:32:53AM -0400, Francesco Romani wrote: - Original Message - From: Dan Kenigsberg dan...@redhat.com To: David Caro dc...@redhat.com Cc: Adam Litke ali...@redhat.com, from...@redhat.com, devel@ovirt.org, Sandro Bonazzola sbona...@redhat.com, David Caro Estevez dcaro...@redhat.com Sent: Monday, August 24, 2015 8:53:07 AM Subject: Re: Merging patches into vdsm without CI 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, Recent breakage in the vdsm CI flows have caused the change upload trigger to be disabled. This means that CI scores are no longer being automatically applied to uploaded changes. This means that patches cannot be merged into vdsm. I have a queue of patches which are otherwise ready for merge (which have passed CI in the past but needed rebasing). These patches have been stalled for almost a week now. What can we to to unfreeze the vdsm development process in 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 solution). Francesco and David were supposed to revive the broken job on Friday, but I see that it is still disabled. What is the reason of the delay? 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). If that's allowable, then it can be enabled right away (you'll have to rebase to pull the scripts in). I'm not sure how this can be done. but in any case, I'd rather NOSE_EXCLUDE tcTests until we solve this issue. Hi all, this is true. I didn't want to rush in a possible half-baked solution without any ack from the owners of the tests. With this patch: https://gerrit.ovirt.org/#/c/45134/3 the CI jobs WORKSFORME. I used these instructions http://www.ovirt.org/index.php?title=CI/Build_and_test_standards#Testing_the_scripts_locally and tested on el7 and fc22 succesfully. I've merged these patches (despite little comments that can wait for a follow-up patch). David, can you now enable CI on the master branch? Later, Francesco, would you backport these two patches to 3.6? Sure thing! I'll just wait for Jenkins second check before to backport patches. Bests, -- Francesco Romani RedHat Engineering Virtualization R D Phone: 8261328 IRC: fromani -- David Caro Red Hat S.L. Continuous Integration Engineer - EMEA ENG Virtualization RD Tel.: +420 532 294 605 Email: dc...@redhat.com Web: www.redhat.com RHT Global #: 82-62605 pgpAqPtWiafre.pgp Description: PGP signature ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] Merging patches into vdsm without CI
On Fri, Aug 21, 2015 at 05:04:03PM -0400, Adam Litke wrote: Hi all, Recent breakage in the vdsm CI flows have caused the change upload trigger to be disabled. This means that CI scores are no longer being automatically applied to uploaded changes. This means that patches cannot be merged into vdsm. I have a queue of patches which are otherwise ready for merge (which have passed CI in the past but needed rebasing). These patches have been stalled for almost a week now. What can we to to unfreeze the vdsm development process in 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 solution). Francesco and David were supposed to revive the broken job on Friday, but I see that it is still disabled. What is the reason of the delay? ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
Re: [ovirt-devel] Merging patches into vdsm without CI
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, Recent breakage in the vdsm CI flows have caused the change upload trigger to be disabled. This means that CI scores are no longer being automatically applied to uploaded changes. This means that patches cannot be merged into vdsm. I have a queue of patches which are otherwise ready for merge (which have passed CI in the past but needed rebasing). These patches have been stalled for almost a week now. What can we to to unfreeze the vdsm development process in 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 solution). Francesco and David were supposed to revive the broken job on Friday, but I see that it is still disabled. What is the reason of the delay? 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). If that's allowable, then it can be enabled right away (you'll have to rebase to pull the scripts in). I'm not sure how this can be done. but in any case, I'd rather NOSE_EXCLUDE tcTests until we solve this issue. ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
[ovirt-devel] Merging patches into vdsm without CI
Hi all, Recent breakage in the vdsm CI flows have caused the change upload trigger to be disabled. This means that CI scores are no longer being automatically applied to uploaded changes. This means that patches cannot be merged into vdsm. I have a queue of patches which are otherwise ready for merge (which have passed CI in the past but needed rebasing). These patches have been stalled for almost a week now. What can we to to unfreeze the vdsm development process in 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 solution). -- Adam Litke ___ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel