Re: [DISCUSS] Git force pushing and deletion of branchs

2016-01-19 Thread Ufuk Celebi
Thank you :D

On Tue, Jan 19, 2016 at 6:41 PM, Till Rohrmann <trohrm...@apache.org> wrote:

> Thanks Max :-)
>
> On Tue, Jan 19, 2016 at 6:05 PM, Fabian Hueske <fhue...@gmail.com> wrote:
>
> > Thanks Max!
> >
> > 2016-01-19 18:04 GMT+01:00 Maximilian Michels <m...@apache.org>:
> >
> > > I've filed an issue at infra to protect the master:
> > > https://issues.apache.org/jira/browse/INFRA-11088
> > >
> > > On Fri, Jan 15, 2016 at 3:40 PM, Maximilian Michels <m...@apache.org>
> > > wrote:
> > > > +1 for a protected master.
> > > > +1 for creating release tags under rel/.
> > > >
> > > > On Thu, Jan 14, 2016 at 10:07 AM, Chiwan Park <chiwanp...@apache.org
> >
> > > wrote:
> > > >> +1 for protecting all branches including master.
> > > >>
> > > >>> On Jan 14, 2016, at 1:20 AM, Aljoscha Krettek <aljos...@apache.org
> >
> > > wrote:
> > > >>>
> > > >>> +1 on protecting the master
> > > >>>> On 13 Jan 2016, at 14:46, Márton Balassi <
> balassi.mar...@gmail.com>
> > > wrote:
> > > >>>>
> > > >>>> +1
> > > >>>>
> > > >>>> On Wed, Jan 13, 2016 at 12:37 PM, Matthias J. Sax <
> mj...@apache.org
> > >
> > > wrote:
> > > >>>>
> > > >>>>> +1
> > > >>>>>
> > > >>>>> On 01/13/2016 11:51 AM, Fabian Hueske wrote:
> > > >>>>>> @Stephan: You mean all tags should be protected, not only those
> > > under
> > > >>>>> rel?
> > > >>>>>>
> > > >>>>>> 2016-01-13 11:43 GMT+01:00 Till Rohrmann <trohrm...@apache.org
> >:
> > > >>>>>>
> > > >>>>>>> +1 for protecting the master branch.
> > > >>>>>>>
> > > >>>>>>> On Wed, Jan 13, 2016 at 11:42 AM, Li, Chengxiang <
> > > >>>>> chengxiang...@intel.com>
> > > >>>>>>> wrote:
> > > >>>>>>>
> > > >>>>>>>> +1 on the original style.
> > > >>>>>>>> Master branch disable force pushing in case of misusing and
> > > feature
> > > >>>>>>> branch
> > > >>>>>>>> enable force pushing for flexible developing.
> > > >>>>>>>>
> > > >>>>>>>> -Original Message-
> > > >>>>>>>> From: Gyula Fóra [mailto:gyf...@apache.org]
> > > >>>>>>>> Sent: Wednesday, January 13, 2016 6:36 PM
> > > >>>>>>>> To: dev@flink.apache.org
> > > >>>>>>>> Subject: Re: [DISCUSS] Git force pushing and deletion of
> branchs
> > > >>>>>>>>
> > > >>>>>>>> +1 for protecting the master branch.
> > > >>>>>>>>
> > > >>>>>>>> I also don't see any reason why anyone should force push there
> > > >>>>>>>>
> > > >>>>>>>> Gyula
> > > >>>>>>>>
> > > >>>>>>>> Fabian Hueske <fhue...@gmail.com> ezt írta (időpont: 2016.
> jan.
> > > 13.,
> > > >>>>>>> Sze,
> > > >>>>>>>> 11:07):
> > > >>>>>>>>
> > > >>>>>>>>> Hi everybody,
> > > >>>>>>>>>
> > > >>>>>>>>> Lately, ASF Infra has changed the write permissions of all
> Git
> > > >>>>>>>>> repositories twice.
> > > >>>>>>>>>
> > > >>>>>>>>> Originally, it was not possible to force into the master
> > branch.
> > > >>>>>>>>> A few weeks ago, infra disabled also force pushing into other
> > > >>>>> branches.
> > > >>>>>>>>>
> > > >>>>>>>>> Now, this has changed again after the issue was discussed
> with
> > > the ASF
> > > >>>>>>>>> board.
> > > >>>>>>>>> The current situation is the following:
> > > >>>>>>>>> - force pushing is allowed on all branched, including master
> > > >>>>>>>>> - branches and tags can be deleted (not sure if this applies
> as
> > > well
> > > >>>>>>>>> for the master branch)
> > > >>>>>>>>> - "the 'protected' portions of git to primarily focus on
> > > refs/tags/rel
> > > >>>>>>>>> - thus any tags under rel, will have their entire commit
> > > history."
> > > >>>>>>>>>
> > > >>>>>>>>> I am not 100% sure which exact parts of the repository are
> > > protected
> > > >>>>>>>>> now as I am not very much into the details of Git.
> > > >>>>>>>>> However, I believe we need to create new tags under rel for
> our
> > > >>>>>>>>> previous releases to protect them.
> > > >>>>>>>>>
> > > >>>>>>>>> In addition, I would like to propose to ask Infra to add
> > > protection
> > > >>>>>>>>> for the master branch. I can only recall very few situations
> > > where
> > > >>>>>>>>> changes had to be reverted. I am much more in favor of a
> > > reverting
> > > >>>>>>>>> commit now and then compared to a branch that can be
> > arbitrarily
> > > >>>>>>> changed.
> > > >>>>>>>>>
> > > >>>>>>>>> What do you think about this?
> > > >>>>>>>>>
> > > >>>>>>>>> Best, Fabian
> > > >>>>>>>>>
> > > >>
> > > >> Regards,
> > > >> Chiwan Park
> > > >>
> > > >>
> > >
> >
>


