On Tue 27 May 2014 05:39:28 PM CEST, Kobi Ianko wrote:
>
>
> ----- Original Message -----
>> From: "David Caro" <[email protected]>
>> To: "Kobi Ianko" <[email protected]>
>> Cc: [email protected]
>> Sent: Tuesday, May 27, 2014 6:25:22 PM
>> Subject: Re: [ovirt-devel] Error pushing to gerrit
>>
>> On Tue 27 May 2014 05:22:34 PM CEST, Kobi Ianko wrote:
>>> Hi All,
>>> when I try to push a patch I get:
>>>
>>>  ! [remote rejected] HEAD -> refs/for/master (not Signed-off-by
>>>  author/committer/uploader in commit message footer)
>>> error: failed to push some refs to 'gerrit.ovirt.org:ovirt-engine'
>>>
>>>
>>> Although my patch is signed.
>>>
>>> Does someone have any clue how to solve this?
>>>
>>> 10x, Kobi
>>>
>>> _______________________________________________
>>> Devel mailing list
>>> [email protected]
>>> http://lists.ovirt.org/mailman/listinfo/devel
>>
>> What does git log -1 show?
>
> git log looks good:
>
> commit b5f56e86c64af232d3794a6f1534100a28615e77
> Author: Kobi Ianko <[email protected]>
> Date:   Tue May 13 19:00:08 2014 +0300
>
>     core: New internal command, VmSlaPolicyCommand
>
>     Adding a new command to handle the SLA parameters.
>     For now the new command only contains the CPU Limit parameter
>     which limits the VM's CPU consumption.
>
>     This Command will be called from the updateVmCommand, and 
> updateMomPolicyCommand.
>     later on it will be called from other parts of the system that configures 
> SLA parameters.
>
>     Change-Id: I26b0794c52cfe1d352f26392a835023afcd9efa0
>     Bug-Url: https://bugzilla.redhat.com/1084930
>     Signed-off-by: Kobi Ianko <[email protected]>
>
>>
>> Probably the Signed-off-by header or it's not well places, or it's not
>> signed by the committer/author/uploader. So you can try to amend the
>> author, or the committer, or the header.
>>
>> --
>> David Caro
>>
>> Red Hat S.L.
>> Continuous Integration Engineer - EMEA ENG Virtualization R&D
>>
>> Email: [email protected]
>> Web: www.redhat.com
>> RHT Global #: 82-62605
>>
>>
>> _______________________________________________
>> Devel mailing list
>> [email protected]
>> http://lists.ovirt.org/mailman/listinfo/devel

Are you sure the are no extra spaces around the headers or extra 
newlines?

--
David Caro

Red Hat S.L.
Continuous Integration Engineer - EMEA ENG Virtualization R&D

Email: [email protected]
Web: www.redhat.com
RHT Global #: 82-62605

Attachment: signature.asc
Description: OpenPGP digital signature

_______________________________________________
Devel mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/devel

Reply via email to