Can not rebase or push updates to patches with different author

Anton Marchukov amarchuk at redhat.com
Thu Oct 20 13:27:34 UTC 2016


Hello Shlomi.

I also had to add it to:

ovirt-engine-sdk
ovirt-engine-sdk-java
ovirt-engine-sdk-js
ovirt-engine-sdk-tests

May it be that defaults are not sane enough for this particular rule?


On Wed, Oct 19, 2016 at 11:50 PM, Shlomo Ben David <sbendavi at redhat.com>
wrote:

> Hi,
>
> This permission was set for project owners only,
> But in this specific case I think that it's the right solution, because
> these projects have customized permissions.
>
> Thanks a lot Anton.
>
> BR,
>
> Shlomi Ben-David
>
> בתאריך 19 באוק׳ 2016 15:34,‏ "Tal Nisan" <tnisan at redhat.com> כתב:
>
> Makes perfect sense for maintainers as we have to rebase or sometimes post
>> minor updates (e.g. commit message) for patches that we are not the authors
>> of.
>> As for non maintainer users it is of small to no benefit at all so I
>> think it's the best course of action.
>>
>> Thanks.
>>
>>
>> On Wed, Oct 19, 2016 at 3:31 PM, Anton Marchukov <amarchuk at redhat.com>
>> wrote:
>>
>>> FYI. I have added Forge Author Identity to corresponding maintainers
>>> groups for master and stables branches of ovirt-engine
>>>
>>> On Wed, Oct 19, 2016 at 1:37 PM, Anton Marchukov <amarchuk at redhat.com>
>>> wrote:
>>>
>>>> Hello Shlomi.
>>>>
>>>> May it be related to your permissions generalization work? This is
>>>> something to consider/check.
>>>>
>>>> Anton.
>>>>
>>>>
>>>>
>>>> On Wed, Oct 19, 2016 at 1:26 PM, Tal Nisan <tnisan at redhat.com> wrote:
>>>>
>>>>> Hi,
>>>>>
>>>>> Up till not long ago it was possible to rebase or push updates to
>>>>> patches even if I was not the author of the patch, now I've noticed that
>>>>> when trying to do so I encounter "invalid author" error.
>>>>> For maintainers (especially in the stable branches) this is crucial
>>>>> and makes the cycle much shorter as no intervention from the original
>>>>> author is needed on trivial rebases, can you please restore the old state?
>>>>>
>>>>> Tal.
>>>>>
>>>>>
>>>>> _______________________________________________
>>>>> Infra mailing list
>>>>> Infra at ovirt.org
>>>>> http://lists.ovirt.org/mailman/listinfo/infra
>>>>>
>>>>>
>>>>
>>>>
>>>> --
>>>> Anton Marchukov
>>>> Senior Software Engineer - RHEV CI - Red Hat
>>>>
>>>>
>>>
>>>
>>> --
>>> Anton Marchukov
>>> Senior Software Engineer - RHEV CI - Red Hat
>>>
>>>
>>


-- 
Anton Marchukov
Senior Software Engineer - RHEV CI - Red Hat
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.ovirt.org/pipermail/infra/attachments/20161020/2440a956/attachment.html>


More information about the Infra mailing list