Re: [DISCUSS] Git force pushing and deletion of branchs

2016-01-19 Thread Maximilian Michels
I've filed an issue at infra to protect the master:
https://issues.apache.org/jira/browse/INFRA-11088

On Fri, Jan 15, 2016 at 3:40 PM, Maximilian Michels <m...@apache.org> wrote:
> +1 for a protected master.
> +1 for creating release tags under rel/.
>
> On Thu, Jan 14, 2016 at 10:07 AM, Chiwan Park <chiwanp...@apache.org> wrote:
>> +1 for protecting all branches including master.
>>
>>> On Jan 14, 2016, at 1:20 AM, Aljoscha Krettek <aljos...@apache.org> wrote:
>>>
>>> +1 on protecting the master
>>>> On 13 Jan 2016, at 14:46, Márton Balassi <balassi.mar...@gmail.com> wrote:
>>>>
>>>> +1
>>>>
>>>> On Wed, Jan 13, 2016 at 12:37 PM, Matthias J. Sax <mj...@apache.org> wrote:
>>>>
>>>>> +1
>>>>>
>>>>> On 01/13/2016 11:51 AM, Fabian Hueske wrote:
>>>>>> @Stephan: You mean all tags should be protected, not only those under
>>>>> rel?
>>>>>>
>>>>>> 2016-01-13 11:43 GMT+01:00 Till Rohrmann <trohrm...@apache.org>:
>>>>>>
>>>>>>> +1 for protecting the master branch.
>>>>>>>
>>>>>>> On Wed, Jan 13, 2016 at 11:42 AM, Li, Chengxiang <
>>>>> chengxiang...@intel.com>
>>>>>>> wrote:
>>>>>>>
>>>>>>>> +1 on the original style.
>>>>>>>> Master branch disable force pushing in case of misusing and feature
>>>>>>> branch
>>>>>>>> enable force pushing for flexible developing.
>>>>>>>>
>>>>>>>> -Original Message-
>>>>>>>> From: Gyula Fóra [mailto:gyf...@apache.org]
>>>>>>>> Sent: Wednesday, January 13, 2016 6:36 PM
>>>>>>>> To: dev@flink.apache.org
>>>>>>>> Subject: Re: [DISCUSS] Git force pushing and deletion of branchs
>>>>>>>>
>>>>>>>> +1 for protecting the master branch.
>>>>>>>>
>>>>>>>> I also don't see any reason why anyone should force push there
>>>>>>>>
>>>>>>>> Gyula
>>>>>>>>
>>>>>>>> Fabian Hueske <fhue...@gmail.com> ezt írta (időpont: 2016. jan. 13.,
>>>>>>> Sze,
>>>>>>>> 11:07):
>>>>>>>>
>>>>>>>>> Hi everybody,
>>>>>>>>>
>>>>>>>>> Lately, ASF Infra has changed the write permissions of all Git
>>>>>>>>> repositories twice.
>>>>>>>>>
>>>>>>>>> Originally, it was not possible to force into the master branch.
>>>>>>>>> A few weeks ago, infra disabled also force pushing into other
>>>>> branches.
>>>>>>>>>
>>>>>>>>> Now, this has changed again after the issue was discussed with the ASF
>>>>>>>>> board.
>>>>>>>>> The current situation is the following:
>>>>>>>>> - force pushing is allowed on all branched, including master
>>>>>>>>> - branches and tags can be deleted (not sure if this applies as well
>>>>>>>>> for the master branch)
>>>>>>>>> - "the 'protected' portions of git to primarily focus on refs/tags/rel
>>>>>>>>> - thus any tags under rel, will have their entire commit history."
>>>>>>>>>
>>>>>>>>> I am not 100% sure which exact parts of the repository are protected
>>>>>>>>> now as I am not very much into the details of Git.
>>>>>>>>> However, I believe we need to create new tags under rel for our
>>>>>>>>> previous releases to protect them.
>>>>>>>>>
>>>>>>>>> In addition, I would like to propose to ask Infra to add protection
>>>>>>>>> for the master branch. I can only recall very few situations where
>>>>>>>>> changes had to be reverted. I am much more in favor of a reverting
>>>>>>>>> commit now and then compared to a branch that can be arbitrarily
>>>>>>> changed.
>>>>>>>>>
>>>>>>>>> What do you think about this?
>>>>>>>>>
>>>>>>>>> Best, Fabian
>>>>>>>>>
>>
>> Regards,
>> Chiwan Park
>>
>>


