Apache Ignite Developers - Legacy Mail Archive

Branch deletion prohibited

Classic

List

Threaded

31 messages Options

Raul Kripalani

Branch deletion prohibited

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

dsetrakyan

Re: Branch deletion prohibited

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
>

Yakov Zhdanov-2

Re: Branch deletion prohibited

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
> >
>

dsetrakyan

Re: Branch deletion prohibited

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
> > >
> >
>

Raul Kripalani-2

Re: Branch deletion prohibited

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:

Artem Shutak

Re: Branch deletion prohibited

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
> > > >
> > >
> >
>

Raul Kripalani

Re: Branch deletion prohibited

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
> > > > >
> > > >
> > >
> >
>

yzhdanov

Re: Branch deletion prohibited

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
> > > >
> > >
> >
>

Raul Kripalani-2

Re: Branch deletion prohibited

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
> > > > >
> > > >
> > >
> >
>

dsetrakyan

Re: Branch deletion prohibited

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
> > > > > >
> > > > >
> > > >
> > >
> >
>

yzhdanov

Re: Branch deletion prohibited

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
> > > > > >
> > > > >
> > > >
> > >
> >
>

Pavel Tupitsyn-3

Re: Branch deletion prohibited

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 Kripalani

Re: Branch deletion prohibited

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

dsetrakyan

Re: Branch deletion prohibited

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
>

Raul Kripalani

Re: Branch deletion prohibited

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
>

Vladimir Ozerov

Re: Branch deletion prohibited

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 Kripalani

Re: Branch deletion prohibited

Exactly. Let's wait for Infra's feedback on why that restriction and when
it'll be lifted.

Semyon Boikov

Re: Branch deletion prohibited

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
> > >
> >
>

Raul Kripalani

Re: Branch deletion prohibited

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
> > > >
> > >
> >
>

Konstantin Boudnik-2

Re: Branch deletion prohibited

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
> > > > >
> > > >
> > >
> >

signature.asc (237 bytes) Download Attachment