Re: [cross-project-issues-dev] When are Gerrit commits automatically applied?
There are really two scenarios for using Gerrit - at least for me: 1) I know I want to commit that fix but want it to be verified by Hudson first. 2) I or a contributor uploads some code for a real review by another committer. In that case it would be very bad if the change got merged after only Hudson is happy. Dani From: Konstantin Komissarchik konstantin.komissarc...@oracle.com To: 'Cross project issues' cross-project-issues-dev@eclipse.org Date: 21.05.2015 00:36 Subject:Re: [cross-project-issues-dev] When are Gerrit commits automatically applied? Sent by:cross-project-issues-dev-boun...@eclipse.org What's the value in encouraging the developer to wait until verification is done? Either, it's critical that the contribution goes in, in which case the developer will wait anyway, or it's not critical and any failures can be dealt with tomorrow. - Konstantin PS: Personally, I'd like to see gerrit/hudson style automated verification combined with bullet-proof aggregation process that is not susceptible to contributed repositories changing from underneath it. -Original Message- From: cross-project-issues-dev-boun...@eclipse.org [mailto:cross-project-issues-dev-boun...@eclipse.org] On Behalf Of Stephan Herrmann Sent: Wednesday, May 20, 2015 3:23 PM To: cross-project-issues-dev@eclipse.org Subject: Re: [cross-project-issues-dev] When are Gerrit commits automatically applied? On 05/21/2015 12:11 AM, Konstantin Komissarchik wrote: [...] but once that verification has been done, what's the value of waiting for the developer to click on things? Maybe encouraging the developer to wait until s/he knows whether the job is done or not? :) Disclaimer: as much as I like gerrit, for simrel contributions I still prefer running the validation at home and then push directly .. Call me I'm old-school, if you like. Stephan -Original Message- From: cross-project-issues-dev-boun...@eclipse.org [mailto:cross-project-issues-dev-boun...@eclipse.org] On Behalf Of Stephan Herrmann Sent: Wednesday, May 20, 2015 2:42 PM To: cross-project-issues-dev@eclipse.org Subject: Re: [cross-project-issues-dev] When are Gerrit commits automatically applied? I'm not sure what's the goal here? Enable submit-to-gerrit-and-leave - without awaiting the result from hudson? ;P If you wait for success, what harm is done by having to do two more clicks? my 2c. Stephan On 05/20/2015 07:44 AM, Eike Stepper wrote: Hi, I'm not a Gerrit expert, but a quick search for gerrit auto approve gives a number of hints, for example a Gerrit hook script: http://renier.morales-rodriguez.net/post/49144483266/configuring-gerr i t-to-auto-submit-on-patch-ap Personally I'd also like it if my simrel patch sets would be submitted automatically if Hudson gives a +1, but I'm not sure if that's a security concern. Cheers /Eike http://www.esc-net.de http://thegordian.blogspot.com http://twitter.com/eikestepper Am 18.05.2015 um 19:24 schrieb Mickael Istria: On 05/18/2015 06:43 PM, David M Williams wrote: I don't know the answer to Sven's question ... quoted here ... does anyone else? Am I right if committer of the org.eclipse.simrel.build Git repo must be manually set them as code reviewer if they want to apply the patch to the Git repo? I've hoped that the push is automatically done if the Hudson build is successful and a fast forward merge is possible. I would hope that it would be applied automatically ... but, from my experience it doesn't seem to be. But, I've only done it a few times, and hope a more Gerrit knowledgeable person can answer authoritatively ... as well as make sure the right thing happens, if it takes a new bug for webmasters or others to change some setting. Gerrit requires a manual approval from a Simrel committer to merge a commit. There is no mechanism to automatically merge a review if automated tests have succeeded; and since I believe the paradigm of Gerrit and code-review is really to involve an additional human step (in best case a 3rd-party) in the change, automating merge is probably not something desirable. However, as a committer, you can apply your own patch via Gerrit UI, or push without using a Gerrit review (and automated tests). https://wiki.eclipse.org/index.php?title=Simrel/Contributing_to_Simr e l_Aggregation_Build#Pushing_your_changes HTH -- Mickael Istria Eclipse developer at JBoss, by Red Hat http://www.jboss.org/tools My blog http://mickaelistria.wordpress.com - My Tweets http://twitter.com/mickaelistria ___ cross-project-issues-dev mailing list cross-project-issues-dev@eclipse.org To change your delivery options, retrieve your password, or unsubscribe from this list, visit https://dev.eclipse.org/mailman/listinfo/cross-project-issues-dev ___ cross-project-issues-dev
Re: [cross-project-issues-dev] When are Gerrit commits automatically applied?
I know I am coming very late to the Gerrit usage „party“ in the simrel. So I am now biten by all the obstacles you have to overcome to be able to contribute again. To me the two scenarios below are descriptions of 1. You want to push to Git directly because you know what you are doing (and you have tested the contribution first before you push the commit) 2. An untrusted contributor does something and then we want this to be verified by Hudson (aka tests) and then by a human being Having two level of Gerrit contributions (one that is auto-committed and others which are not) is IMHO process overkill….. I rather want the old style before Gerrit back (but then maybe I am alone in this) and then we use Gerrit for contributions which need to be reviewed. Just my 2 cents christian Von: Daniel Megert daniel_meg...@ch.ibm.commailto:daniel_meg...@ch.ibm.com Antworten an: Cross issues cross-project-issues-dev@eclipse.orgmailto:cross-project-issues-dev@eclipse.org Datum: Donnerstag, 21. Mai 2015 12:23 An: Cross issues cross-project-issues-dev@eclipse.orgmailto:cross-project-issues-dev@eclipse.org Betreff: Re: [cross-project-issues-dev] When are Gerrit commits automatically applied? There are really two scenarios for using Gerrit - at least for me: 1) I know I want to commit that fix but want it to be verified by Hudson first. 2) I or a contributor uploads some code for a real review by another committer. In that case it would be very bad if the change got merged after only Hudson is happy. Dani From:Konstantin Komissarchik konstantin.komissarc...@oracle.commailto:konstantin.komissarc...@oracle.com To:'Cross project issues' cross-project-issues-dev@eclipse.orgmailto:cross-project-issues-dev@eclipse.org Date:21.05.2015 00:36 Subject:Re: [cross-project-issues-dev] When are Gerrit commits automatically applied? Sent by: cross-project-issues-dev-boun...@eclipse.orgmailto:cross-project-issues-dev-boun...@eclipse.org What's the value in encouraging the developer to wait until verification is done? Either, it's critical that the contribution goes in, in which case the developer will wait anyway, or it's not critical and any failures can be dealt with tomorrow. - Konstantin PS: Personally, I'd like to see gerrit/hudson style automated verification combined with bullet-proof aggregation process that is not susceptible to contributed repositories changing from underneath it. -Original Message- From: cross-project-issues-dev-boun...@eclipse.orgmailto:cross-project-issues-dev-boun...@eclipse.org [mailto:cross-project-issues-dev-boun...@eclipse.org] On Behalf Of Stephan Herrmann Sent: Wednesday, May 20, 2015 3:23 PM To: cross-project-issues-dev@eclipse.orgmailto:cross-project-issues-dev@eclipse.org Subject: Re: [cross-project-issues-dev] When are Gerrit commits automatically applied? On 05/21/2015 12:11 AM, Konstantin Komissarchik wrote: [...] but once that verification has been done, what's the value of waiting for the developer to click on things? Maybe encouraging the developer to wait until s/he knows whether the job is done or not? :) Disclaimer: as much as I like gerrit, for simrel contributions I still prefer running the validation at home and then push directly .. Call me I'm old-school, if you like. Stephan -Original Message- From: cross-project-issues-dev-boun...@eclipse.orgmailto:cross-project-issues-dev-boun...@eclipse.org [mailto:cross-project-issues-dev-boun...@eclipse.org] On Behalf Of Stephan Herrmann Sent: Wednesday, May 20, 2015 2:42 PM To: cross-project-issues-dev@eclipse.orgmailto:cross-project-issues-dev@eclipse.org Subject: Re: [cross-project-issues-dev] When are Gerrit commits automatically applied? I'm not sure what's the goal here? Enable submit-to-gerrit-and-leave - without awaiting the result from hudson? ;P If you wait for success, what harm is done by having to do two more clicks? my 2c. Stephan On 05/20/2015 07:44 AM, Eike Stepper wrote: Hi, I'm not a Gerrit expert, but a quick search for gerrit auto approve gives a number of hints, for example a Gerrit hook script: http://renier.morales-rodriguez.net/post/49144483266/configuring-gerr i t-to-auto-submit-on-patch-ap Personally I'd also like it if my simrel patch sets would be submitted automatically if Hudson gives a +1, but I'm not sure if that's a security concern. Cheers /Eike http://www.esc-net.dehttp://www.esc-net.de/ http://thegordian.blogspot.comhttp://thegordian.blogspot.com/ http://twitter.com/eikestepper Am 18.05.2015 um 19:24 schrieb Mickael Istria: On 05/18/2015 06:43 PM, David M Williams wrote: I don't know the answer to Sven's question ... quoted here ... does anyone else? Am I right if committer of the org.eclipse.simrel.build Git repo must be manually set them as code reviewer if they want to apply the patch to the Git repo? I've hoped
Re: [cross-project-issues-dev] When are Gerrit commits automatically applied?
I am sorry for that…. I should have said „previous style“… Still I dont get it why I cant directly push to Git and rather push to Gerrit which immediatelly pushes to Git ? Is that extra route a benefit for anyone. I would prefer a „model“ („style ? Whatever) where committers can push to Git and contributors push to Gerrit. (using different URLs). So currently the two groups push to different refs rather than different URLs. More confusion for me :-) Gerrit comes with frustration for me. I missed RC1 yesterday (for Riena) because of Gerrit and still cant push to the simrel.build. One day we will get this sorted out….. Am 21.05.15 12:57 schrieb Mickael Istria unter mist...@redhat.com: I rather want the old style before Gerrit back (but then maybe I am alone in this) and then we use Gerrit for contributions which need to be reviewed. This is not old-style, it's exactly the current behaviour for most Eclipse projects using Gerrit, including Simrel. Just the URL of push has changed, and if you're a committer push directly to master it gets pushed to master, if you push to refs/for/master, it creates a Gerrit review. https://wiki.eclipse.org/index.php?title=Simrel/Contributing_to_Simrel_Agg regation_Build#Pushing_your_changes ___ cross-project-issues-dev mailing list cross-project-issues-dev@eclipse.org To change your delivery options, retrieve your password, or unsubscribe from this list, visit https://dev.eclipse.org/mailman/listinfo/cross-project-issues-dev - compeople AG Untermainanlage 8 60329 Frankfurt/Main fon: +49 (0) 69 / 27 22 18 0 fax: +49 (0) 69 / 27 22 18 22 web: www.compeople.de Vorstand: Jürgen Wiesmaier Aufsichtsratsvorsitzender: Christian Glanz Sitz der Gesellschaft: Frankfurt/Main Handelsregister Frankfurt HRB 56759 USt-IdNr. DE207665352 - ___ cross-project-issues-dev mailing list cross-project-issues-dev@eclipse.org To change your delivery options, retrieve your password, or unsubscribe from this list, visit https://dev.eclipse.org/mailman/listinfo/cross-project-issues-dev
Re: [cross-project-issues-dev] When are Gerrit commits automatically applied?
I rather want the old style before Gerrit back (but then maybe I am alone in this) and then we use Gerrit for contributions which need to be reviewed. This is not old-style, it's exactly the current behaviour for most Eclipse projects using Gerrit, including Simrel. Just the URL of push has changed, and if you're a committer push directly to master it gets pushed to master, if you push to refs/for/master, it creates a Gerrit review. https://wiki.eclipse.org/index.php?title=Simrel/Contributing_to_Simrel_Aggregation_Build#Pushing_your_changes ___ cross-project-issues-dev mailing list cross-project-issues-dev@eclipse.org To change your delivery options, retrieve your password, or unsubscribe from this list, visit https://dev.eclipse.org/mailman/listinfo/cross-project-issues-dev
Re: [cross-project-issues-dev] When are Gerrit commits automatically applied?
On 05/21/2015 07:09 AM, Christian Campo wrote: Still I dont get it why I cant directly push to Git and rather push to Gerrit which immediatelly pushes to Git ? Gerrit == Git One of them is written in C The other is written in Java. They both do the exact same thing. One of them has a bunch of value-added features, such as granular permissions and a code review UI. The other does not. Denis ___ cross-project-issues-dev mailing list cross-project-issues-dev@eclipse.org To change your delivery options, retrieve your password, or unsubscribe from this list, visit https://dev.eclipse.org/mailman/listinfo/cross-project-issues-dev
Re: [cross-project-issues-dev] When are Gerrit commits automatically applied?
I think what people trip over, as I did, is that Gerrit is two independent things: 1- a code review tool 2- a git portal that can be treated as a normal git remote. This one has nothing to do with code reviews and provides the same abilities as direct git (and more). Marc -Original Message- From: Denis Roy [denis@eclipse.org] Received: Thursday, 21 May 2015, 9:26 To: cross-project-issues-dev@eclipse.org [cross-project-issues-dev@eclipse.org] Subject: Re: [cross-project-issues-dev] When are Gerrit commits automatically applied? On 05/21/2015 07:09 AM, Christian Campo wrote: Still I dont get it why I cant directly push to Git and rather push to Gerrit which immediatelly pushes to Git ? Gerrit == Git One of them is written in C The other is written in Java. They both do the exact same thing. One of them has a bunch of value-added features, such as granular permissions and a code review UI. The other does not. Denis ___ cross-project-issues-dev mailing list cross-project-issues-dev@eclipse.org To change your delivery options, retrieve your password, or unsubscribe from this list, visit https://dev.eclipse.org/mailman/listinfo/cross-project-issues-dev ___ cross-project-issues-dev mailing list cross-project-issues-dev@eclipse.org To change your delivery options, retrieve your password, or unsubscribe from this list, visit https://dev.eclipse.org/mailman/listinfo/cross-project-issues-dev
Re: [cross-project-issues-dev] When are Gerrit commits automatically applied?
I'm not sure what's the goal here? Enable submit-to-gerrit-and-leave - without awaiting the result from hudson? ;P If you wait for success, what harm is done by having to do two more clicks? my 2c. Stephan On 05/20/2015 07:44 AM, Eike Stepper wrote: Hi, I'm not a Gerrit expert, but a quick search for gerrit auto approve gives a number of hints, for example a Gerrit hook script: http://renier.morales-rodriguez.net/post/49144483266/configuring-gerrit-to-auto-submit-on-patch-ap Personally I'd also like it if my simrel patch sets would be submitted automatically if Hudson gives a +1, but I'm not sure if that's a security concern. Cheers /Eike http://www.esc-net.de http://thegordian.blogspot.com http://twitter.com/eikestepper Am 18.05.2015 um 19:24 schrieb Mickael Istria: On 05/18/2015 06:43 PM, David M Williams wrote: I don't know the answer to Sven's question ... quoted here ... does anyone else? Am I right if committer of the org.eclipse.simrel.build Git repo must be manually set them as code reviewer if they want to apply the patch to the Git repo? I've hoped that the push is automatically done if the Hudson build is successful and a fast forward merge is possible. I would hope that it would be applied automatically ... but, from my experience it doesn't seem to be. But, I've only done it a few times, and hope a more Gerrit knowledgeable person can answer authoritatively ... as well as make sure the right thing happens, if it takes a new bug for webmasters or others to change some setting. Gerrit requires a manual approval from a Simrel committer to merge a commit. There is no mechanism to automatically merge a review if automated tests have succeeded; and since I believe the paradigm of Gerrit and code-review is really to involve an additional human step (in best case a 3rd-party) in the change, automating merge is probably not something desirable. However, as a committer, you can apply your own patch via Gerrit UI, or push without using a Gerrit review (and automated tests). https://wiki.eclipse.org/index.php?title=Simrel/Contributing_to_Simrel_Aggregation_Build#Pushing_your_changes HTH -- Mickael Istria Eclipse developer at JBoss, by Red Hat http://www.jboss.org/tools My blog http://mickaelistria.wordpress.com - My Tweets http://twitter.com/mickaelistria ___ cross-project-issues-dev mailing list cross-project-issues-dev@eclipse.org To change your delivery options, retrieve your password, or unsubscribe from this list, visit https://dev.eclipse.org/mailman/listinfo/cross-project-issues-dev ___ cross-project-issues-dev mailing list cross-project-issues-dev@eclipse.org To change your delivery options, retrieve your password, or unsubscribe from this list, visit https://dev.eclipse.org/mailman/listinfo/cross-project-issues-dev ___ cross-project-issues-dev mailing list cross-project-issues-dev@eclipse.org To change your delivery options, retrieve your password, or unsubscribe from this list, visit https://dev.eclipse.org/mailman/listinfo/cross-project-issues-dev
Re: [cross-project-issues-dev] When are Gerrit commits automatically applied?
On 05/21/2015 12:11 AM, Konstantin Komissarchik wrote: [...] but once that verification has been done, what's the value of waiting for the developer to click on things? Maybe encouraging the developer to wait until s/he knows whether the job is done or not? :) Disclaimer: as much as I like gerrit, for simrel contributions I still prefer running the validation at home and then push directly .. Call me I'm old-school, if you like. Stephan -Original Message- From: cross-project-issues-dev-boun...@eclipse.org [mailto:cross-project-issues-dev-boun...@eclipse.org] On Behalf Of Stephan Herrmann Sent: Wednesday, May 20, 2015 2:42 PM To: cross-project-issues-dev@eclipse.org Subject: Re: [cross-project-issues-dev] When are Gerrit commits automatically applied? I'm not sure what's the goal here? Enable submit-to-gerrit-and-leave - without awaiting the result from hudson? ;P If you wait for success, what harm is done by having to do two more clicks? my 2c. Stephan On 05/20/2015 07:44 AM, Eike Stepper wrote: Hi, I'm not a Gerrit expert, but a quick search for gerrit auto approve gives a number of hints, for example a Gerrit hook script: http://renier.morales-rodriguez.net/post/49144483266/configuring-gerri t-to-auto-submit-on-patch-ap Personally I'd also like it if my simrel patch sets would be submitted automatically if Hudson gives a +1, but I'm not sure if that's a security concern. Cheers /Eike http://www.esc-net.de http://thegordian.blogspot.com http://twitter.com/eikestepper Am 18.05.2015 um 19:24 schrieb Mickael Istria: On 05/18/2015 06:43 PM, David M Williams wrote: I don't know the answer to Sven's question ... quoted here ... does anyone else? Am I right if committer of the org.eclipse.simrel.build Git repo must be manually set them as code reviewer if they want to apply the patch to the Git repo? I've hoped that the push is automatically done if the Hudson build is successful and a fast forward merge is possible. I would hope that it would be applied automatically ... but, from my experience it doesn't seem to be. But, I've only done it a few times, and hope a more Gerrit knowledgeable person can answer authoritatively ... as well as make sure the right thing happens, if it takes a new bug for webmasters or others to change some setting. Gerrit requires a manual approval from a Simrel committer to merge a commit. There is no mechanism to automatically merge a review if automated tests have succeeded; and since I believe the paradigm of Gerrit and code-review is really to involve an additional human step (in best case a 3rd-party) in the change, automating merge is probably not something desirable. However, as a committer, you can apply your own patch via Gerrit UI, or push without using a Gerrit review (and automated tests). https://wiki.eclipse.org/index.php?title=Simrel/Contributing_to_Simre l_Aggregation_Build#Pushing_your_changes HTH -- Mickael Istria Eclipse developer at JBoss, by Red Hat http://www.jboss.org/tools My blog http://mickaelistria.wordpress.com - My Tweets http://twitter.com/mickaelistria ___ cross-project-issues-dev mailing list cross-project-issues-dev@eclipse.org To change your delivery options, retrieve your password, or unsubscribe from this list, visit https://dev.eclipse.org/mailman/listinfo/cross-project-issues-dev ___ cross-project-issues-dev mailing list cross-project-issues-dev@eclipse.org To change your delivery options, retrieve your password, or unsubscribe from this list, visit https://dev.eclipse.org/mailman/listinfo/cross-project-issues-dev ___ cross-project-issues-dev mailing list cross-project-issues-dev@eclipse.org To change your delivery options, retrieve your password, or unsubscribe from this list, visit https://dev.eclipse.org/mailman/listinfo/cross-project-issues-dev ___ cross-project-issues-dev mailing list cross-project-issues-dev@eclipse.org To change your delivery options, retrieve your password, or unsubscribe from this list, visit https://dev.eclipse.org/mailman/listinfo/cross-project-issues-dev ___ cross-project-issues-dev mailing list cross-project-issues-dev@eclipse.org To change your delivery options, retrieve your password, or unsubscribe from this list, visit https://dev.eclipse.org/mailman/listinfo/cross-project-issues-dev
Re: [cross-project-issues-dev] When are Gerrit commits automatically applied?
Umm... As software guys, our raison d'etre is to rid the world of manual steps wherever we see them. :) If I squint just right, I can sort of see the reason that Hudson is a useful contributor to the process here (see my prior objections), but once that verification has been done, what's the value of waiting for the developer to click on things? - Konstantin -Original Message- From: cross-project-issues-dev-boun...@eclipse.org [mailto:cross-project-issues-dev-boun...@eclipse.org] On Behalf Of Stephan Herrmann Sent: Wednesday, May 20, 2015 2:42 PM To: cross-project-issues-dev@eclipse.org Subject: Re: [cross-project-issues-dev] When are Gerrit commits automatically applied? I'm not sure what's the goal here? Enable submit-to-gerrit-and-leave - without awaiting the result from hudson? ;P If you wait for success, what harm is done by having to do two more clicks? my 2c. Stephan On 05/20/2015 07:44 AM, Eike Stepper wrote: Hi, I'm not a Gerrit expert, but a quick search for gerrit auto approve gives a number of hints, for example a Gerrit hook script: http://renier.morales-rodriguez.net/post/49144483266/configuring-gerri t-to-auto-submit-on-patch-ap Personally I'd also like it if my simrel patch sets would be submitted automatically if Hudson gives a +1, but I'm not sure if that's a security concern. Cheers /Eike http://www.esc-net.de http://thegordian.blogspot.com http://twitter.com/eikestepper Am 18.05.2015 um 19:24 schrieb Mickael Istria: On 05/18/2015 06:43 PM, David M Williams wrote: I don't know the answer to Sven's question ... quoted here ... does anyone else? Am I right if committer of the org.eclipse.simrel.build Git repo must be manually set them as code reviewer if they want to apply the patch to the Git repo? I've hoped that the push is automatically done if the Hudson build is successful and a fast forward merge is possible. I would hope that it would be applied automatically ... but, from my experience it doesn't seem to be. But, I've only done it a few times, and hope a more Gerrit knowledgeable person can answer authoritatively ... as well as make sure the right thing happens, if it takes a new bug for webmasters or others to change some setting. Gerrit requires a manual approval from a Simrel committer to merge a commit. There is no mechanism to automatically merge a review if automated tests have succeeded; and since I believe the paradigm of Gerrit and code-review is really to involve an additional human step (in best case a 3rd-party) in the change, automating merge is probably not something desirable. However, as a committer, you can apply your own patch via Gerrit UI, or push without using a Gerrit review (and automated tests). https://wiki.eclipse.org/index.php?title=Simrel/Contributing_to_Simre l_Aggregation_Build#Pushing_your_changes HTH -- Mickael Istria Eclipse developer at JBoss, by Red Hat http://www.jboss.org/tools My blog http://mickaelistria.wordpress.com - My Tweets http://twitter.com/mickaelistria ___ cross-project-issues-dev mailing list cross-project-issues-dev@eclipse.org To change your delivery options, retrieve your password, or unsubscribe from this list, visit https://dev.eclipse.org/mailman/listinfo/cross-project-issues-dev ___ cross-project-issues-dev mailing list cross-project-issues-dev@eclipse.org To change your delivery options, retrieve your password, or unsubscribe from this list, visit https://dev.eclipse.org/mailman/listinfo/cross-project-issues-dev ___ cross-project-issues-dev mailing list cross-project-issues-dev@eclipse.org To change your delivery options, retrieve your password, or unsubscribe from this list, visit https://dev.eclipse.org/mailman/listinfo/cross-project-issues-dev ___ cross-project-issues-dev mailing list cross-project-issues-dev@eclipse.org To change your delivery options, retrieve your password, or unsubscribe from this list, visit https://dev.eclipse.org/mailman/listinfo/cross-project-issues-dev
Re: [cross-project-issues-dev] When are Gerrit commits automatically applied?
What's the value in encouraging the developer to wait until verification is done? Either, it's critical that the contribution goes in, in which case the developer will wait anyway, or it's not critical and any failures can be dealt with tomorrow. - Konstantin PS: Personally, I'd like to see gerrit/hudson style automated verification combined with bullet-proof aggregation process that is not susceptible to contributed repositories changing from underneath it. -Original Message- From: cross-project-issues-dev-boun...@eclipse.org [mailto:cross-project-issues-dev-boun...@eclipse.org] On Behalf Of Stephan Herrmann Sent: Wednesday, May 20, 2015 3:23 PM To: cross-project-issues-dev@eclipse.org Subject: Re: [cross-project-issues-dev] When are Gerrit commits automatically applied? On 05/21/2015 12:11 AM, Konstantin Komissarchik wrote: [...] but once that verification has been done, what's the value of waiting for the developer to click on things? Maybe encouraging the developer to wait until s/he knows whether the job is done or not? :) Disclaimer: as much as I like gerrit, for simrel contributions I still prefer running the validation at home and then push directly .. Call me I'm old-school, if you like. Stephan -Original Message- From: cross-project-issues-dev-boun...@eclipse.org [mailto:cross-project-issues-dev-boun...@eclipse.org] On Behalf Of Stephan Herrmann Sent: Wednesday, May 20, 2015 2:42 PM To: cross-project-issues-dev@eclipse.org Subject: Re: [cross-project-issues-dev] When are Gerrit commits automatically applied? I'm not sure what's the goal here? Enable submit-to-gerrit-and-leave - without awaiting the result from hudson? ;P If you wait for success, what harm is done by having to do two more clicks? my 2c. Stephan On 05/20/2015 07:44 AM, Eike Stepper wrote: Hi, I'm not a Gerrit expert, but a quick search for gerrit auto approve gives a number of hints, for example a Gerrit hook script: http://renier.morales-rodriguez.net/post/49144483266/configuring-gerr i t-to-auto-submit-on-patch-ap Personally I'd also like it if my simrel patch sets would be submitted automatically if Hudson gives a +1, but I'm not sure if that's a security concern. Cheers /Eike http://www.esc-net.de http://thegordian.blogspot.com http://twitter.com/eikestepper Am 18.05.2015 um 19:24 schrieb Mickael Istria: On 05/18/2015 06:43 PM, David M Williams wrote: I don't know the answer to Sven's question ... quoted here ... does anyone else? Am I right if committer of the org.eclipse.simrel.build Git repo must be manually set them as code reviewer if they want to apply the patch to the Git repo? I've hoped that the push is automatically done if the Hudson build is successful and a fast forward merge is possible. I would hope that it would be applied automatically ... but, from my experience it doesn't seem to be. But, I've only done it a few times, and hope a more Gerrit knowledgeable person can answer authoritatively ... as well as make sure the right thing happens, if it takes a new bug for webmasters or others to change some setting. Gerrit requires a manual approval from a Simrel committer to merge a commit. There is no mechanism to automatically merge a review if automated tests have succeeded; and since I believe the paradigm of Gerrit and code-review is really to involve an additional human step (in best case a 3rd-party) in the change, automating merge is probably not something desirable. However, as a committer, you can apply your own patch via Gerrit UI, or push without using a Gerrit review (and automated tests). https://wiki.eclipse.org/index.php?title=Simrel/Contributing_to_Simr e l_Aggregation_Build#Pushing_your_changes HTH -- Mickael Istria Eclipse developer at JBoss, by Red Hat http://www.jboss.org/tools My blog http://mickaelistria.wordpress.com - My Tweets http://twitter.com/mickaelistria ___ cross-project-issues-dev mailing list cross-project-issues-dev@eclipse.org To change your delivery options, retrieve your password, or unsubscribe from this list, visit https://dev.eclipse.org/mailman/listinfo/cross-project-issues-dev ___ cross-project-issues-dev mailing list cross-project-issues-dev@eclipse.org To change your delivery options, retrieve your password, or unsubscribe from this list, visit https://dev.eclipse.org/mailman/listinfo/cross-project-issues-dev ___ cross-project-issues-dev mailing list cross-project-issues-dev@eclipse.org To change your delivery options, retrieve your password, or unsubscribe from this list, visit https://dev.eclipse.org/mailman/listinfo/cross-project-issues-dev ___ cross-project-issues-dev mailing list cross-project-issues-dev@eclipse.org To change your
Re: [cross-project-issues-dev] When are Gerrit commits automatically applied?
On 05/18/2015 06:43 PM, David M Williams wrote: I don't know the answer to Sven's question ... quoted here ... does anyone else? Am I right if committer of the org.eclipse.simrel.build Git repo must be manually set them as code reviewer if they want to apply the patch to the Git repo? I've hoped that the push is automatically done if the Hudson build is successful and a fast forward merge is possible. I would hope that it would be applied automatically ... but, from my experience it doesn't seem to be. But, I've only done it a few times, and hope a more Gerrit knowledgeable person can answer authoritatively ... as well as make sure the right thing happens, if it takes a new bug for webmasters or others to change some setting. Gerrit requires a manual approval from a Simrel committer to merge a commit. There is no mechanism to automatically merge a review if automated tests have succeeded; and since I believe the paradigm of Gerrit and code-review is really to involve an additional human step (in best case a 3rd-party) in the change, automating merge is probably not something desirable. However, as a committer, you can apply your own patch via Gerrit UI, or push without using a Gerrit review (and automated tests). https://wiki.eclipse.org/index.php?title=Simrel/Contributing_to_Simrel_Aggregation_Build#Pushing_your_changes HTH -- Mickael Istria Eclipse developer at JBoss, by Red Hat http://www.jboss.org/tools My blog http://mickaelistria.wordpress.com - My Tweets http://twitter.com/mickaelistria ___ cross-project-issues-dev mailing list cross-project-issues-dev@eclipse.org To change your delivery options, retrieve your password, or unsubscribe from this list, visit https://dev.eclipse.org/mailman/listinfo/cross-project-issues-dev