Re: [DISCUSS] Git force pushing and deletion of branchs

2016-01-19 Thread Fabian Hueske
Thanks Max!

2016-01-19 18:04 GMT+01:00 Maximilian Michels <m...@apache.org>:

> I've filed an issue at infra to protect the master:
> https://issues.apache.org/jira/browse/INFRA-11088
>
> On Fri, Jan 15, 2016 at 3:40 PM, Maximilian Michels <m...@apache.org>
> wrote:
> > +1 for a protected master.
> > +1 for creating release tags under rel/.
> >
> > On Thu, Jan 14, 2016 at 10:07 AM, Chiwan Park <chiwanp...@apache.org>
> wrote:
> >> +1 for protecting all branches including master.
> >>
> >>> On Jan 14, 2016, at 1:20 AM, Aljoscha Krettek <aljos...@apache.org>
> wrote:
> >>>
> >>> +1 on protecting the master
> >>>> On 13 Jan 2016, at 14:46, Márton Balassi <balassi.mar...@gmail.com>
> wrote:
> >>>>
> >>>> +1
> >>>>
> >>>> On Wed, Jan 13, 2016 at 12:37 PM, Matthias J. Sax <mj...@apache.org>
> wrote:
> >>>>
> >>>>> +1
> >>>>>
> >>>>> On 01/13/2016 11:51 AM, Fabian Hueske wrote:
> >>>>>> @Stephan: You mean all tags should be protected, not only those
> under
> >>>>> rel?
> >>>>>>
> >>>>>> 2016-01-13 11:43 GMT+01:00 Till Rohrmann <trohrm...@apache.org>:
> >>>>>>
> >>>>>>> +1 for protecting the master branch.
> >>>>>>>
> >>>>>>> On Wed, Jan 13, 2016 at 11:42 AM, Li, Chengxiang <
> >>>>> chengxiang...@intel.com>
> >>>>>>> wrote:
> >>>>>>>
> >>>>>>>> +1 on the original style.
> >>>>>>>> Master branch disable force pushing in case of misusing and
> feature
> >>>>>>> branch
> >>>>>>>> enable force pushing for flexible developing.
> >>>>>>>>
> >>>>>>>> -Original Message-
> >>>>>>>> From: Gyula Fóra [mailto:gyf...@apache.org]
> >>>>>>>> Sent: Wednesday, January 13, 2016 6:36 PM
> >>>>>>>> To: dev@flink.apache.org
> >>>>>>>> Subject: Re: [DISCUSS] Git force pushing and deletion of branchs
> >>>>>>>>
> >>>>>>>> +1 for protecting the master branch.
> >>>>>>>>
> >>>>>>>> I also don't see any reason why anyone should force push there
> >>>>>>>>
> >>>>>>>> Gyula
> >>>>>>>>
> >>>>>>>> Fabian Hueske <fhue...@gmail.com> ezt írta (időpont: 2016. jan.
> 13.,
> >>>>>>> Sze,
> >>>>>>>> 11:07):
> >>>>>>>>
> >>>>>>>>> Hi everybody,
> >>>>>>>>>
> >>>>>>>>> Lately, ASF Infra has changed the write permissions of all Git
> >>>>>>>>> repositories twice.
> >>>>>>>>>
> >>>>>>>>> Originally, it was not possible to force into the master branch.
> >>>>>>>>> A few weeks ago, infra disabled also force pushing into other
> >>>>> branches.
> >>>>>>>>>
> >>>>>>>>> Now, this has changed again after the issue was discussed with
> the ASF
> >>>>>>>>> board.
> >>>>>>>>> The current situation is the following:
> >>>>>>>>> - force pushing is allowed on all branched, including master
> >>>>>>>>> - branches and tags can be deleted (not sure if this applies as
> well
> >>>>>>>>> for the master branch)
> >>>>>>>>> - "the 'protected' portions of git to primarily focus on
> refs/tags/rel
> >>>>>>>>> - thus any tags under rel, will have their entire commit
> history."
> >>>>>>>>>
> >>>>>>>>> I am not 100% sure which exact parts of the repository are
> protected
> >>>>>>>>> now as I am not very much into the details of Git.
> >>>>>>>>> However, I believe we need to create new tags under rel for our
> >>>>>>>>> previous releases to protect them.
> >>>>>>>>>
> >>>>>>>>> In addition, I would like to propose to ask Infra to add
> protection
> >>>>>>>>> for the master branch. I can only recall very few situations
> where
> >>>>>>>>> changes had to be reverted. I am much more in favor of a
> reverting
> >>>>>>>>> commit now and then compared to a branch that can be arbitrarily
> >>>>>>> changed.
> >>>>>>>>>
> >>>>>>>>> What do you think about this?
> >>>>>>>>>
> >>>>>>>>> Best, Fabian
> >>>>>>>>>
> >>
> >> Regards,
> >> Chiwan Park
> >>
> >>
>


Re: [DISCUSS] Git force pushing and deletion of branchs

