Re: [vdsm] Jenkins and Gerrit.
On 08/09/2012 03:37 AM, Eyal Edri wrote: - Original Message - From: "Robert Middleswarth" To: "Dan Kenigsberg" Cc: "VDSM Project Development" , "infra" Sent: Wednesday, August 8, 2012 10:53:35 PM Subject: Re: [vdsm] Jenkins and Gerrit. On 08/08/2012 03:07 PM, Dan Kenigsberg wrote: On Wed, Aug 08, 2012 at 09:58:17AM -0400, Robert Middleswarth wrote: On 08/08/2012 09:50 AM, Dan Kenigsberg wrote: On Wed, Aug 08, 2012 at 02:55:17PM +0200, Ewoud Kohl van Wijngaarden wrote: On Wed, Aug 08, 2012 at 03:48:13PM +0300, Dan Kenigsberg wrote: On Wed, Aug 08, 2012 at 07:47:02AM -0400, Robert Middleswarth wrote: I have setup patch review on Jenkins.info for newly submitted patches and it seems to be working pretty well over all but last night well tweaking the process I broken it for a few min but that was long enough that about 50 jobs were marked -1 I will be fixing that today by rerunning the jobs. I am sorry if one of your patches was dinged and it should be fixed by this time tomorrow. Thanks, Robert, for working on this. It is highly important for me to know that something is going to break the build before taking it in. However, would it be possible to have a repository where we can review the code of the robot? It's Gerrit Trigger[1] and the code is on github[2]. I think it is important for the robot to be less noisy, and particularly, never give V+1. This task is reserved to humans that actually know what the patch should be doing. The V+1 has been fixed. Will give 0 when they pass, -1 when they fail. Also, I am not at all sure that the robot is limitting itself to be running code of trustworthy authors. Eyal added a feature request for this[3]. This was the result of a discussion on the infra mailing list[4]. As much as I like (and need) this per-commit verification, I think we should not deploy it before the feature is implemented. BTW, Federico suggested to initiate the test only on request (when oVirt Jenkins CI Server is added as reviewer). This would allow a more silent start for CI. Thanks, Dan. I already wrote a little bash code to do this outside the plug-in. It will be in place by the end of the day. This kind of script is exactly the thing I'd like to be peer-reviewed before applied en mass to gerrit changes. Particularly due to the security implications. Regards, Dan. If you are talking about the jenkins app that updates Gerrit that is has been in use on ovirt-node-devel for some time. As for the whitelist script that is like 4 lines. git log --pretty="%ce" -n 1 > $WORKSPACE/current_author.txt grep -f $WORKSPACE/current_author.txt $WORKSPACE/jenkins-whitelist.txt RETVAL=$? [ $RETVAL -ne 0 ] && curl -u jenkins_bot:xx $BUILD_URL/stop; It is simple and the files are generated outside of the repo so it should be safe. i think it's better to use $GERRIT_CHANGE_OWNER_NAME or $GERRIT_CHANGE_OWNER_EMAIL. I didn't see that documented any places and your are right $GERRIT_CHANGE_OWNER_EMAIL would be a better choice. Test updated to use that variable. Thanks Robert -- Thanks Robert Middleswarth @rmiddle (twitter/IRC) ___ Infra mailing list in...@ovirt.org http://lists.ovirt.org/mailman/listinfo/infra -- Thanks Robert Middleswarth @rmiddle (twitter/IRC) ___ vdsm-devel mailing list vdsm-devel@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-devel
Re: [vdsm] Jenkins and Gerrit.
On 08/09/2012 03:30 AM, Dan Kenigsberg wrote: On Wed, Aug 08, 2012 at 03:53:35PM -0400, Robert Middleswarth wrote: On 08/08/2012 03:07 PM, Dan Kenigsberg wrote: On Wed, Aug 08, 2012 at 09:58:17AM -0400, Robert Middleswarth wrote: On 08/08/2012 09:50 AM, Dan Kenigsberg wrote: On Wed, Aug 08, 2012 at 02:55:17PM +0200, Ewoud Kohl van Wijngaarden wrote: On Wed, Aug 08, 2012 at 03:48:13PM +0300, Dan Kenigsberg wrote: On Wed, Aug 08, 2012 at 07:47:02AM -0400, Robert Middleswarth wrote: I have setup patch review on Jenkins.info for newly submitted patches and it seems to be working pretty well over all but last night well tweaking the process I broken it for a few min but that was long enough that about 50 jobs were marked -1 I will be fixing that today by rerunning the jobs. I am sorry if one of your patches was dinged and it should be fixed by this time tomorrow. Thanks, Robert, for working on this. It is highly important for me to know that something is going to break the build before taking it in. However, would it be possible to have a repository where we can review the code of the robot? It's Gerrit Trigger[1] and the code is on github[2]. I think it is important for the robot to be less noisy, and particularly, never give V+1. This task is reserved to humans that actually know what the patch should be doing. The V+1 has been fixed. Will give 0 when they pass, -1 when they fail. Also, I am not at all sure that the robot is limitting itself to be running code of trustworthy authors. Eyal added a feature request for this[3]. This was the result of a discussion on the infra mailing list[4]. As much as I like (and need) this per-commit verification, I think we should not deploy it before the feature is implemented. BTW, Federico suggested to initiate the test only on request (when oVirt Jenkins CI Server is added as reviewer). This would allow a more silent start for CI. Thanks, Dan. I already wrote a little bash code to do this outside the plug-in. It will be in place by the end of the day. This kind of script is exactly the thing I'd like to be peer-reviewed before applied en mass to gerrit changes. Particularly due to the security implications. Regards, Dan. If you are talking about the jenkins app that updates Gerrit that is has been in use on ovirt-node-devel for some time. As for the whitelist script that is like 4 lines. git log --pretty="%ce" -n 1 > $WORKSPACE/current_author.txt Are we sure that the top author is good enough? What if a trusted user builds on top of a non-trusted user? Does it mean that the lower commits are automatically trusted? You would assume that is someone we trusted were to use code from a non-trusted users they reviewed the code to make sure there wasn't some code in there to try and hack the system. Once you take the code and include it you have taken ownership of that code. Thanks Robert grep -f $WORKSPACE/current_author.txt $WORKSPACE/jenkins-whitelist.txt RETVAL=$? [ $RETVAL -ne 0 ] && curl -u jenkins_bot:xx $BUILD_URL/stop; It is simple and the files are generated outside of the repo so it should be safe. -- Thanks Robert Middleswarth @rmiddle (twitter/IRC) ___ vdsm-devel mailing list vdsm-devel@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-devel
Re: [vdsm] Jenkins and Gerrit.
- Original Message - > From: "Robert Middleswarth" > To: "Dan Kenigsberg" > Cc: "VDSM Project Development" , "infra" > > Sent: Wednesday, August 8, 2012 10:53:35 PM > Subject: Re: [vdsm] Jenkins and Gerrit. > > On 08/08/2012 03:07 PM, Dan Kenigsberg wrote: > > On Wed, Aug 08, 2012 at 09:58:17AM -0400, Robert Middleswarth > > wrote: > >> On 08/08/2012 09:50 AM, Dan Kenigsberg wrote: > >>> On Wed, Aug 08, 2012 at 02:55:17PM +0200, Ewoud Kohl van > >>> Wijngaarden wrote: > >>>> On Wed, Aug 08, 2012 at 03:48:13PM +0300, Dan Kenigsberg wrote: > >>>>> On Wed, Aug 08, 2012 at 07:47:02AM -0400, Robert Middleswarth > >>>>> wrote: > >>>>>> I have setup patch review on Jenkins.info for newly submitted > >>>>>> patches and it seems to be working pretty well over all but > >>>>>> last > >>>>>> night well tweaking the process I broken it for a few min but > >>>>>> that > >>>>>> was long enough that about 50 jobs were marked -1 I will be > >>>>>> fixing > >>>>>> that today by rerunning the jobs. I am sorry if one of your > >>>>>> patches > >>>>>> was dinged and it should be fixed by this time tomorrow. > >>>>> Thanks, Robert, for working on this. It is highly important for > >>>>> me to > >>>>> know that something is going to break the build before taking > >>>>> it in. > >>>>> > >>>>> However, would it be possible to have a repository where we can > >>>>> review > >>>>> the code of the robot? > >>>> It's Gerrit Trigger[1] and the code is on github[2]. > >>>> > >>>>> I think it is important for the robot to be less noisy, and > >>>>> particularly, never give V+1. This task is reserved to humans > >>>>> that > >>>>> actually know what the patch should be doing. > >>>> The V+1 has been fixed. Will give 0 when they pass, -1 when they > >>>> fail. > >>>> > >>>>> Also, I am not at all sure that the robot is limitting itself > >>>>> to be > >>>>> running code of trustworthy authors. > >>>> Eyal added a feature request for this[3]. This was the result of > >>>> a > >>>> discussion on the infra mailing list[4]. > >>> As much as I like (and need) this per-commit verification, I > >>> think we > >>> should not deploy it before the feature is implemented. > >>> > >>> BTW, Federico suggested to initiate the test only on request > >>> (when oVirt > >>> Jenkins CI Server is added as reviewer). This would allow a more > >>> silent > >>> start for CI. > >>> > >>> Thanks, > >>> Dan. > >> I already wrote a little bash code to do this outside the plug-in. > >> It will be in place by the end of the day. > > This kind of script is exactly the thing I'd like to be > > peer-reviewed > > before applied en mass to gerrit changes. Particularly due to the > > security implications. > > > > Regards, > > Dan. > If you are talking about the jenkins app that updates Gerrit that is > has > been in use on ovirt-node-devel for some time. As for the whitelist > script that is like 4 lines. > > git log --pretty="%ce" -n 1 > $WORKSPACE/current_author.txt > grep -f $WORKSPACE/current_author.txt > $WORKSPACE/jenkins-whitelist.txt > RETVAL=$? > [ $RETVAL -ne 0 ] && curl -u jenkins_bot:xx $BUILD_URL/stop; > > It is simple and the files are generated outside of the repo so it > should be safe. i think it's better to use $GERRIT_CHANGE_OWNER_NAME or $GERRIT_CHANGE_OWNER_EMAIL. > > -- > Thanks > Robert Middleswarth > @rmiddle (twitter/IRC) > > ___ > Infra mailing list > in...@ovirt.org > http://lists.ovirt.org/mailman/listinfo/infra > ___ vdsm-devel mailing list vdsm-devel@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-devel
Re: [vdsm] Jenkins and Gerrit.
On Wed, Aug 08, 2012 at 03:53:35PM -0400, Robert Middleswarth wrote: > On 08/08/2012 03:07 PM, Dan Kenigsberg wrote: > >On Wed, Aug 08, 2012 at 09:58:17AM -0400, Robert Middleswarth wrote: > >>On 08/08/2012 09:50 AM, Dan Kenigsberg wrote: > >>>On Wed, Aug 08, 2012 at 02:55:17PM +0200, Ewoud Kohl van Wijngaarden wrote: > On Wed, Aug 08, 2012 at 03:48:13PM +0300, Dan Kenigsberg wrote: > >On Wed, Aug 08, 2012 at 07:47:02AM -0400, Robert Middleswarth wrote: > >>I have setup patch review on Jenkins.info for newly submitted > >>patches and it seems to be working pretty well over all but last > >>night well tweaking the process I broken it for a few min but that > >>was long enough that about 50 jobs were marked -1 I will be fixing > >>that today by rerunning the jobs. I am sorry if one of your patches > >>was dinged and it should be fixed by this time tomorrow. > >Thanks, Robert, for working on this. It is highly important for me to > >know that something is going to break the build before taking it in. > > > >However, would it be possible to have a repository where we can review > >the code of the robot? > It's Gerrit Trigger[1] and the code is on github[2]. > > >I think it is important for the robot to be less noisy, and > >particularly, never give V+1. This task is reserved to humans that > >actually know what the patch should be doing. > The V+1 has been fixed. Will give 0 when they pass, -1 when they fail. > > >Also, I am not at all sure that the robot is limitting itself to be > >running code of trustworthy authors. > Eyal added a feature request for this[3]. This was the result of a > discussion on the infra mailing list[4]. > >>>As much as I like (and need) this per-commit verification, I think we > >>>should not deploy it before the feature is implemented. > >>> > >>>BTW, Federico suggested to initiate the test only on request (when oVirt > >>>Jenkins CI Server is added as reviewer). This would allow a more silent > >>>start for CI. > >>> > >>>Thanks, > >>>Dan. > >>I already wrote a little bash code to do this outside the plug-in. > >>It will be in place by the end of the day. > >This kind of script is exactly the thing I'd like to be peer-reviewed > >before applied en mass to gerrit changes. Particularly due to the > >security implications. > > > >Regards, > >Dan. > If you are talking about the jenkins app that updates Gerrit that is > has been in use on ovirt-node-devel for some time. As for the > whitelist script that is like 4 lines. > > git log --pretty="%ce" -n 1 > $WORKSPACE/current_author.txt Are we sure that the top author is good enough? What if a trusted user builds on top of a non-trusted user? Does it mean that the lower commits are automatically trusted? > grep -f $WORKSPACE/current_author.txt $WORKSPACE/jenkins-whitelist.txt > RETVAL=$? > [ $RETVAL -ne 0 ] && curl -u jenkins_bot:xx $BUILD_URL/stop; > > It is simple and the files are generated outside of the repo so it > should be safe. ___ vdsm-devel mailing list vdsm-devel@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-devel
Re: [vdsm] Jenkins and Gerrit.
On 08/08/2012 03:07 PM, Dan Kenigsberg wrote: On Wed, Aug 08, 2012 at 09:58:17AM -0400, Robert Middleswarth wrote: On 08/08/2012 09:50 AM, Dan Kenigsberg wrote: On Wed, Aug 08, 2012 at 02:55:17PM +0200, Ewoud Kohl van Wijngaarden wrote: On Wed, Aug 08, 2012 at 03:48:13PM +0300, Dan Kenigsberg wrote: On Wed, Aug 08, 2012 at 07:47:02AM -0400, Robert Middleswarth wrote: I have setup patch review on Jenkins.info for newly submitted patches and it seems to be working pretty well over all but last night well tweaking the process I broken it for a few min but that was long enough that about 50 jobs were marked -1 I will be fixing that today by rerunning the jobs. I am sorry if one of your patches was dinged and it should be fixed by this time tomorrow. Thanks, Robert, for working on this. It is highly important for me to know that something is going to break the build before taking it in. However, would it be possible to have a repository where we can review the code of the robot? It's Gerrit Trigger[1] and the code is on github[2]. I think it is important for the robot to be less noisy, and particularly, never give V+1. This task is reserved to humans that actually know what the patch should be doing. The V+1 has been fixed. Will give 0 when they pass, -1 when they fail. Also, I am not at all sure that the robot is limitting itself to be running code of trustworthy authors. Eyal added a feature request for this[3]. This was the result of a discussion on the infra mailing list[4]. As much as I like (and need) this per-commit verification, I think we should not deploy it before the feature is implemented. BTW, Federico suggested to initiate the test only on request (when oVirt Jenkins CI Server is added as reviewer). This would allow a more silent start for CI. Thanks, Dan. I already wrote a little bash code to do this outside the plug-in. It will be in place by the end of the day. This kind of script is exactly the thing I'd like to be peer-reviewed before applied en mass to gerrit changes. Particularly due to the security implications. Regards, Dan. If you are talking about the jenkins app that updates Gerrit that is has been in use on ovirt-node-devel for some time. As for the whitelist script that is like 4 lines. git log --pretty="%ce" -n 1 > $WORKSPACE/current_author.txt grep -f $WORKSPACE/current_author.txt $WORKSPACE/jenkins-whitelist.txt RETVAL=$? [ $RETVAL -ne 0 ] && curl -u jenkins_bot:xx $BUILD_URL/stop; It is simple and the files are generated outside of the repo so it should be safe. -- Thanks Robert Middleswarth @rmiddle (twitter/IRC) ___ vdsm-devel mailing list vdsm-devel@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-devel
Re: [vdsm] Jenkins and Gerrit.
On Wed, Aug 08, 2012 at 09:58:17AM -0400, Robert Middleswarth wrote: > On 08/08/2012 09:50 AM, Dan Kenigsberg wrote: > >On Wed, Aug 08, 2012 at 02:55:17PM +0200, Ewoud Kohl van Wijngaarden wrote: > >>On Wed, Aug 08, 2012 at 03:48:13PM +0300, Dan Kenigsberg wrote: > >>>On Wed, Aug 08, 2012 at 07:47:02AM -0400, Robert Middleswarth wrote: > I have setup patch review on Jenkins.info for newly submitted > patches and it seems to be working pretty well over all but last > night well tweaking the process I broken it for a few min but that > was long enough that about 50 jobs were marked -1 I will be fixing > that today by rerunning the jobs. I am sorry if one of your patches > was dinged and it should be fixed by this time tomorrow. > >>>Thanks, Robert, for working on this. It is highly important for me to > >>>know that something is going to break the build before taking it in. > >>> > >>>However, would it be possible to have a repository where we can review > >>>the code of the robot? > >>It's Gerrit Trigger[1] and the code is on github[2]. > >> > >>>I think it is important for the robot to be less noisy, and > >>>particularly, never give V+1. This task is reserved to humans that > >>>actually know what the patch should be doing. > >>The V+1 has been fixed. Will give 0 when they pass, -1 when they fail. > >> > >>>Also, I am not at all sure that the robot is limitting itself to be > >>>running code of trustworthy authors. > >>Eyal added a feature request for this[3]. This was the result of a > >>discussion on the infra mailing list[4]. > >As much as I like (and need) this per-commit verification, I think we > >should not deploy it before the feature is implemented. > > > >BTW, Federico suggested to initiate the test only on request (when oVirt > >Jenkins CI Server is added as reviewer). This would allow a more silent > >start for CI. > > > >Thanks, > >Dan. > I already wrote a little bash code to do this outside the plug-in. > It will be in place by the end of the day. This kind of script is exactly the thing I'd like to be peer-reviewed before applied en mass to gerrit changes. Particularly due to the security implications. Regards, Dan. ___ vdsm-devel mailing list vdsm-devel@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-devel
Re: [vdsm] Jenkins and Gerrit.
On 08/08/2012 07:47 AM, Robert Middleswarth wrote: I have setup patch review on Jenkins.info for newly submitted patches and it seems to be working pretty well over all but last night well tweaking the process I broken it for a few min but that was long enough that about 50 jobs were marked -1 I will be fixing that today by rerunning the jobs. I am sorry if one of your patches was dinged and it should be fixed by this time tomorrow. I just resubmitted the failed jobs to rerun. They will be marked zero if the pass and -1 if they fail for real. All know issues with the integration for VDSM is finished. -- Thanks Robert Middleswarth @rmiddle (twitter/IRC) ___ vdsm-devel mailing list vdsm-devel@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-devel
Re: [vdsm] Jenkins and Gerrit.
On 08/08/2012 09:50 AM, Dan Kenigsberg wrote: On Wed, Aug 08, 2012 at 02:55:17PM +0200, Ewoud Kohl van Wijngaarden wrote: On Wed, Aug 08, 2012 at 03:48:13PM +0300, Dan Kenigsberg wrote: On Wed, Aug 08, 2012 at 07:47:02AM -0400, Robert Middleswarth wrote: I have setup patch review on Jenkins.info for newly submitted patches and it seems to be working pretty well over all but last night well tweaking the process I broken it for a few min but that was long enough that about 50 jobs were marked -1 I will be fixing that today by rerunning the jobs. I am sorry if one of your patches was dinged and it should be fixed by this time tomorrow. Thanks, Robert, for working on this. It is highly important for me to know that something is going to break the build before taking it in. However, would it be possible to have a repository where we can review the code of the robot? It's Gerrit Trigger[1] and the code is on github[2]. I think it is important for the robot to be less noisy, and particularly, never give V+1. This task is reserved to humans that actually know what the patch should be doing. The V+1 has been fixed. Will give 0 when they pass, -1 when they fail. Also, I am not at all sure that the robot is limitting itself to be running code of trustworthy authors. Eyal added a feature request for this[3]. This was the result of a discussion on the infra mailing list[4]. As much as I like (and need) this per-commit verification, I think we should not deploy it before the feature is implemented. BTW, Federico suggested to initiate the test only on request (when oVirt Jenkins CI Server is added as reviewer). This would allow a more silent start for CI. Thanks, Dan. I already wrote a little bash code to do this outside the plug-in. It will be in place by the end of the day. Thanks Robert [1]: https://wiki.jenkins-ci.org/display/JENKINS/Gerrit+Trigger [2]: https://github.com/jenkinsci/gerrit-trigger-plugin [3]: https://issues.jenkins-ci.org/browse/JENKINS-14655 [4]: http://lists.ovirt.org/pipermail/infra/2012-August/000759.html -- Thanks Robert Middleswarth @rmiddle (twitter/IRC) ___ vdsm-devel mailing list vdsm-devel@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-devel
Re: [vdsm] Jenkins and Gerrit.
On Wed, Aug 08, 2012 at 02:55:17PM +0200, Ewoud Kohl van Wijngaarden wrote: > On Wed, Aug 08, 2012 at 03:48:13PM +0300, Dan Kenigsberg wrote: > > On Wed, Aug 08, 2012 at 07:47:02AM -0400, Robert Middleswarth wrote: > > > I have setup patch review on Jenkins.info for newly submitted > > > patches and it seems to be working pretty well over all but last > > > night well tweaking the process I broken it for a few min but that > > > was long enough that about 50 jobs were marked -1 I will be fixing > > > that today by rerunning the jobs. I am sorry if one of your patches > > > was dinged and it should be fixed by this time tomorrow. > > > > Thanks, Robert, for working on this. It is highly important for me to > > know that something is going to break the build before taking it in. > > > > However, would it be possible to have a repository where we can review > > the code of the robot? > It's Gerrit Trigger[1] and the code is on github[2]. > > > I think it is important for the robot to be less noisy, and > > particularly, never give V+1. This task is reserved to humans that > > actually know what the patch should be doing. > The V+1 has been fixed. Will give 0 when they pass, -1 when they fail. > > > Also, I am not at all sure that the robot is limitting itself to be > > running code of trustworthy authors. > Eyal added a feature request for this[3]. This was the result of a > discussion on the infra mailing list[4]. As much as I like (and need) this per-commit verification, I think we should not deploy it before the feature is implemented. BTW, Federico suggested to initiate the test only on request (when oVirt Jenkins CI Server is added as reviewer). This would allow a more silent start for CI. Thanks, Dan. > > [1]: https://wiki.jenkins-ci.org/display/JENKINS/Gerrit+Trigger > [2]: https://github.com/jenkinsci/gerrit-trigger-plugin > [3]: https://issues.jenkins-ci.org/browse/JENKINS-14655 > [4]: http://lists.ovirt.org/pipermail/infra/2012-August/000759.html ___ vdsm-devel mailing list vdsm-devel@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-devel
Re: [vdsm] Jenkins and Gerrit.
On 08/08/2012 08:48 AM, Dan Kenigsberg wrote: On Wed, Aug 08, 2012 at 07:47:02AM -0400, Robert Middleswarth wrote: I have setup patch review on Jenkins.info for newly submitted patches and it seems to be working pretty well over all but last night well tweaking the process I broken it for a few min but that was long enough that about 50 jobs were marked -1 I will be fixing that today by rerunning the jobs. I am sorry if one of your patches was dinged and it should be fixed by this time tomorrow. Thanks, Robert, for working on this. It is highly important for me to know that something is going to break the build before taking it in. However, would it be possible to have a repository where we can review the code of the robot? It is the Jenkins Gerrit Tigger bot direct from jenkins-ci.org that is running the same unit jenkins has been running just against new patches. I think it is important for the robot to be less noisy, and particularly, never give V+1. Already adjusted new runs will give -1 for fail but 0 for success going forward. There were about 400 jobs/patches it gave +1 to if needed those can be re-run please let me know is this is required. This task is reserved to humans that actually know what the patch should be doing. Also, I am not at all sure that the robot is limitting itself to be running code of trustworthy authors. That is correct the reason there were some fails last night was my attempt to add in the limit and I broke things for a few min. I have already tested all the logic and should be in place by the end of today. Regards, Dan. -- Thanks Robert Middleswarth @rmiddle (twitter/IRC) ___ vdsm-devel mailing list vdsm-devel@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-devel
Re: [vdsm] Jenkins and Gerrit.
On Wed, Aug 08, 2012 at 03:48:13PM +0300, Dan Kenigsberg wrote: > On Wed, Aug 08, 2012 at 07:47:02AM -0400, Robert Middleswarth wrote: > > I have setup patch review on Jenkins.info for newly submitted > > patches and it seems to be working pretty well over all but last > > night well tweaking the process I broken it for a few min but that > > was long enough that about 50 jobs were marked -1 I will be fixing > > that today by rerunning the jobs. I am sorry if one of your patches > > was dinged and it should be fixed by this time tomorrow. > > Thanks, Robert, for working on this. It is highly important for me to > know that something is going to break the build before taking it in. > > However, would it be possible to have a repository where we can review > the code of the robot? It's Gerrit Trigger[1] and the code is on github[2]. > I think it is important for the robot to be less noisy, and > particularly, never give V+1. This task is reserved to humans that > actually know what the patch should be doing. The V+1 has been fixed. Will give 0 when they pass, -1 when they fail. > Also, I am not at all sure that the robot is limitting itself to be > running code of trustworthy authors. Eyal added a feature request for this[3]. This was the result of a discussion on the infra mailing list[4]. [1]: https://wiki.jenkins-ci.org/display/JENKINS/Gerrit+Trigger [2]: https://github.com/jenkinsci/gerrit-trigger-plugin [3]: https://issues.jenkins-ci.org/browse/JENKINS-14655 [4]: http://lists.ovirt.org/pipermail/infra/2012-August/000759.html ___ vdsm-devel mailing list vdsm-devel@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-devel
Re: [vdsm] Jenkins and Gerrit.
On Wed, Aug 08, 2012 at 07:47:02AM -0400, Robert Middleswarth wrote: > I have setup patch review on Jenkins.info for newly submitted > patches and it seems to be working pretty well over all but last > night well tweaking the process I broken it for a few min but that > was long enough that about 50 jobs were marked -1 I will be fixing > that today by rerunning the jobs. I am sorry if one of your patches > was dinged and it should be fixed by this time tomorrow. Thanks, Robert, for working on this. It is highly important for me to know that something is going to break the build before taking it in. However, would it be possible to have a repository where we can review the code of the robot? I think it is important for the robot to be less noisy, and particularly, never give V+1. This task is reserved to humans that actually know what the patch should be doing. Also, I am not at all sure that the robot is limitting itself to be running code of trustworthy authors. Regards, Dan. ___ vdsm-devel mailing list vdsm-devel@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-devel