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(a)ovirt.org http://lists.ovirt.org/mailman/listinfo/infra

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(a)redhat.com&gt; 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(a)redhat.com&gt; 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(a)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

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(a)redhat.com&gt; כתב: 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(a)redhat.com&gt; > 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(a)redhat.com&gt; >> 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(a)redhat.com&gt; 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(a)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 >> >> >

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(a)redhat.com&gt; 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(a)redhat.com&gt; כתב: > > 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(a)redhat.com&gt; >> 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(a)redhat.com&gt; >>> 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(a)redhat.com&gt; 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(a)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