2016-01-19 Thread Till Rohrmann
Thanks Max :-)

On Tue, Jan 19, 2016 at 6:05 PM, Fabian Hueske <fhue...@gmail.com> wrote:

> Thanks Max!
>
> 2016-01-19 18:04 GMT+01:00 Maximilian Michels <m...@apache.org>:
>
> > I've filed an issue at infra to protect the master:
> > https://issues.apache.org/jira/browse/INFRA-11088
> >
> > On Fri, Jan 15, 2016 at 3:40 PM, Maximilian Michels <m...@apache.org>
> > wrote:
> > > +1 for a protected master.
> > > +1 for creating release tags under rel/.
> > >
> > > On Thu, Jan 14, 2016 at 10:07 AM, Chiwan Park <chiwanp...@apache.org>
> > wrote:
> > >> +1 for protecting all branches including master.
> > >>
> > >>> On Jan 14, 2016, at 1:20 AM, Aljoscha Krettek <aljos...@apache.org>
> > wrote:
> > >>>
> > >>> +1 on protecting the master
> > >>>> On 13 Jan 2016, at 14:46, Márton Balassi <balassi.mar...@gmail.com>
> > wrote:
> > >>>>
> > >>>> +1
> > >>>>
> > >>>> On Wed, Jan 13, 2016 at 12:37 PM, Matthias J. Sax <mj...@apache.org
> >
> > wrote:
> > >>>>
> > >>>>> +1
> > >>>>>
> > >>>>> On 01/13/2016 11:51 AM, Fabian Hueske wrote:
> > >>>>>> @Stephan: You mean all tags should be protected, not only those
> > under
> > >>>>> rel?
> > >>>>>>
> > >>>>>> 2016-01-13 11:43 GMT+01:00 Till Rohrmann <trohrm...@apache.org>:
> > >>>>>>
> > >>>>>>> +1 for protecting the master branch.
> > >>>>>>>
> > >>>>>>> On Wed, Jan 13, 2016 at 11:42 AM, Li, Chengxiang <
> > >>>>> chengxiang...@intel.com>
> > >>>>>>> wrote:
> > >>>>>>>
> > >>>>>>>> +1 on the original style.
> > >>>>>>>> Master branch disable force pushing in case of misusing and
> > feature
> > >>>>>>> branch
> > >>>>>>>> enable force pushing for flexible developing.
> > >>>>>>>>
> > >>>>>>>> -Original Message-
> > >>>>>>>> From: Gyula Fóra [mailto:gyf...@apache.org]
> > >>>>>>>> Sent: Wednesday, January 13, 2016 6:36 PM
> > >>>>>>>> To: dev@flink.apache.org
> > >>>>>>>> Subject: Re: [DISCUSS] Git force pushing and deletion of branchs
> > >>>>>>>>
> > >>>>>>>> +1 for protecting the master branch.
> > >>>>>>>>
> > >>>>>>>> I also don't see any reason why anyone should force push there
> > >>>>>>>>
> > >>>>>>>> Gyula
> > >>>>>>>>
> > >>>>>>>> Fabian Hueske <fhue...@gmail.com> ezt írta (időpont: 2016. jan.
> > 13.,
> > >>>>>>> Sze,
> > >>>>>>>> 11:07):
> > >>>>>>>>
> > >>>>>>>>> Hi everybody,
> > >>>>>>>>>
> > >>>>>>>>> Lately, ASF Infra has changed the write permissions of all Git
> > >>>>>>>>> repositories twice.
> > >>>>>>>>>
> > >>>>>>>>> Originally, it was not possible to force into the master
> branch.
> > >>>>>>>>> A few weeks ago, infra disabled also force pushing into other
> > >>>>> branches.
> > >>>>>>>>>
> > >>>>>>>>> Now, this has changed again after the issue was discussed with
> > the ASF
> > >>>>>>>>> board.
> > >>>>>>>>> The current situation is the following:
> > >>>>>>>>> - force pushing is allowed on all branched, including master
> > >>>>>>>>> - branches and tags can be deleted (not sure if this applies as
> > well
> > >>>>>>>>> for the master branch)
> > >>>>>>>>> - "the 'protected' portions of git to primarily focus on
> > refs/tags/rel
> > >>>>>>>>> - thus any tags under rel, will have their entire commit
> > history."
> > >>>>>>>>>
> > >>>>>>>>> I am not 100% sure which exact parts of the repository are
> > protected
> > >>>>>>>>> now as I am not very much into the details of Git.
> > >>>>>>>>> However, I believe we need to create new tags under rel for our
> > >>>>>>>>> previous releases to protect them.
> > >>>>>>>>>
> > >>>>>>>>> In addition, I would like to propose to ask Infra to add
> > protection
> > >>>>>>>>> for the master branch. I can only recall very few situations
> > where
> > >>>>>>>>> changes had to be reverted. I am much more in favor of a
> > reverting
> > >>>>>>>>> commit now and then compared to a branch that can be
> arbitrarily
> > >>>>>>> changed.
> > >>>>>>>>>
> > >>>>>>>>> What do you think about this?
> > >>>>>>>>>
> > >>>>>>>>> Best, Fabian
> > >>>>>>>>>
> > >>
> > >> Regards,
> > >> Chiwan Park
> > >>
> > >>
> >
>


