Fellows,
I'm trying to push a branch deletion and the ASF Git tells me that branch deletion is prohibited. Has someone changed something? [raul@~/Workbench/Source/ignite$] git push -f origin :ignite-1790 remote: error: denying ref deletion for refs/heads/ignite-1790 To https://git-wip-us.apache.org/repos/asf/ignite ! [remote rejected] ignite-1790 (deletion prohibited) error: failed to push some refs to ' https://git-wip-us.apache.org/repos/asf/ignite' Regards, *Raúl Kripalani* PMC & Committer @ Apache Ignite, Apache Camel | Integration, Big Data and Messaging Engineer http://about.me/raulkripalani | http://www.linkedin.com/in/raulkripalani http://blog.raulkr.net | twitter: @raulvk |
Raul,
ASF is currently prohibiting deletion of GIT branches until further notice. Please add your branch to this Wiki page, so we don’t loose track: https://cwiki.apache.org/confluence/display/IGNITE/Git+branches+to+delete Thanks, D. On Wed, Nov 18, 2015 at 10:16 AM, Raul Kripalani <[hidden email]> wrote: > Fellows, > > I'm trying to push a branch deletion and the ASF Git tells me that branch > deletion is prohibited. > > Has someone changed something? > > [raul@~/Workbench/Source/ignite$] git push -f origin :ignite-1790 > remote: error: denying ref deletion for refs/heads/ignite-1790 > To https://git-wip-us.apache.org/repos/asf/ignite > ! [remote rejected] ignite-1790 (deletion prohibited) > error: failed to push some refs to ' > https://git-wip-us.apache.org/repos/asf/ignite' > > Regards, > > *Raúl Kripalani* > PMC & Committer @ Apache Ignite, Apache Camel | Integration, Big Data and > Messaging Engineer > http://about.me/raulkripalani | http://www.linkedin.com/in/raulkripalani > http://blog.raulkr.net | twitter: @raulvk > |
Guys, therefore, let's develop new functionality in personal forks, test on
TC with pull requests and then merge to apache git as described here - https://cwiki.apache.org/confluence/display/IGNITE/How+to+Contribute We should create new branches only if this is really necessary. Thanks! -- Yakov Zhdanov, Director R&D *GridGain Systems* www.gridgain.com 2015-11-18 23:04 GMT+03:00 Dmitriy Setrakyan <[hidden email]>: > Raul, > > ASF is currently prohibiting deletion of GIT branches until further notice. > Please add your branch to this Wiki page, so we don’t loose track: > > https://cwiki.apache.org/confluence/display/IGNITE/Git+branches+to+delete > > Thanks, > D. > > On Wed, Nov 18, 2015 at 10:16 AM, Raul Kripalani <[hidden email]> wrote: > > > Fellows, > > > > I'm trying to push a branch deletion and the ASF Git tells me that branch > > deletion is prohibited. > > > > Has someone changed something? > > > > [raul@~/Workbench/Source/ignite$] git push -f origin :ignite-1790 > > remote: error: denying ref deletion for refs/heads/ignite-1790 > > To https://git-wip-us.apache.org/repos/asf/ignite > > ! [remote rejected] ignite-1790 (deletion prohibited) > > error: failed to push some refs to ' > > https://git-wip-us.apache.org/repos/asf/ignite' > > > > Regards, > > > > *Raúl Kripalani* > > PMC & Committer @ Apache Ignite, Apache Camel | Integration, Big Data and > > Messaging Engineer > > http://about.me/raulkripalani | http://www.linkedin.com/in/raulkripalani > > http://blog.raulkr.net | twitter: @raulvk > > > |
Should we file a ticket with INFRA asking them to delete all the branches
listed on this page: https://cwiki.apache.org/confluence/display/IGNITE/Git+branches+to+delete Is it safe? D. On Thu, Nov 19, 2015 at 12:57 AM, Yakov Zhdanov <[hidden email]> wrote: > Guys, therefore, let's develop new functionality in personal forks, test on > TC with pull requests and then merge to apache git as described here - > https://cwiki.apache.org/confluence/display/IGNITE/How+to+Contribute > > We should create new branches only if this is really necessary. > > Thanks! > -- > Yakov Zhdanov, Director R&D > *GridGain Systems* > www.gridgain.com > > 2015-11-18 23:04 GMT+03:00 Dmitriy Setrakyan <[hidden email]>: > > > Raul, > > > > ASF is currently prohibiting deletion of GIT branches until further > notice. > > Please add your branch to this Wiki page, so we don’t loose track: > > > > > https://cwiki.apache.org/confluence/display/IGNITE/Git+branches+to+delete > > > > Thanks, > > D. > > > > On Wed, Nov 18, 2015 at 10:16 AM, Raul Kripalani <[hidden email]> > wrote: > > > > > Fellows, > > > > > > I'm trying to push a branch deletion and the ASF Git tells me that > branch > > > deletion is prohibited. > > > > > > Has someone changed something? > > > > > > [raul@~/Workbench/Source/ignite$] git push -f origin :ignite-1790 > > > remote: error: denying ref deletion for refs/heads/ignite-1790 > > > To https://git-wip-us.apache.org/repos/asf/ignite > > > ! [remote rejected] ignite-1790 (deletion prohibited) > > > error: failed to push some refs to ' > > > https://git-wip-us.apache.org/repos/asf/ignite' > > > > > > Regards, > > > > > > *Raúl Kripalani* > > > PMC & Committer @ Apache Ignite, Apache Camel | Integration, Big Data > and > > > Messaging Engineer > > > http://about.me/raulkripalani | > http://www.linkedin.com/in/raulkripalani > > > http://blog.raulkr.net | twitter: @raulvk > > > > > > |
In reply to this post by Yakov Zhdanov-2
I disagree.
Code should be in the ASF infra. Pull requests make sense when outsiders want to make contributions. The usage of ASF infra is not a mere formality. Raúl. On 19 Nov 2015 08:57, "Yakov Zhdanov" <[hidden email]> wrote: > Guys, therefore, let's develop new functionality in personal forks, test on > TC with pull requests and then merge to apache git as described here - > https://cwiki.apache.org/confluence/display/IGNITE/How+to+Contribute > > We should create new branches only if this is really necessary. > > Thanks! > -- > Yakov Zhdanov, Director R&D > *GridGain Systems* > www.gridgain.com > > 2015-11-18 23:04 GMT+03:00 Dmitriy Setrakyan <[hidden email]>: > > > Raul, > > > > ASF is currently prohibiting deletion of GIT branches until further > notice. > > Please add your branch to this Wiki page, so we don’t loose track: > > > > > https://cwiki.apache.org/confluence/display/IGNITE/Git+branches+to+delete > > > > Thanks, > > D. > > > > On Wed, Nov 18, 2015 at 10:16 AM, Raul Kripalani <[hidden email]> > wrote: > > > > > Fellows, > > > > > > I'm trying to push a branch deletion and the ASF Git tells me that > branch > > > deletion is prohibited. > > > > > > Has someone changed something? > > > > > > [raul@~/Workbench/Source/ignite$] git push -f origin :ignite-1790 > > > remote: error: denying ref deletion for refs/heads/ignite-1790 > > > To https://git-wip-us.apache.org/repos/asf/ignite > > > ! [remote rejected] ignite-1790 (deletion prohibited) > > > error: failed to push some refs to ' > > > https://git-wip-us.apache.org/repos/asf/ignite' > > > > > > Regards, > > > > > > *Raúl Kripalani* > > > PMC & Committer @ Apache Ignite, Apache Camel | Integration, Big Data > and > > > Messaging Engineer > > > http://about.me/raulkripalani | > http://www.linkedin.com/in/raulkripalani > > > http://blog.raulkr.net | twitter: @raulvk > > > > > > |
In reply to this post by dsetrakyan
Dmitriy,
No, according to a note [1] on the wiki page [2] we need to check on unmerged changes before. Looks like we should add a status column (someone already has proposed it) where we can track is a branch ready to delete or not. [1] *Note. We don’t remove any *unmerged* branches. Even if the ticket was closed, it is possible that we forgot to merge. Before deleting a branch, we take an extra step to verify that the changes are in the master.* [2] https://cwiki.apache.org/confluence/display/IGNITE/Git+branches+to+delete -- Artem -- On Thu, Nov 19, 2015 at 12:04 PM, Dmitriy Setrakyan <[hidden email]> wrote: > Should we file a ticket with INFRA asking them to delete all the branches > listed on this page: > https://cwiki.apache.org/confluence/display/IGNITE/Git+branches+to+delete > > Is it safe? > > D. > > On Thu, Nov 19, 2015 at 12:57 AM, Yakov Zhdanov <[hidden email]> > wrote: > > > Guys, therefore, let's develop new functionality in personal forks, test > on > > TC with pull requests and then merge to apache git as described here - > > https://cwiki.apache.org/confluence/display/IGNITE/How+to+Contribute > > > > We should create new branches only if this is really necessary. > > > > Thanks! > > -- > > Yakov Zhdanov, Director R&D > > *GridGain Systems* > > www.gridgain.com > > > > 2015-11-18 23:04 GMT+03:00 Dmitriy Setrakyan <[hidden email]>: > > > > > Raul, > > > > > > ASF is currently prohibiting deletion of GIT branches until further > > notice. > > > Please add your branch to this Wiki page, so we don’t loose track: > > > > > > > > > https://cwiki.apache.org/confluence/display/IGNITE/Git+branches+to+delete > > > > > > Thanks, > > > D. > > > > > > On Wed, Nov 18, 2015 at 10:16 AM, Raul Kripalani <[hidden email]> > > wrote: > > > > > > > Fellows, > > > > > > > > I'm trying to push a branch deletion and the ASF Git tells me that > > branch > > > > deletion is prohibited. > > > > > > > > Has someone changed something? > > > > > > > > [raul@~/Workbench/Source/ignite$] git push -f origin :ignite-1790 > > > > remote: error: denying ref deletion for refs/heads/ignite-1790 > > > > To https://git-wip-us.apache.org/repos/asf/ignite > > > > ! [remote rejected] ignite-1790 (deletion prohibited) > > > > error: failed to push some refs to ' > > > > https://git-wip-us.apache.org/repos/asf/ignite' > > > > > > > > Regards, > > > > > > > > *Raúl Kripalani* > > > > PMC & Committer @ Apache Ignite, Apache Camel | Integration, Big Data > > and > > > > Messaging Engineer > > > > http://about.me/raulkripalani | > > http://www.linkedin.com/in/raulkripalani > > > > http://blog.raulkr.net | twitter: @raulvk > > > > > > > > > > |
So branch deletion is disabled on purpose?
On 19 Nov 2015 09:10, "Artem Shutak" <[hidden email]> wrote: > Dmitriy, > > No, according to a note [1] on the wiki page [2] we need to check on > unmerged changes before. Looks like we should add a status column (someone > already has proposed it) where we can track is a branch ready to delete or > not. > > [1] *Note. We don’t remove any *unmerged* branches. Even if the ticket was > closed, it is possible that we forgot to merge. Before deleting a branch, > we take an extra step to verify that the changes are in the master.* > [2] > https://cwiki.apache.org/confluence/display/IGNITE/Git+branches+to+delete > > -- Artem -- > > On Thu, Nov 19, 2015 at 12:04 PM, Dmitriy Setrakyan <[hidden email] > > > wrote: > > > Should we file a ticket with INFRA asking them to delete all the branches > > listed on this page: > > > https://cwiki.apache.org/confluence/display/IGNITE/Git+branches+to+delete > > > > Is it safe? > > > > D. > > > > On Thu, Nov 19, 2015 at 12:57 AM, Yakov Zhdanov <[hidden email]> > > wrote: > > > > > Guys, therefore, let's develop new functionality in personal forks, > test > > on > > > TC with pull requests and then merge to apache git as described here - > > > https://cwiki.apache.org/confluence/display/IGNITE/How+to+Contribute > > > > > > We should create new branches only if this is really necessary. > > > > > > Thanks! > > > -- > > > Yakov Zhdanov, Director R&D > > > *GridGain Systems* > > > www.gridgain.com > > > > > > 2015-11-18 23:04 GMT+03:00 Dmitriy Setrakyan <[hidden email]>: > > > > > > > Raul, > > > > > > > > ASF is currently prohibiting deletion of GIT branches until further > > > notice. > > > > Please add your branch to this Wiki page, so we don’t loose track: > > > > > > > > > > > > > > https://cwiki.apache.org/confluence/display/IGNITE/Git+branches+to+delete > > > > > > > > Thanks, > > > > D. > > > > > > > > On Wed, Nov 18, 2015 at 10:16 AM, Raul Kripalani <[hidden email]> > > > wrote: > > > > > > > > > Fellows, > > > > > > > > > > I'm trying to push a branch deletion and the ASF Git tells me that > > > branch > > > > > deletion is prohibited. > > > > > > > > > > Has someone changed something? > > > > > > > > > > [raul@~/Workbench/Source/ignite$] git push -f origin :ignite-1790 > > > > > remote: error: denying ref deletion for refs/heads/ignite-1790 > > > > > To https://git-wip-us.apache.org/repos/asf/ignite > > > > > ! [remote rejected] ignite-1790 (deletion prohibited) > > > > > error: failed to push some refs to ' > > > > > https://git-wip-us.apache.org/repos/asf/ignite' > > > > > > > > > > Regards, > > > > > > > > > > *Raúl Kripalani* > > > > > PMC & Committer @ Apache Ignite, Apache Camel | Integration, Big > Data > > > and > > > > > Messaging Engineer > > > > > http://about.me/raulkripalani | > > > http://www.linkedin.com/in/raulkripalani > > > > > http://blog.raulkr.net | twitter: @raulvk > > > > > > > > > > > > > > > |
In reply to this post by Raul Kripalani-2
Disagree. This means none but committer can contribute.
--Yakov 2015-11-19 12:08 GMT+03:00 Raul Kripalani <[hidden email]>: > I disagree. > > Code should be in the ASF infra. > > Pull requests make sense when outsiders want to make contributions. > > The usage of ASF infra is not a mere formality. > > Raúl. > On 19 Nov 2015 08:57, "Yakov Zhdanov" <[hidden email]> wrote: > > > Guys, therefore, let's develop new functionality in personal forks, test > on > > TC with pull requests and then merge to apache git as described here - > > https://cwiki.apache.org/confluence/display/IGNITE/How+to+Contribute > > > > We should create new branches only if this is really necessary. > > > > Thanks! > > -- > > Yakov Zhdanov, Director R&D > > *GridGain Systems* > > www.gridgain.com > > > > 2015-11-18 23:04 GMT+03:00 Dmitriy Setrakyan <[hidden email]>: > > > > > Raul, > > > > > > ASF is currently prohibiting deletion of GIT branches until further > > notice. > > > Please add your branch to this Wiki page, so we don’t loose track: > > > > > > > > > https://cwiki.apache.org/confluence/display/IGNITE/Git+branches+to+delete > > > > > > Thanks, > > > D. > > > > > > On Wed, Nov 18, 2015 at 10:16 AM, Raul Kripalani <[hidden email]> > > wrote: > > > > > > > Fellows, > > > > > > > > I'm trying to push a branch deletion and the ASF Git tells me that > > branch > > > > deletion is prohibited. > > > > > > > > Has someone changed something? > > > > > > > > [raul@~/Workbench/Source/ignite$] git push -f origin :ignite-1790 > > > > remote: error: denying ref deletion for refs/heads/ignite-1790 > > > > To https://git-wip-us.apache.org/repos/asf/ignite > > > > ! [remote rejected] ignite-1790 (deletion prohibited) > > > > error: failed to push some refs to ' > > > > https://git-wip-us.apache.org/repos/asf/ignite' > > > > > > > > Regards, > > > > > > > > *Raúl Kripalani* > > > > PMC & Committer @ Apache Ignite, Apache Camel | Integration, Big Data > > and > > > > Messaging Engineer > > > > http://about.me/raulkripalani | > > http://www.linkedin.com/in/raulkripalani > > > > http://blog.raulkr.net | twitter: @raulvk > > > > > > > > > > |
As I said: "Pull requests make sense when outsiders want to make
contributions." Committers with write access to ASF Git have no reason to develop in Github. On 19 Nov 2015 09:13, "Yakov Zhdanov" <[hidden email]> wrote: > Disagree. This means none but committer can contribute. > > --Yakov > > 2015-11-19 12:08 GMT+03:00 Raul Kripalani <[hidden email]>: > > > I disagree. > > > > Code should be in the ASF infra. > > > > Pull requests make sense when outsiders want to make contributions. > > > > The usage of ASF infra is not a mere formality. > > > > Raúl. > > On 19 Nov 2015 08:57, "Yakov Zhdanov" <[hidden email]> wrote: > > > > > Guys, therefore, let's develop new functionality in personal forks, > test > > on > > > TC with pull requests and then merge to apache git as described here - > > > https://cwiki.apache.org/confluence/display/IGNITE/How+to+Contribute > > > > > > We should create new branches only if this is really necessary. > > > > > > Thanks! > > > -- > > > Yakov Zhdanov, Director R&D > > > *GridGain Systems* > > > www.gridgain.com > > > > > > 2015-11-18 23:04 GMT+03:00 Dmitriy Setrakyan <[hidden email]>: > > > > > > > Raul, > > > > > > > > ASF is currently prohibiting deletion of GIT branches until further > > > notice. > > > > Please add your branch to this Wiki page, so we don’t loose track: > > > > > > > > > > > > > > https://cwiki.apache.org/confluence/display/IGNITE/Git+branches+to+delete > > > > > > > > Thanks, > > > > D. > > > > > > > > On Wed, Nov 18, 2015 at 10:16 AM, Raul Kripalani <[hidden email]> > > > wrote: > > > > > > > > > Fellows, > > > > > > > > > > I'm trying to push a branch deletion and the ASF Git tells me that > > > branch > > > > > deletion is prohibited. > > > > > > > > > > Has someone changed something? > > > > > > > > > > [raul@~/Workbench/Source/ignite$] git push -f origin :ignite-1790 > > > > > remote: error: denying ref deletion for refs/heads/ignite-1790 > > > > > To https://git-wip-us.apache.org/repos/asf/ignite > > > > > ! [remote rejected] ignite-1790 (deletion prohibited) > > > > > error: failed to push some refs to ' > > > > > https://git-wip-us.apache.org/repos/asf/ignite' > > > > > > > > > > Regards, > > > > > > > > > > *Raúl Kripalani* > > > > > PMC & Committer @ Apache Ignite, Apache Camel | Integration, Big > Data > > > and > > > > > Messaging Engineer > > > > > http://about.me/raulkripalani | > > > http://www.linkedin.com/in/raulkripalani > > > > > http://blog.raulkr.net | twitter: @raulvk > > > > > > > > > > > > > > > |
On Thu, Nov 19, 2015 at 1:19 AM, Raul Kripalani <[hidden email]> wrote:
> As I said: "Pull requests make sense when outsiders want to make > contributions." > > Committers with write access to ASF Git have no reason to develop > in Github. > Raul, given that we cannot delete branches, I think we should very closely look at the GitHub PRs even for committers. Many committers prefer Github PR process instead of submitting patches. > On 19 Nov 2015 09:13, "Yakov Zhdanov" <[hidden email]> wrote: > > > Disagree. This means none but committer can contribute. > > > > --Yakov > > > > 2015-11-19 12:08 GMT+03:00 Raul Kripalani <[hidden email]>: > > > > > I disagree. > > > > > > Code should be in the ASF infra. > > > > > > Pull requests make sense when outsiders want to make contributions. > > > > > > The usage of ASF infra is not a mere formality. > > > > > > Raúl. > > > On 19 Nov 2015 08:57, "Yakov Zhdanov" <[hidden email]> wrote: > > > > > > > Guys, therefore, let's develop new functionality in personal forks, > > test > > > on > > > > TC with pull requests and then merge to apache git as described here > - > > > > https://cwiki.apache.org/confluence/display/IGNITE/How+to+Contribute > > > > > > > > We should create new branches only if this is really necessary. > > > > > > > > Thanks! > > > > -- > > > > Yakov Zhdanov, Director R&D > > > > *GridGain Systems* > > > > www.gridgain.com > > > > > > > > 2015-11-18 23:04 GMT+03:00 Dmitriy Setrakyan <[hidden email] > >: > > > > > > > > > Raul, > > > > > > > > > > ASF is currently prohibiting deletion of GIT branches until further > > > > notice. > > > > > Please add your branch to this Wiki page, so we don’t loose track: > > > > > > > > > > > > > > > > > > > > https://cwiki.apache.org/confluence/display/IGNITE/Git+branches+to+delete > > > > > > > > > > Thanks, > > > > > D. > > > > > > > > > > On Wed, Nov 18, 2015 at 10:16 AM, Raul Kripalani <[hidden email] > > > > > > wrote: > > > > > > > > > > > Fellows, > > > > > > > > > > > > I'm trying to push a branch deletion and the ASF Git tells me > that > > > > branch > > > > > > deletion is prohibited. > > > > > > > > > > > > Has someone changed something? > > > > > > > > > > > > [raul@~/Workbench/Source/ignite$] git push -f origin > :ignite-1790 > > > > > > remote: error: denying ref deletion for refs/heads/ignite-1790 > > > > > > To https://git-wip-us.apache.org/repos/asf/ignite > > > > > > ! [remote rejected] ignite-1790 (deletion prohibited) > > > > > > error: failed to push some refs to ' > > > > > > https://git-wip-us.apache.org/repos/asf/ignite' > > > > > > > > > > > > Regards, > > > > > > > > > > > > *Raúl Kripalani* > > > > > > PMC & Committer @ Apache Ignite, Apache Camel | Integration, Big > > Data > > > > and > > > > > > Messaging Engineer > > > > > > http://about.me/raulkripalani | > > > > http://www.linkedin.com/in/raulkripalani > > > > > > http://blog.raulkr.net | twitter: @raulvk > > > > > > > > > > > > > > > > > > > > > |
In reply to this post by Raul Kripalani-2
But this leads to tons of garbage in repo and abandoned branches, etc.
--Yakov 2015-11-19 12:19 GMT+03:00 Raul Kripalani <[hidden email]>: > As I said: "Pull requests make sense when outsiders want to make > contributions." > > Committers with write access to ASF Git have no reason to develop in > Github. > On 19 Nov 2015 09:13, "Yakov Zhdanov" <[hidden email]> wrote: > > > Disagree. This means none but committer can contribute. > > > > --Yakov > > > > 2015-11-19 12:08 GMT+03:00 Raul Kripalani <[hidden email]>: > > > > > I disagree. > > > > > > Code should be in the ASF infra. > > > > > > Pull requests make sense when outsiders want to make contributions. > > > > > > The usage of ASF infra is not a mere formality. > > > > > > Raúl. > > > On 19 Nov 2015 08:57, "Yakov Zhdanov" <[hidden email]> wrote: > > > > > > > Guys, therefore, let's develop new functionality in personal forks, > > test > > > on > > > > TC with pull requests and then merge to apache git as described here > - > > > > https://cwiki.apache.org/confluence/display/IGNITE/How+to+Contribute > > > > > > > > We should create new branches only if this is really necessary. > > > > > > > > Thanks! > > > > -- > > > > Yakov Zhdanov, Director R&D > > > > *GridGain Systems* > > > > www.gridgain.com > > > > > > > > 2015-11-18 23:04 GMT+03:00 Dmitriy Setrakyan <[hidden email] > >: > > > > > > > > > Raul, > > > > > > > > > > ASF is currently prohibiting deletion of GIT branches until further > > > > notice. > > > > > Please add your branch to this Wiki page, so we don’t loose track: > > > > > > > > > > > > > > > > > > > > https://cwiki.apache.org/confluence/display/IGNITE/Git+branches+to+delete > > > > > > > > > > Thanks, > > > > > D. > > > > > > > > > > On Wed, Nov 18, 2015 at 10:16 AM, Raul Kripalani <[hidden email] > > > > > > wrote: > > > > > > > > > > > Fellows, > > > > > > > > > > > > I'm trying to push a branch deletion and the ASF Git tells me > that > > > > branch > > > > > > deletion is prohibited. > > > > > > > > > > > > Has someone changed something? > > > > > > > > > > > > [raul@~/Workbench/Source/ignite$] git push -f origin > :ignite-1790 > > > > > > remote: error: denying ref deletion for refs/heads/ignite-1790 > > > > > > To https://git-wip-us.apache.org/repos/asf/ignite > > > > > > ! [remote rejected] ignite-1790 (deletion prohibited) > > > > > > error: failed to push some refs to ' > > > > > > https://git-wip-us.apache.org/repos/asf/ignite' > > > > > > > > > > > > Regards, > > > > > > > > > > > > *Raúl Kripalani* > > > > > > PMC & Committer @ Apache Ignite, Apache Camel | Integration, Big > > Data > > > > and > > > > > > Messaging Engineer > > > > > > http://about.me/raulkripalani | > > > > http://www.linkedin.com/in/raulkripalani > > > > > > http://blog.raulkr.net | twitter: @raulvk > > > > > > > > > > > > > > > > > > > > > |
I'd like to add that there is virtually no difference between using a
branch in original repo and a branch in your personal fork on GitHub. Merges and other operations work seamlessly between multiple remotes. Committers don't even have to create PRs (except to run a TC build). Thanks, On Thu, Nov 19, 2015 at 12:28 PM, Yakov Zhdanov <[hidden email]> wrote: > But this leads to tons of garbage in repo and abandoned branches, etc. > > --Yakov > > 2015-11-19 12:19 GMT+03:00 Raul Kripalani <[hidden email]>: > > > As I said: "Pull requests make sense when outsiders want to make > > contributions." > > > > Committers with write access to ASF Git have no reason to develop in > > Github. > > On 19 Nov 2015 09:13, "Yakov Zhdanov" <[hidden email]> wrote: > > > > > Disagree. This means none but committer can contribute. > > > > > > --Yakov > > > > > > 2015-11-19 12:08 GMT+03:00 Raul Kripalani <[hidden email]>: > > > > > > > I disagree. > > > > > > > > Code should be in the ASF infra. > > > > > > > > Pull requests make sense when outsiders want to make contributions. > > > > > > > > The usage of ASF infra is not a mere formality. > > > > > > > > Raúl. > > > > On 19 Nov 2015 08:57, "Yakov Zhdanov" <[hidden email]> wrote: > > > > > > > > > Guys, therefore, let's develop new functionality in personal forks, > > > test > > > > on > > > > > TC with pull requests and then merge to apache git as described > here > > - > > > > > > https://cwiki.apache.org/confluence/display/IGNITE/How+to+Contribute > > > > > > > > > > We should create new branches only if this is really necessary. > > > > > > > > > > Thanks! > > > > > -- > > > > > Yakov Zhdanov, Director R&D > > > > > *GridGain Systems* > > > > > www.gridgain.com > > > > > > > > > > 2015-11-18 23:04 GMT+03:00 Dmitriy Setrakyan < > [hidden email] > > >: > > > > > > > > > > > Raul, > > > > > > > > > > > > ASF is currently prohibiting deletion of GIT branches until > further > > > > > notice. > > > > > > Please add your branch to this Wiki page, so we don’t loose > track: > > > > > > > > > > > > > > > > > > > > > > > > > > > https://cwiki.apache.org/confluence/display/IGNITE/Git+branches+to+delete > > > > > > > > > > > > Thanks, > > > > > > D. > > > > > > > > > > > > On Wed, Nov 18, 2015 at 10:16 AM, Raul Kripalani < > [hidden email] > > > > > > > > wrote: > > > > > > > > > > > > > Fellows, > > > > > > > > > > > > > > I'm trying to push a branch deletion and the ASF Git tells me > > that > > > > > branch > > > > > > > deletion is prohibited. > > > > > > > > > > > > > > Has someone changed something? > > > > > > > > > > > > > > [raul@~/Workbench/Source/ignite$] git push -f origin > > :ignite-1790 > > > > > > > remote: error: denying ref deletion for refs/heads/ignite-1790 > > > > > > > To https://git-wip-us.apache.org/repos/asf/ignite > > > > > > > ! [remote rejected] ignite-1790 (deletion prohibited) > > > > > > > error: failed to push some refs to ' > > > > > > > https://git-wip-us.apache.org/repos/asf/ignite' > > > > > > > > > > > > > > Regards, > > > > > > > > > > > > > > *Raúl Kripalani* > > > > > > > PMC & Committer @ Apache Ignite, Apache Camel | Integration, > Big > > > Data > > > > > and > > > > > > > Messaging Engineer > > > > > > > http://about.me/raulkripalani | > > > > > http://www.linkedin.com/in/raulkripalani > > > > > > > http://blog.raulkr.net | twitter: @raulvk > > > > > > > > > > > > > > > > > > > > > > > > > > > > -- -- Pavel Tupitsyn GridGain Systems, Inc. www.gridgain.com |
In reply to this post by yzhdanov
On Thu, Nov 19, 2015 at 9:28 AM, Yakov Zhdanov <[hidden email]> wrote:
> But this leads to tons of garbage in repo and abandoned branches, etc. > "Let's not do X because our developers are irresponsible and leave branches behind?" -- Really? We have a zillion-pages set of contribution guidelines and we still think our developers are any liberty? Let's just add one more section to Coding guidelines about branch lifecycle, if that's what you're worried about. *Raúl Kripalani* PMC & Committer @ Apache Ignite, Apache Camel | Integration, Big Data and Messaging Engineer http://about.me/raulkripalani | http://www.linkedin.com/in/raulkripalani http://blog.raulkr.net | twitter: @raulvk |
On Thu, Nov 19, 2015 at 1:41 AM, Raul Kripalani <[hidden email]> wrote:
> On Thu, Nov 19, 2015 at 9:28 AM, Yakov Zhdanov <[hidden email]> > wrote: > > > But this leads to tons of garbage in repo and abandoned branches, etc. > > > > "Let's not do X because our developers are irresponsible and leave branches > behind?" -- Really? We have a zillion-pages set of contribution guidelines > and we still think our developers are any liberty? > > Let's just add one more section to Coding guidelines about branch > lifecycle, if that's what you're worried about. > Raul, we physically cannot delete branches. ASF GIT does not allow it. > > *Raúl Kripalani* > PMC & Committer @ Apache Ignite, Apache Camel | Integration, Big Data and > Messaging Engineer > http://about.me/raulkripalani | http://www.linkedin.com/in/raulkripalani > http://blog.raulkr.net | twitter: @raulvk > |
In reply to this post by Pavel Tupitsyn-3
Lads,
It is not clear to me whether branch deletion is prohibited ASF-wide (Dmitriy: "we *cannot* delete branches") or by express project request. I've understood both things from the thread. So let's wait for INFRA to clarify: [1]. Can someone please explain why we resort to Github in the first place? Was it for CI integration purposes? Regards, [1] https://issues.apache.org/jira/servicedesk/agent/INFRA/issue/INFRA-10798 *Raúl Kripalani* PMC & Committer @ Apache Ignite, Apache Camel | Integration, Big Data and Messaging Engineer http://about.me/raulkripalani | http://www.linkedin.com/in/raulkripalani http://blog.raulkr.net | twitter: @raulvk On Thu, Nov 19, 2015 at 9:34 AM, Pavel Tupitsyn <[hidden email]> wrote: > I'd like to add that there is virtually no difference between using a > branch in original repo and a branch in your personal fork on GitHub. > Merges and other operations work seamlessly between multiple remotes. > Committers don't even have to create PRs (except to run a TC build). > > Thanks, > > On Thu, Nov 19, 2015 at 12:28 PM, Yakov Zhdanov <[hidden email]> > wrote: > > > But this leads to tons of garbage in repo and abandoned branches, etc. > > > > --Yakov > > > > 2015-11-19 12:19 GMT+03:00 Raul Kripalani <[hidden email]>: > > > > > As I said: "Pull requests make sense when outsiders want to make > > > contributions." > > > > > > Committers with write access to ASF Git have no reason to develop in > > > Github. > > > On 19 Nov 2015 09:13, "Yakov Zhdanov" <[hidden email]> wrote: > > > > > > > Disagree. This means none but committer can contribute. > > > > > > > > --Yakov > > > > > > > > 2015-11-19 12:08 GMT+03:00 Raul Kripalani <[hidden email]>: > > > > > > > > > I disagree. > > > > > > > > > > Code should be in the ASF infra. > > > > > > > > > > Pull requests make sense when outsiders want to make contributions. > > > > > > > > > > The usage of ASF infra is not a mere formality. > > > > > > > > > > Raúl. > > > > > On 19 Nov 2015 08:57, "Yakov Zhdanov" <[hidden email]> > wrote: > > > > > > > > > > > Guys, therefore, let's develop new functionality in personal > forks, > > > > test > > > > > on > > > > > > TC with pull requests and then merge to apache git as described > > here > > > - > > > > > > > > https://cwiki.apache.org/confluence/display/IGNITE/How+to+Contribute > > > > > > > > > > > > We should create new branches only if this is really necessary. > > > > > > > > > > > > Thanks! > > > > > > -- > > > > > > Yakov Zhdanov, Director R&D > > > > > > *GridGain Systems* > > > > > > www.gridgain.com > > > > > > > > > > > > 2015-11-18 23:04 GMT+03:00 Dmitriy Setrakyan < > > [hidden email] > > > >: > > > > > > > > > > > > > Raul, > > > > > > > > > > > > > > ASF is currently prohibiting deletion of GIT branches until > > further > > > > > > notice. > > > > > > > Please add your branch to this Wiki page, so we don’t loose > > track: > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > https://cwiki.apache.org/confluence/display/IGNITE/Git+branches+to+delete > > > > > > > > > > > > > > Thanks, > > > > > > > D. > > > > > > > > > > > > > > On Wed, Nov 18, 2015 at 10:16 AM, Raul Kripalani < > > [hidden email] > > > > > > > > > > wrote: > > > > > > > > > > > > > > > Fellows, > > > > > > > > > > > > > > > > I'm trying to push a branch deletion and the ASF Git tells me > > > that > > > > > > branch > > > > > > > > deletion is prohibited. > > > > > > > > > > > > > > > > Has someone changed something? > > > > > > > > > > > > > > > > [raul@~/Workbench/Source/ignite$] git push -f origin > > > :ignite-1790 > > > > > > > > remote: error: denying ref deletion for > refs/heads/ignite-1790 > > > > > > > > To https://git-wip-us.apache.org/repos/asf/ignite > > > > > > > > ! [remote rejected] ignite-1790 (deletion prohibited) > > > > > > > > error: failed to push some refs to ' > > > > > > > > https://git-wip-us.apache.org/repos/asf/ignite' > > > > > > > > > > > > > > > > Regards, > > > > > > > > > > > > > > > > *Raúl Kripalani* > > > > > > > > PMC & Committer @ Apache Ignite, Apache Camel | Integration, > > Big > > > > Data > > > > > > and > > > > > > > > Messaging Engineer > > > > > > > > http://about.me/raulkripalani | > > > > > > http://www.linkedin.com/in/raulkripalani > > > > > > > > http://blog.raulkr.net | twitter: @raulvk > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > -- > -- > Pavel Tupitsyn > GridGain Systems, Inc. > www.gridgain.com > |
I agree that there is absolutely no problems of have multiple ways to
provide contributions. If you are contributor, you can: - Provide a patch; - Provide a PR using GitHub mirror. If you are committer, you can: - Provide a patch; - Provide a PR using GitHub mirror; - Use branch in ASF repo and remove it in the end. ASF branches removal is temporary restricted by INFRA. As soon as it is enabled again why not using it? It is the easiest way to provide contributions and review them. On Thu, Nov 19, 2015 at 12:49 PM, Raul Kripalani <[hidden email]> wrote: > Lads, > > It is not clear to me whether branch deletion is prohibited ASF-wide > (Dmitriy: "we *cannot* delete branches") or by express project request. > I've understood both things from the thread. So let's wait for INFRA to > clarify: [1]. > > Can someone please explain why we resort to Github in the first place? Was > it for CI integration purposes? > > Regards, > > [1] > https://issues.apache.org/jira/servicedesk/agent/INFRA/issue/INFRA-10798 > > *Raúl Kripalani* > PMC & Committer @ Apache Ignite, Apache Camel | Integration, Big Data and > Messaging Engineer > http://about.me/raulkripalani | http://www.linkedin.com/in/raulkripalani > http://blog.raulkr.net | twitter: @raulvk > > On Thu, Nov 19, 2015 at 9:34 AM, Pavel Tupitsyn <[hidden email]> > wrote: > > > I'd like to add that there is virtually no difference between using a > > branch in original repo and a branch in your personal fork on GitHub. > > Merges and other operations work seamlessly between multiple remotes. > > Committers don't even have to create PRs (except to run a TC build). > > > > Thanks, > > > > On Thu, Nov 19, 2015 at 12:28 PM, Yakov Zhdanov <[hidden email]> > > wrote: > > > > > But this leads to tons of garbage in repo and abandoned branches, etc. > > > > > > --Yakov > > > > > > 2015-11-19 12:19 GMT+03:00 Raul Kripalani <[hidden email]>: > > > > > > > As I said: "Pull requests make sense when outsiders want to make > > > > contributions." > > > > > > > > Committers with write access to ASF Git have no reason to develop in > > > > Github. > > > > On 19 Nov 2015 09:13, "Yakov Zhdanov" <[hidden email]> wrote: > > > > > > > > > Disagree. This means none but committer can contribute. > > > > > > > > > > --Yakov > > > > > > > > > > 2015-11-19 12:08 GMT+03:00 Raul Kripalani <[hidden email]>: > > > > > > > > > > > I disagree. > > > > > > > > > > > > Code should be in the ASF infra. > > > > > > > > > > > > Pull requests make sense when outsiders want to make > contributions. > > > > > > > > > > > > The usage of ASF infra is not a mere formality. > > > > > > > > > > > > Raúl. > > > > > > On 19 Nov 2015 08:57, "Yakov Zhdanov" <[hidden email]> > > wrote: > > > > > > > > > > > > > Guys, therefore, let's develop new functionality in personal > > forks, > > > > > test > > > > > > on > > > > > > > TC with pull requests and then merge to apache git as described > > > here > > > > - > > > > > > > > > > https://cwiki.apache.org/confluence/display/IGNITE/How+to+Contribute > > > > > > > > > > > > > > We should create new branches only if this is really necessary. > > > > > > > > > > > > > > Thanks! > > > > > > > -- > > > > > > > Yakov Zhdanov, Director R&D > > > > > > > *GridGain Systems* > > > > > > > www.gridgain.com > > > > > > > > > > > > > > 2015-11-18 23:04 GMT+03:00 Dmitriy Setrakyan < > > > [hidden email] > > > > >: > > > > > > > > > > > > > > > Raul, > > > > > > > > > > > > > > > > ASF is currently prohibiting deletion of GIT branches until > > > further > > > > > > > notice. > > > > > > > > Please add your branch to this Wiki page, so we don’t loose > > > track: > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > https://cwiki.apache.org/confluence/display/IGNITE/Git+branches+to+delete > > > > > > > > > > > > > > > > Thanks, > > > > > > > > D. > > > > > > > > > > > > > > > > On Wed, Nov 18, 2015 at 10:16 AM, Raul Kripalani < > > > [hidden email] > > > > > > > > > > > > wrote: > > > > > > > > > > > > > > > > > Fellows, > > > > > > > > > > > > > > > > > > I'm trying to push a branch deletion and the ASF Git tells > me > > > > that > > > > > > > branch > > > > > > > > > deletion is prohibited. > > > > > > > > > > > > > > > > > > Has someone changed something? > > > > > > > > > > > > > > > > > > [raul@~/Workbench/Source/ignite$] git push -f origin > > > > :ignite-1790 > > > > > > > > > remote: error: denying ref deletion for > > refs/heads/ignite-1790 > > > > > > > > > To https://git-wip-us.apache.org/repos/asf/ignite > > > > > > > > > ! [remote rejected] ignite-1790 (deletion prohibited) > > > > > > > > > error: failed to push some refs to ' > > > > > > > > > https://git-wip-us.apache.org/repos/asf/ignite' > > > > > > > > > > > > > > > > > > Regards, > > > > > > > > > > > > > > > > > > *Raúl Kripalani* > > > > > > > > > PMC & Committer @ Apache Ignite, Apache Camel | > Integration, > > > Big > > > > > Data > > > > > > > and > > > > > > > > > Messaging Engineer > > > > > > > > > http://about.me/raulkripalani | > > > > > > > http://www.linkedin.com/in/raulkripalani > > > > > > > > > http://blog.raulkr.net | twitter: @raulvk > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > -- > > -- > > Pavel Tupitsyn > > GridGain Systems, Inc. > > www.gridgain.com > > > |
Exactly. Let's wait for Infra's feedback on why that restriction and when
it'll be lifted. |
In reply to this post by Vladimir Ozerov
+1
On Thu, Nov 19, 2015 at 1:07 PM, Vladimir Ozerov <[hidden email]> wrote: > I agree that there is absolutely no problems of have multiple ways to > provide contributions. > > If you are contributor, you can: > - Provide a patch; > - Provide a PR using GitHub mirror. > > If you are committer, you can: > - Provide a patch; > - Provide a PR using GitHub mirror; > - Use branch in ASF repo and remove it in the end. > > ASF branches removal is temporary restricted by INFRA. As soon as it is > enabled again why not using it? It is the easiest way to provide > contributions and review them. > > On Thu, Nov 19, 2015 at 12:49 PM, Raul Kripalani <[hidden email]> wrote: > > > Lads, > > > > It is not clear to me whether branch deletion is prohibited ASF-wide > > (Dmitriy: "we *cannot* delete branches") or by express project request. > > I've understood both things from the thread. So let's wait for INFRA to > > clarify: [1]. > > > > Can someone please explain why we resort to Github in the first place? > Was > > it for CI integration purposes? > > > > Regards, > > > > [1] > > https://issues.apache.org/jira/servicedesk/agent/INFRA/issue/INFRA-10798 > > > > *Raúl Kripalani* > > PMC & Committer @ Apache Ignite, Apache Camel | Integration, Big Data and > > Messaging Engineer > > http://about.me/raulkripalani | http://www.linkedin.com/in/raulkripalani > > http://blog.raulkr.net | twitter: @raulvk > > > > On Thu, Nov 19, 2015 at 9:34 AM, Pavel Tupitsyn <[hidden email]> > > wrote: > > > > > I'd like to add that there is virtually no difference between using a > > > branch in original repo and a branch in your personal fork on GitHub. > > > Merges and other operations work seamlessly between multiple remotes. > > > Committers don't even have to create PRs (except to run a TC build). > > > > > > Thanks, > > > > > > On Thu, Nov 19, 2015 at 12:28 PM, Yakov Zhdanov <[hidden email]> > > > wrote: > > > > > > > But this leads to tons of garbage in repo and abandoned branches, > etc. > > > > > > > > --Yakov > > > > > > > > 2015-11-19 12:19 GMT+03:00 Raul Kripalani <[hidden email]>: > > > > > > > > > As I said: "Pull requests make sense when outsiders want to make > > > > > contributions." > > > > > > > > > > Committers with write access to ASF Git have no reason to develop > in > > > > > Github. > > > > > On 19 Nov 2015 09:13, "Yakov Zhdanov" <[hidden email]> wrote: > > > > > > > > > > > Disagree. This means none but committer can contribute. > > > > > > > > > > > > --Yakov > > > > > > > > > > > > 2015-11-19 12:08 GMT+03:00 Raul Kripalani <[hidden email]>: > > > > > > > > > > > > > I disagree. > > > > > > > > > > > > > > Code should be in the ASF infra. > > > > > > > > > > > > > > Pull requests make sense when outsiders want to make > > contributions. > > > > > > > > > > > > > > The usage of ASF infra is not a mere formality. > > > > > > > > > > > > > > Raúl. > > > > > > > On 19 Nov 2015 08:57, "Yakov Zhdanov" <[hidden email]> > > > wrote: > > > > > > > > > > > > > > > Guys, therefore, let's develop new functionality in personal > > > forks, > > > > > > test > > > > > > > on > > > > > > > > TC with pull requests and then merge to apache git as > described > > > > here > > > > > - > > > > > > > > > > > > https://cwiki.apache.org/confluence/display/IGNITE/How+to+Contribute > > > > > > > > > > > > > > > > We should create new branches only if this is really > necessary. > > > > > > > > > > > > > > > > Thanks! > > > > > > > > -- > > > > > > > > Yakov Zhdanov, Director R&D > > > > > > > > *GridGain Systems* > > > > > > > > www.gridgain.com > > > > > > > > > > > > > > > > 2015-11-18 23:04 GMT+03:00 Dmitriy Setrakyan < > > > > [hidden email] > > > > > >: > > > > > > > > > > > > > > > > > Raul, > > > > > > > > > > > > > > > > > > ASF is currently prohibiting deletion of GIT branches until > > > > further > > > > > > > > notice. > > > > > > > > > Please add your branch to this Wiki page, so we don’t loose > > > > track: > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > https://cwiki.apache.org/confluence/display/IGNITE/Git+branches+to+delete > > > > > > > > > > > > > > > > > > Thanks, > > > > > > > > > D. > > > > > > > > > > > > > > > > > > On Wed, Nov 18, 2015 at 10:16 AM, Raul Kripalani < > > > > [hidden email] > > > > > > > > > > > > > > wrote: > > > > > > > > > > > > > > > > > > > Fellows, > > > > > > > > > > > > > > > > > > > > I'm trying to push a branch deletion and the ASF Git > tells > > me > > > > > that > > > > > > > > branch > > > > > > > > > > deletion is prohibited. > > > > > > > > > > > > > > > > > > > > Has someone changed something? > > > > > > > > > > > > > > > > > > > > [raul@~/Workbench/Source/ignite$] git push -f origin > > > > > :ignite-1790 > > > > > > > > > > remote: error: denying ref deletion for > > > refs/heads/ignite-1790 > > > > > > > > > > To https://git-wip-us.apache.org/repos/asf/ignite > > > > > > > > > > ! [remote rejected] ignite-1790 (deletion prohibited) > > > > > > > > > > error: failed to push some refs to ' > > > > > > > > > > https://git-wip-us.apache.org/repos/asf/ignite' > > > > > > > > > > > > > > > > > > > > Regards, > > > > > > > > > > > > > > > > > > > > *Raúl Kripalani* > > > > > > > > > > PMC & Committer @ Apache Ignite, Apache Camel | > > Integration, > > > > Big > > > > > > Data > > > > > > > > and > > > > > > > > > > Messaging Engineer > > > > > > > > > > http://about.me/raulkripalani | > > > > > > > > http://www.linkedin.com/in/raulkripalani > > > > > > > > > > http://blog.raulkr.net | twitter: @raulvk > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > -- > > > -- > > > Pavel Tupitsyn > > > GridGain Systems, Inc. > > > www.gridgain.com > > > > > > |
Ok. So Infra replied and referred to an email on November 3rd which I
somehow missed, indicating that it is indeed an ASF-wide temporary restriction. +1 to liberty of options to contribute/commit and to temporarily keeping track of branches to delete in the Wiki. Regards, *Raúl Kripalani* PMC & Committer @ Apache Ignite, Apache Camel | Integration, Big Data and Messaging Engineer http://about.me/raulkripalani | http://www.linkedin.com/in/raulkripalani http://blog.raulkr.net | twitter: @raulvk On Thu, Nov 19, 2015 at 10:21 AM, Semyon Boikov <[hidden email]> wrote: > +1 > > On Thu, Nov 19, 2015 at 1:07 PM, Vladimir Ozerov <[hidden email]> > wrote: > > > I agree that there is absolutely no problems of have multiple ways to > > provide contributions. > > > > If you are contributor, you can: > > - Provide a patch; > > - Provide a PR using GitHub mirror. > > > > If you are committer, you can: > > - Provide a patch; > > - Provide a PR using GitHub mirror; > > - Use branch in ASF repo and remove it in the end. > > > > ASF branches removal is temporary restricted by INFRA. As soon as it is > > enabled again why not using it? It is the easiest way to provide > > contributions and review them. > > > > On Thu, Nov 19, 2015 at 12:49 PM, Raul Kripalani <[hidden email]> > wrote: > > > > > Lads, > > > > > > It is not clear to me whether branch deletion is prohibited ASF-wide > > > (Dmitriy: "we *cannot* delete branches") or by express project request. > > > I've understood both things from the thread. So let's wait for INFRA to > > > clarify: [1]. > > > > > > Can someone please explain why we resort to Github in the first place? > > Was > > > it for CI integration purposes? > > > > > > Regards, > > > > > > [1] > > > > https://issues.apache.org/jira/servicedesk/agent/INFRA/issue/INFRA-10798 > > > > > > *Raúl Kripalani* > > > PMC & Committer @ Apache Ignite, Apache Camel | Integration, Big Data > and > > > Messaging Engineer > > > http://about.me/raulkripalani | > http://www.linkedin.com/in/raulkripalani > > > http://blog.raulkr.net | twitter: @raulvk > > > > > > On Thu, Nov 19, 2015 at 9:34 AM, Pavel Tupitsyn < > [hidden email]> > > > wrote: > > > > > > > I'd like to add that there is virtually no difference between using a > > > > branch in original repo and a branch in your personal fork on GitHub. > > > > Merges and other operations work seamlessly between multiple remotes. > > > > Committers don't even have to create PRs (except to run a TC build). > > > > > > > > Thanks, > > > > > > > > On Thu, Nov 19, 2015 at 12:28 PM, Yakov Zhdanov <[hidden email] > > > > > > wrote: > > > > > > > > > But this leads to tons of garbage in repo and abandoned branches, > > etc. > > > > > > > > > > --Yakov > > > > > > > > > > 2015-11-19 12:19 GMT+03:00 Raul Kripalani <[hidden email]>: > > > > > > > > > > > As I said: "Pull requests make sense when outsiders want to make > > > > > > contributions." > > > > > > > > > > > > Committers with write access to ASF Git have no reason to develop > > in > > > > > > Github. > > > > > > On 19 Nov 2015 09:13, "Yakov Zhdanov" <[hidden email]> > wrote: > > > > > > > > > > > > > Disagree. This means none but committer can contribute. > > > > > > > > > > > > > > --Yakov > > > > > > > > > > > > > > 2015-11-19 12:08 GMT+03:00 Raul Kripalani <[hidden email]>: > > > > > > > > > > > > > > > I disagree. > > > > > > > > > > > > > > > > Code should be in the ASF infra. > > > > > > > > > > > > > > > > Pull requests make sense when outsiders want to make > > > contributions. > > > > > > > > > > > > > > > > The usage of ASF infra is not a mere formality. > > > > > > > > > > > > > > > > Raúl. > > > > > > > > On 19 Nov 2015 08:57, "Yakov Zhdanov" <[hidden email] > > > > > > wrote: > > > > > > > > > > > > > > > > > Guys, therefore, let's develop new functionality in > personal > > > > forks, > > > > > > > test > > > > > > > > on > > > > > > > > > TC with pull requests and then merge to apache git as > > described > > > > > here > > > > > > - > > > > > > > > > > > > > > > https://cwiki.apache.org/confluence/display/IGNITE/How+to+Contribute > > > > > > > > > > > > > > > > > > We should create new branches only if this is really > > necessary. > > > > > > > > > > > > > > > > > > Thanks! > > > > > > > > > -- > > > > > > > > > Yakov Zhdanov, Director R&D > > > > > > > > > *GridGain Systems* > > > > > > > > > www.gridgain.com > > > > > > > > > > > > > > > > > > 2015-11-18 23:04 GMT+03:00 Dmitriy Setrakyan < > > > > > [hidden email] > > > > > > >: > > > > > > > > > > > > > > > > > > > Raul, > > > > > > > > > > > > > > > > > > > > ASF is currently prohibiting deletion of GIT branches > until > > > > > further > > > > > > > > > notice. > > > > > > > > > > Please add your branch to this Wiki page, so we don’t > loose > > > > > track: > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > https://cwiki.apache.org/confluence/display/IGNITE/Git+branches+to+delete > > > > > > > > > > > > > > > > > > > > Thanks, > > > > > > > > > > D. > > > > > > > > > > > > > > > > > > > > On Wed, Nov 18, 2015 at 10:16 AM, Raul Kripalani < > > > > > [hidden email] > > > > > > > > > > > > > > > > wrote: > > > > > > > > > > > > > > > > > > > > > Fellows, > > > > > > > > > > > > > > > > > > > > > > I'm trying to push a branch deletion and the ASF Git > > tells > > > me > > > > > > that > > > > > > > > > branch > > > > > > > > > > > deletion is prohibited. > > > > > > > > > > > > > > > > > > > > > > Has someone changed something? > > > > > > > > > > > > > > > > > > > > > > [raul@~/Workbench/Source/ignite$] git push -f origin > > > > > > :ignite-1790 > > > > > > > > > > > remote: error: denying ref deletion for > > > > refs/heads/ignite-1790 > > > > > > > > > > > To https://git-wip-us.apache.org/repos/asf/ignite > > > > > > > > > > > ! [remote rejected] ignite-1790 (deletion prohibited) > > > > > > > > > > > error: failed to push some refs to ' > > > > > > > > > > > https://git-wip-us.apache.org/repos/asf/ignite' > > > > > > > > > > > > > > > > > > > > > > Regards, > > > > > > > > > > > > > > > > > > > > > > *Raúl Kripalani* > > > > > > > > > > > PMC & Committer @ Apache Ignite, Apache Camel | > > > Integration, > > > > > Big > > > > > > > Data > > > > > > > > > and > > > > > > > > > > > Messaging Engineer > > > > > > > > > > > http://about.me/raulkripalani | > > > > > > > > > http://www.linkedin.com/in/raulkripalani > > > > > > > > > > > http://blog.raulkr.net | twitter: @raulvk > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > -- > > > > -- > > > > Pavel Tupitsyn > > > > GridGain Systems, Inc. > > > > www.gridgain.com > > > > > > > > > > |
Raul
is there are a ticket for this INFRA ticket? Sorry if I missed it... I regurgitate about this no-branch-delete policy imposed by a myopic "policy" considerations. And I want to increase the pressure on board@ for this sort of things. Would appreciate the pointer if exist. Thanks! Cos On Thu, Nov 19, 2015 at 11:47AM, Raul Kripalani wrote: > Ok. So Infra replied and referred to an email on November 3rd which I > somehow missed, indicating that it is indeed an ASF-wide temporary > restriction. > > +1 to liberty of options to contribute/commit and to temporarily keeping > track of branches to delete in the Wiki. > > Regards, > > *Raúl Kripalani* > PMC & Committer @ Apache Ignite, Apache Camel | Integration, Big Data and > Messaging Engineer > http://about.me/raulkripalani | http://www.linkedin.com/in/raulkripalani > http://blog.raulkr.net | twitter: @raulvk > > On Thu, Nov 19, 2015 at 10:21 AM, Semyon Boikov <[hidden email]> > wrote: > > > +1 > > > > On Thu, Nov 19, 2015 at 1:07 PM, Vladimir Ozerov <[hidden email]> > > wrote: > > > > > I agree that there is absolutely no problems of have multiple ways to > > > provide contributions. > > > > > > If you are contributor, you can: > > > - Provide a patch; > > > - Provide a PR using GitHub mirror. > > > > > > If you are committer, you can: > > > - Provide a patch; > > > - Provide a PR using GitHub mirror; > > > - Use branch in ASF repo and remove it in the end. > > > > > > ASF branches removal is temporary restricted by INFRA. As soon as it is > > > enabled again why not using it? It is the easiest way to provide > > > contributions and review them. > > > > > > On Thu, Nov 19, 2015 at 12:49 PM, Raul Kripalani <[hidden email]> > > wrote: > > > > > > > Lads, > > > > > > > > It is not clear to me whether branch deletion is prohibited ASF-wide > > > > (Dmitriy: "we *cannot* delete branches") or by express project request. > > > > I've understood both things from the thread. So let's wait for INFRA to > > > > clarify: [1]. > > > > > > > > Can someone please explain why we resort to Github in the first place? > > > Was > > > > it for CI integration purposes? > > > > > > > > Regards, > > > > > > > > [1] > > > > > > https://issues.apache.org/jira/servicedesk/agent/INFRA/issue/INFRA-10798 > > > > > > > > *Raúl Kripalani* > > > > PMC & Committer @ Apache Ignite, Apache Camel | Integration, Big Data > > and > > > > Messaging Engineer > > > > http://about.me/raulkripalani | > > http://www.linkedin.com/in/raulkripalani > > > > http://blog.raulkr.net | twitter: @raulvk > > > > > > > > On Thu, Nov 19, 2015 at 9:34 AM, Pavel Tupitsyn < > > [hidden email]> > > > > wrote: > > > > > > > > > I'd like to add that there is virtually no difference between using a > > > > > branch in original repo and a branch in your personal fork on GitHub. > > > > > Merges and other operations work seamlessly between multiple remotes. > > > > > Committers don't even have to create PRs (except to run a TC build). > > > > > > > > > > Thanks, > > > > > > > > > > On Thu, Nov 19, 2015 at 12:28 PM, Yakov Zhdanov <[hidden email] > > > > > > > > wrote: > > > > > > > > > > > But this leads to tons of garbage in repo and abandoned branches, > > > etc. > > > > > > > > > > > > --Yakov > > > > > > > > > > > > 2015-11-19 12:19 GMT+03:00 Raul Kripalani <[hidden email]>: > > > > > > > > > > > > > As I said: "Pull requests make sense when outsiders want to make > > > > > > > contributions." > > > > > > > > > > > > > > Committers with write access to ASF Git have no reason to develop > > > in > > > > > > > Github. > > > > > > > On 19 Nov 2015 09:13, "Yakov Zhdanov" <[hidden email]> > > wrote: > > > > > > > > > > > > > > > Disagree. This means none but committer can contribute. > > > > > > > > > > > > > > > > --Yakov > > > > > > > > > > > > > > > > 2015-11-19 12:08 GMT+03:00 Raul Kripalani <[hidden email]>: > > > > > > > > > > > > > > > > > I disagree. > > > > > > > > > > > > > > > > > > Code should be in the ASF infra. > > > > > > > > > > > > > > > > > > Pull requests make sense when outsiders want to make > > > > contributions. > > > > > > > > > > > > > > > > > > The usage of ASF infra is not a mere formality. > > > > > > > > > > > > > > > > > > Raúl. > > > > > > > > > On 19 Nov 2015 08:57, "Yakov Zhdanov" <[hidden email] > > > > > > > > wrote: > > > > > > > > > > > > > > > > > > > Guys, therefore, let's develop new functionality in > > personal > > > > > forks, > > > > > > > > test > > > > > > > > > on > > > > > > > > > > TC with pull requests and then merge to apache git as > > > described > > > > > > here > > > > > > > - > > > > > > > > > > > > > > > > > > https://cwiki.apache.org/confluence/display/IGNITE/How+to+Contribute > > > > > > > > > > > > > > > > > > > > We should create new branches only if this is really > > > necessary. > > > > > > > > > > > > > > > > > > > > Thanks! > > > > > > > > > > -- > > > > > > > > > > Yakov Zhdanov, Director R&D > > > > > > > > > > *GridGain Systems* > > > > > > > > > > www.gridgain.com > > > > > > > > > > > > > > > > > > > > 2015-11-18 23:04 GMT+03:00 Dmitriy Setrakyan < > > > > > > [hidden email] > > > > > > > >: > > > > > > > > > > > > > > > > > > > > > Raul, > > > > > > > > > > > > > > > > > > > > > > ASF is currently prohibiting deletion of GIT branches > > until > > > > > > further > > > > > > > > > > notice. > > > > > > > > > > > Please add your branch to this Wiki page, so we don’t > > loose > > > > > > track: > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > https://cwiki.apache.org/confluence/display/IGNITE/Git+branches+to+delete > > > > > > > > > > > > > > > > > > > > > > Thanks, > > > > > > > > > > > D. > > > > > > > > > > > > > > > > > > > > > > On Wed, Nov 18, 2015 at 10:16 AM, Raul Kripalani < > > > > > > [hidden email] > > > > > > > > > > > > > > > > > > wrote: > > > > > > > > > > > > > > > > > > > > > > > Fellows, > > > > > > > > > > > > > > > > > > > > > > > > I'm trying to push a branch deletion and the ASF Git > > > tells > > > > me > > > > > > > that > > > > > > > > > > branch > > > > > > > > > > > > deletion is prohibited. > > > > > > > > > > > > > > > > > > > > > > > > Has someone changed something? > > > > > > > > > > > > > > > > > > > > > > > > [raul@~/Workbench/Source/ignite$] git push -f origin > > > > > > > :ignite-1790 > > > > > > > > > > > > remote: error: denying ref deletion for > > > > > refs/heads/ignite-1790 > > > > > > > > > > > > To https://git-wip-us.apache.org/repos/asf/ignite > > > > > > > > > > > > ! [remote rejected] ignite-1790 (deletion prohibited) > > > > > > > > > > > > error: failed to push some refs to ' > > > > > > > > > > > > https://git-wip-us.apache.org/repos/asf/ignite' > > > > > > > > > > > > > > > > > > > > > > > > Regards, > > > > > > > > > > > > > > > > > > > > > > > > *Raúl Kripalani* > > > > > > > > > > > > PMC & Committer @ Apache Ignite, Apache Camel | > > > > Integration, > > > > > > Big > > > > > > > > Data > > > > > > > > > > and > > > > > > > > > > > > Messaging Engineer > > > > > > > > > > > > http://about.me/raulkripalani | > > > > > > > > > > http://www.linkedin.com/in/raulkripalani > > > > > > > > > > > > http://blog.raulkr.net | twitter: @raulvk > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > -- > > > > > -- > > > > > Pavel Tupitsyn > > > > > GridGain Systems, Inc. > > > > > www.gridgain.com > > > > > > > > > > > > > > |
Free forum by Nabble | Edit this page |