[JIRA] (OVIRT-990) Re: Gerrit don't check for "Signed-Off" anymore

eyal edri [Administrator] (oVirt JIRA) jira at ovirt-jira.atlassian.net
Wed Dec 28 14:20:02 UTC 2016


    [ https://ovirt-jira.atlassian.net/browse/OVIRT-990?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=25030#comment-25030 ] 

eyal edri [Administrator] commented on OVIRT-990:
-------------------------------------------------

[~tnisan at redhat.com] the permission needed to rebase on other commits is 'forge author identity' and that is still valid for all registered uses.

> Re: Gerrit don't check for "Signed-Off" anymore
> -----------------------------------------------
>
>                 Key: OVIRT-990
>                 URL: https://ovirt-jira.atlassian.net/browse/OVIRT-990
>             Project: oVirt - virtualization made easy
>          Issue Type: By-EMAIL
>            Reporter: eyal edri [Administrator]
>            Assignee: Shlomo Ben David
>
> On Wed, Dec 28, 2016 at 11:42 AM, Tal Nisan <tnisan at redhat.com> wrote:
> > Last night I've push a serious of patches and forgot to sign off, up until
> > not long ago there was a hook that enforces that but it seems like it's not
> > working anymore and I was able to push the patches without the sign off:
> > https://gerrit.ovirt.org/#/c/69219/1
> >
> >
> Its not a hook, its a definition per project, each project has the
> defenition if to enforce signed-off commit:
> 'Require Signed-off-by in commit message:'  - and its on TRUE now for
> ovirt-engine, so I'm not sure how its possible that a patch got in, we'll
> need to check gerrit logs to see.
> Opening a ticket on it.
> >
> > _______________________________________________
> > Infra mailing list
> > Infra at ovirt.org
> > http://lists.ovirt.org/mailman/listinfo/infra
> >
> >
> -- 
> Eyal Edri
> Associate Manager
> RHV DevOps
> EMEA ENG Virtualization R&D
> Red Hat Israel
> phone: +972-9-7692018
> irc: eedri (on #tlv #rhev-dev #rhev-integ)



--
This message was sent by Atlassian JIRA
(v1000.621.5#100023)


More information about the Infra mailing list