Re: [DISCUSS] Git force pushing and deletion of branchs

2016-01-15 Thread Maximilian Michels
+1 for a protected master.
+1 for creating release tags under rel/.

On Thu, Jan 14, 2016 at 10:07 AM, Chiwan Park <chiwanp...@apache.org> wrote:
> +1 for protecting all branches including master.
>
>> On Jan 14, 2016, at 1:20 AM, Aljoscha Krettek <aljos...@apache.org> wrote:
>>
>> +1 on protecting the master
>>> On 13 Jan 2016, at 14:46, Márton Balassi <balassi.mar...@gmail.com> wrote:
>>>
>>> +1
>>>
>>> On Wed, Jan 13, 2016 at 12:37 PM, Matthias J. Sax <mj...@apache.org> wrote:
>>>
>>>> +1
>>>>
>>>> On 01/13/2016 11:51 AM, Fabian Hueske wrote:
>>>>> @Stephan: You mean all tags should be protected, not only those under
>>>> rel?
>>>>>
>>>>> 2016-01-13 11:43 GMT+01:00 Till Rohrmann <trohrm...@apache.org>:
>>>>>
>>>>>> +1 for protecting the master branch.
>>>>>>
>>>>>> On Wed, Jan 13, 2016 at 11:42 AM, Li, Chengxiang <
>>>> chengxiang...@intel.com>
>>>>>> wrote:
>>>>>>
>>>>>>> +1 on the original style.
>>>>>>> Master branch disable force pushing in case of misusing and feature
>>>>>> branch
>>>>>>> enable force pushing for flexible developing.
>>>>>>>
>>>>>>> -Original Message-
>>>>>>> From: Gyula Fóra [mailto:gyf...@apache.org]
>>>>>>> Sent: Wednesday, January 13, 2016 6:36 PM
>>>>>>> To: dev@flink.apache.org
>>>>>>> Subject: Re: [DISCUSS] Git force pushing and deletion of branchs
>>>>>>>
>>>>>>> +1 for protecting the master branch.
>>>>>>>
>>>>>>> I also don't see any reason why anyone should force push there
>>>>>>>
>>>>>>> Gyula
>>>>>>>
>>>>>>> Fabian Hueske <fhue...@gmail.com> ezt írta (időpont: 2016. jan. 13.,
>>>>>> Sze,
>>>>>>> 11:07):
>>>>>>>
>>>>>>>> Hi everybody,
>>>>>>>>
>>>>>>>> Lately, ASF Infra has changed the write permissions of all Git
>>>>>>>> repositories twice.
>>>>>>>>
>>>>>>>> Originally, it was not possible to force into the master branch.
>>>>>>>> A few weeks ago, infra disabled also force pushing into other
>>>> branches.
>>>>>>>>
>>>>>>>> Now, this has changed again after the issue was discussed with the ASF
>>>>>>>> board.
>>>>>>>> The current situation is the following:
>>>>>>>> - force pushing is allowed on all branched, including master
>>>>>>>> - branches and tags can be deleted (not sure if this applies as well
>>>>>>>> for the master branch)
>>>>>>>> - "the 'protected' portions of git to primarily focus on refs/tags/rel
>>>>>>>> - thus any tags under rel, will have their entire commit history."
>>>>>>>>
>>>>>>>> I am not 100% sure which exact parts of the repository are protected
>>>>>>>> now as I am not very much into the details of Git.
>>>>>>>> However, I believe we need to create new tags under rel for our
>>>>>>>> previous releases to protect them.
>>>>>>>>
>>>>>>>> In addition, I would like to propose to ask Infra to add protection
>>>>>>>> for the master branch. I can only recall very few situations where
>>>>>>>> changes had to be reverted. I am much more in favor of a reverting
>>>>>>>> commit now and then compared to a branch that can be arbitrarily
>>>>>> changed.
>>>>>>>>
>>>>>>>> What do you think about this?
>>>>>>>>
>>>>>>>> Best, Fabian
>>>>>>>>
>
> Regards,
> Chiwan Park
>
>


[DISCUSS] Git force pushing and deletion of branchs

2016-01-13 Thread Fabian Hueske
Hi everybody,

Lately, ASF Infra has changed the write permissions of all Git repositories
twice.

Originally, it was not possible to force into the master branch.
A few weeks ago, infra disabled also force pushing into other branches.

Now, this has changed again after the issue was discussed with the ASF
board.
The current situation is the following:
- force pushing is allowed on all branched, including master
- branches and tags can be deleted (not sure if this applies as well for
the master branch)
- "the 'protected' portions of git to primarily focus on refs/tags/rel -
thus any tags under rel, will have their entire commit history."

I am not 100% sure which exact parts of the repository are protected now as
I am not very much into the details of Git.
However, I believe we need to create new tags under rel for our previous
releases to protect them.

In addition, I would like to propose to ask Infra to add protection for the
master branch. I can only recall very few situations where changes had to
be reverted. I am much more in favor of a reverting commit now and then
compared to a branch that can be arbitrarily changed.

What do you think about this?

Best, Fabian


Re: [DISCUSS] Git force pushing and deletion of branchs

2016-01-13 Thread Gyula Fóra
+1 for protecting the master branch.

I also don't see any reason why anyone should force push there

Gyula

Fabian Hueske  ezt írta (időpont: 2016. jan. 13., Sze,
11:07):

> Hi everybody,
>
> Lately, ASF Infra has changed the write permissions of all Git repositories
> twice.
>
> Originally, it was not possible to force into the master branch.
> A few weeks ago, infra disabled also force pushing into other branches.
>
> Now, this has changed again after the issue was discussed with the ASF
> board.
> The current situation is the following:
> - force pushing is allowed on all branched, including master
> - branches and tags can be deleted (not sure if this applies as well for
> the master branch)
> - "the 'protected' portions of git to primarily focus on refs/tags/rel -
> thus any tags under rel, will have their entire commit history."
>
> I am not 100% sure which exact parts of the repository are protected now as
> I am not very much into the details of Git.
> However, I believe we need to create new tags under rel for our previous
> releases to protect them.
>
> In addition, I would like to propose to ask Infra to add protection for the
> master branch. I can only recall very few situations where changes had to
> be reverted. I am much more in favor of a reverting commit now and then
> compared to a branch that can be arbitrarily changed.
>
> What do you think about this?
>
> Best, Fabian
>


RE: [DISCUSS] Git force pushing and deletion of branchs

2016-01-13 Thread Li, Chengxiang
+1 on the original style.
Master branch disable force pushing in case of misusing and feature branch 
enable force pushing for flexible developing.

-Original Message-
From: Gyula Fóra [mailto:gyf...@apache.org] 
Sent: Wednesday, January 13, 2016 6:36 PM
To: dev@flink.apache.org
Subject: Re: [DISCUSS] Git force pushing and deletion of branchs

+1 for protecting the master branch.

I also don't see any reason why anyone should force push there

Gyula

Fabian Hueske <fhue...@gmail.com> ezt írta (időpont: 2016. jan. 13., Sze,
11:07):

> Hi everybody,
>
> Lately, ASF Infra has changed the write permissions of all Git 
> repositories twice.
>
> Originally, it was not possible to force into the master branch.
> A few weeks ago, infra disabled also force pushing into other branches.
>
> Now, this has changed again after the issue was discussed with the ASF 
> board.
> The current situation is the following:
> - force pushing is allowed on all branched, including master
> - branches and tags can be deleted (not sure if this applies as well 
> for the master branch)
> - "the 'protected' portions of git to primarily focus on refs/tags/rel 
> - thus any tags under rel, will have their entire commit history."
>
> I am not 100% sure which exact parts of the repository are protected 
> now as I am not very much into the details of Git.
> However, I believe we need to create new tags under rel for our 
> previous releases to protect them.
>
> In addition, I would like to propose to ask Infra to add protection 
> for the master branch. I can only recall very few situations where 
> changes had to be reverted. I am much more in favor of a reverting 
> commit now and then compared to a branch that can be arbitrarily changed.
>
> What do you think about this?
>
> Best, Fabian
>


Re: [DISCUSS] Git force pushing and deletion of branchs

2016-01-13 Thread Stephan Ewen
+1

Important to protect the master.

I think we should also protect Tags.

On Wed, Jan 13, 2016 at 11:36 AM, Gyula Fóra  wrote:

> +1 for protecting the master branch.
>
> I also don't see any reason why anyone should force push there
>
> Gyula
>
> Fabian Hueske  ezt írta (időpont: 2016. jan. 13., Sze,
> 11:07):
>
> > Hi everybody,
> >
> > Lately, ASF Infra has changed the write permissions of all Git
> repositories
> > twice.
> >
> > Originally, it was not possible to force into the master branch.
> > A few weeks ago, infra disabled also force pushing into other branches.
> >
> > Now, this has changed again after the issue was discussed with the ASF
> > board.
> > The current situation is the following:
> > - force pushing is allowed on all branched, including master
> > - branches and tags can be deleted (not sure if this applies as well for
> > the master branch)
> > - "the 'protected' portions of git to primarily focus on refs/tags/rel -
> > thus any tags under rel, will have their entire commit history."
> >
> > I am not 100% sure which exact parts of the repository are protected now
> as
> > I am not very much into the details of Git.
> > However, I believe we need to create new tags under rel for our previous
> > releases to protect them.
> >
> > In addition, I would like to propose to ask Infra to add protection for
> the
> > master branch. I can only recall very few situations where changes had to
> > be reverted. I am much more in favor of a reverting commit now and then
> > compared to a branch that can be arbitrarily changed.
> >
> > What do you think about this?
> >
> > Best, Fabian
> >
>


Re: [DISCUSS] Git force pushing and deletion of branchs

2016-01-13 Thread Till Rohrmann
+1 for protecting the master branch.

On Wed, Jan 13, 2016 at 11:42 AM, Li, Chengxiang <chengxiang...@intel.com>
wrote:

> +1 on the original style.
> Master branch disable force pushing in case of misusing and feature branch
> enable force pushing for flexible developing.
>
> -Original Message-
> From: Gyula Fóra [mailto:gyf...@apache.org]
> Sent: Wednesday, January 13, 2016 6:36 PM
> To: dev@flink.apache.org
> Subject: Re: [DISCUSS] Git force pushing and deletion of branchs
>
> +1 for protecting the master branch.
>
> I also don't see any reason why anyone should force push there
>
> Gyula
>
> Fabian Hueske <fhue...@gmail.com> ezt írta (időpont: 2016. jan. 13., Sze,
> 11:07):
>
> > Hi everybody,
> >
> > Lately, ASF Infra has changed the write permissions of all Git
> > repositories twice.
> >
> > Originally, it was not possible to force into the master branch.
> > A few weeks ago, infra disabled also force pushing into other branches.
> >
> > Now, this has changed again after the issue was discussed with the ASF
> > board.
> > The current situation is the following:
> > - force pushing is allowed on all branched, including master
> > - branches and tags can be deleted (not sure if this applies as well
> > for the master branch)
> > - "the 'protected' portions of git to primarily focus on refs/tags/rel
> > - thus any tags under rel, will have their entire commit history."
> >
> > I am not 100% sure which exact parts of the repository are protected
> > now as I am not very much into the details of Git.
> > However, I believe we need to create new tags under rel for our
> > previous releases to protect them.
> >
> > In addition, I would like to propose to ask Infra to add protection
> > for the master branch. I can only recall very few situations where
> > changes had to be reverted. I am much more in favor of a reverting
> > commit now and then compared to a branch that can be arbitrarily changed.
> >
> > What do you think about this?
> >
> > Best, Fabian
> >
>


Re: [DISCUSS] Git force pushing and deletion of branchs

2016-01-13 Thread Fabian Hueske
@Stephan: You mean all tags should be protected, not only those under rel?

2016-01-13 11:43 GMT+01:00 Till Rohrmann <trohrm...@apache.org>:

> +1 for protecting the master branch.
>
> On Wed, Jan 13, 2016 at 11:42 AM, Li, Chengxiang <chengxiang...@intel.com>
> wrote:
>
> > +1 on the original style.
> > Master branch disable force pushing in case of misusing and feature
> branch
> > enable force pushing for flexible developing.
> >
> > -Original Message-
> > From: Gyula Fóra [mailto:gyf...@apache.org]
> > Sent: Wednesday, January 13, 2016 6:36 PM
> > To: dev@flink.apache.org
> > Subject: Re: [DISCUSS] Git force pushing and deletion of branchs
> >
> > +1 for protecting the master branch.
> >
> > I also don't see any reason why anyone should force push there
> >
> > Gyula
> >
> > Fabian Hueske <fhue...@gmail.com> ezt írta (időpont: 2016. jan. 13.,
> Sze,
> > 11:07):
> >
> > > Hi everybody,
> > >
> > > Lately, ASF Infra has changed the write permissions of all Git
> > > repositories twice.
> > >
> > > Originally, it was not possible to force into the master branch.
> > > A few weeks ago, infra disabled also force pushing into other branches.
> > >
> > > Now, this has changed again after the issue was discussed with the ASF
> > > board.
> > > The current situation is the following:
> > > - force pushing is allowed on all branched, including master
> > > - branches and tags can be deleted (not sure if this applies as well
> > > for the master branch)
> > > - "the 'protected' portions of git to primarily focus on refs/tags/rel
> > > - thus any tags under rel, will have their entire commit history."
> > >
> > > I am not 100% sure which exact parts of the repository are protected
> > > now as I am not very much into the details of Git.
> > > However, I believe we need to create new tags under rel for our
> > > previous releases to protect them.
> > >
> > > In addition, I would like to propose to ask Infra to add protection
> > > for the master branch. I can only recall very few situations where
> > > changes had to be reverted. I am much more in favor of a reverting
> > > commit now and then compared to a branch that can be arbitrarily
> changed.
> > >
> > > What do you think about this?
> > >
> > > Best, Fabian
> > >
> >
>


Re: [DISCUSS] Git force pushing and deletion of branchs

2016-01-13 Thread Matthias J. Sax
+1

On 01/13/2016 11:51 AM, Fabian Hueske wrote:
> @Stephan: You mean all tags should be protected, not only those under rel?
> 
> 2016-01-13 11:43 GMT+01:00 Till Rohrmann <trohrm...@apache.org>:
> 
>> +1 for protecting the master branch.
>>
>> On Wed, Jan 13, 2016 at 11:42 AM, Li, Chengxiang <chengxiang...@intel.com>
>> wrote:
>>
>>> +1 on the original style.
>>> Master branch disable force pushing in case of misusing and feature
>> branch
>>> enable force pushing for flexible developing.
>>>
>>> -Original Message-
>>> From: Gyula Fóra [mailto:gyf...@apache.org]
>>> Sent: Wednesday, January 13, 2016 6:36 PM
>>> To: dev@flink.apache.org
>>> Subject: Re: [DISCUSS] Git force pushing and deletion of branchs
>>>
>>> +1 for protecting the master branch.
>>>
>>> I also don't see any reason why anyone should force push there
>>>
>>> Gyula
>>>
>>> Fabian Hueske <fhue...@gmail.com> ezt írta (időpont: 2016. jan. 13.,
>> Sze,
>>> 11:07):
>>>
>>>> Hi everybody,
>>>>
>>>> Lately, ASF Infra has changed the write permissions of all Git
>>>> repositories twice.
>>>>
>>>> Originally, it was not possible to force into the master branch.
>>>> A few weeks ago, infra disabled also force pushing into other branches.
>>>>
>>>> Now, this has changed again after the issue was discussed with the ASF
>>>> board.
>>>> The current situation is the following:
>>>> - force pushing is allowed on all branched, including master
>>>> - branches and tags can be deleted (not sure if this applies as well
>>>> for the master branch)
>>>> - "the 'protected' portions of git to primarily focus on refs/tags/rel
>>>> - thus any tags under rel, will have their entire commit history."
>>>>
>>>> I am not 100% sure which exact parts of the repository are protected
>>>> now as I am not very much into the details of Git.
>>>> However, I believe we need to create new tags under rel for our
>>>> previous releases to protect them.
>>>>
>>>> In addition, I would like to propose to ask Infra to add protection
>>>> for the master branch. I can only recall very few situations where
>>>> changes had to be reverted. I am much more in favor of a reverting
>>>> commit now and then compared to a branch that can be arbitrarily
>> changed.
>>>>
>>>> What do you think about this?
>>>>
>>>> Best, Fabian
>>>>
>>>
>>
> 



signature.asc
Description: OpenPGP digital signature


Re: [DISCUSS] Git force pushing and deletion of branchs

2016-01-13 Thread Aljoscha Krettek
+1 on protecting the master
> On 13 Jan 2016, at 14:46, Márton Balassi <balassi.mar...@gmail.com> wrote:
> 
> +1
> 
> On Wed, Jan 13, 2016 at 12:37 PM, Matthias J. Sax <mj...@apache.org> wrote:
> 
>> +1
>> 
>> On 01/13/2016 11:51 AM, Fabian Hueske wrote:
>>> @Stephan: You mean all tags should be protected, not only those under
>> rel?
>>> 
>>> 2016-01-13 11:43 GMT+01:00 Till Rohrmann <trohrm...@apache.org>:
>>> 
>>>> +1 for protecting the master branch.
>>>> 
>>>> On Wed, Jan 13, 2016 at 11:42 AM, Li, Chengxiang <
>> chengxiang...@intel.com>
>>>> wrote:
>>>> 
>>>>> +1 on the original style.
>>>>> Master branch disable force pushing in case of misusing and feature
>>>> branch
>>>>> enable force pushing for flexible developing.
>>>>> 
>>>>> -Original Message-
>>>>> From: Gyula Fóra [mailto:gyf...@apache.org]
>>>>> Sent: Wednesday, January 13, 2016 6:36 PM
>>>>> To: dev@flink.apache.org
>>>>> Subject: Re: [DISCUSS] Git force pushing and deletion of branchs
>>>>> 
>>>>> +1 for protecting the master branch.
>>>>> 
>>>>> I also don't see any reason why anyone should force push there
>>>>> 
>>>>> Gyula
>>>>> 
>>>>> Fabian Hueske <fhue...@gmail.com> ezt írta (időpont: 2016. jan. 13.,
>>>> Sze,
>>>>> 11:07):
>>>>> 
>>>>>> Hi everybody,
>>>>>> 
>>>>>> Lately, ASF Infra has changed the write permissions of all Git
>>>>>> repositories twice.
>>>>>> 
>>>>>> Originally, it was not possible to force into the master branch.
>>>>>> A few weeks ago, infra disabled also force pushing into other
>> branches.
>>>>>> 
>>>>>> Now, this has changed again after the issue was discussed with the ASF
>>>>>> board.
>>>>>> The current situation is the following:
>>>>>> - force pushing is allowed on all branched, including master
>>>>>> - branches and tags can be deleted (not sure if this applies as well
>>>>>> for the master branch)
>>>>>> - "the 'protected' portions of git to primarily focus on refs/tags/rel
>>>>>> - thus any tags under rel, will have their entire commit history."
>>>>>> 
>>>>>> I am not 100% sure which exact parts of the repository are protected
>>>>>> now as I am not very much into the details of Git.
>>>>>> However, I believe we need to create new tags under rel for our
>>>>>> previous releases to protect them.
>>>>>> 
>>>>>> In addition, I would like to propose to ask Infra to add protection
>>>>>> for the master branch. I can only recall very few situations where
>>>>>> changes had to be reverted. I am much more in favor of a reverting
>>>>>> commit now and then compared to a branch that can be arbitrarily
>>>> changed.
>>>>>> 
>>>>>> What do you think about this?
>>>>>> 
>>>>>> Best, Fabian
>>>>>> 
>>>>> 
>>>> 
>>> 
>> 
>>