Hi Ed,
Yes – it blocked you because John M had the only Signed-off-by but he was not the author nor the uploader.
Or to put it another way, you were both the author and the uploader, but you did not have a Signed-off-by line.
-Jim
From: SPDK <spdk-bounces@lists.01.org> on behalf of "Rodriguez, Edwin" <Ed.Rodriguez@netapp.com>
Reply-To: Storage Performance Development Kit <spdk@lists.01.org>
Date: Friday, May 4, 2018 at 9:03 AM
To: Storage Performance Development Kit <spdk@lists.01.org>
Subject: Re: [SPDK] GerritHub rejects push
Well, I was the author and uploader with John M being the signer yet it still blocked me.
From: SPDK <spdk-bounces@lists.01.org> on behalf of "Harris, James R" <james.r.harris@intel.com>
Reply-To: Storage Performance Development Kit <spdk@lists.01.org>
Date: Friday, May 4, 2018 at 12:00 PM
To: Storage Performance Development Kit <spdk@lists.01.org>
Subject: Re: [SPDK] GerritHub rejects push
Hi Ed,
Yes – adding yourself was the right way to fix this. Either the author (as designated in the patch) or the uploader must match one of the Signed-off-by lines.
-Jim
From:
SPDK <spdk-bounces@lists.01.org> on behalf of "Rodriguez, Edwin" <Ed.Rodriguez@netapp.com>
Reply-To: Storage Performance Development Kit <spdk@lists.01.org>
Date: Friday, May 4, 2018 at 8:55 AM
To: Storage Performance Development Kit <spdk@lists.01.org>
Subject: Re: [SPDK] GerritHub rejects push
I then signed it myself and the push succeeded:
[edwinr@edwinr-mac-0:16]~/p4/SPDK/gerrithub/add_git_hooks/spdk(bdev_fprint_removal)>
git commit --amend --sign
[bdev_fprint_removal 97fe5a5a] bdev: Replace fprintf with log message
Author: Ed Rodriguez <ed.rodriguez@netapp.com>
Date: Thu May 3 14:36:36 2018 -0400
1 file changed, 1 insertion(+), 1 deletion(-)
[edwinr@edwinr-mac-0:17]~/p4/SPDK/gerrithub/add_git_hooks/spdk(bdev_fprint_removal)>
git log -1
commit 97fe5a5af80c0e9b48c7d50625bbf3ea747b3879 (HEAD
-> bdev_fprint_removal)
Author: Ed Rodriguez <ed.rodriguez@netapp.com>
Date: Thu May 3 14:36:36 2018 -0400
bdev: Replace fprintf with log message
Change-Id: Ic980f3f2f04db61e3ff7ce305817eede7a09cb5a
Signed-off-by: John Meneghini <johnm@netapp.com>
Signed-off-by: Ed rodriguez <edwinr@netapp.com>
[edwinr@edwinr-mac-0:18]~/p4/SPDK/gerrithub/add_git_hooks/spdk(bdev_fprint_removal)>
git push origin HEAD:refs/for/master
Counting objects: 5, done.
Delta compression using up to 8 threads.
Compressing objects: 100% (5/5), done.
Writing objects: 100% (5/5), 538 bytes | 76.00 KiB/s, done.
Total 5 (delta 4), reused 0 (delta 0)
remote: Resolving deltas: 100% (4/4)
remote: Processing changes: new: 1, done
remote:
remote: New Changes:
remote: https://review.gerrithub.io/#/c/spdk/spdk/+/410056 bdev: Replace fprintf with log message
remote:
To https://review.gerrithub.io/spdk/spdk
* [new branch] HEAD -> refs/for/master
From:
Ed Rodriguez <Ed.Rodriguez@netapp.com>
Date: Friday, May 4, 2018 at 11:52 AM
To: Storage Performance Development Kit <spdk@lists.01.org>
Subject: Re: [SPDK] GerritHub rejects push
In this case I’m trying to push a change to gerrithub that was signed by John M. And I get:
[edwinr@edwinr-mac-0:14]~/p4/SPDK/gerrithub/add_git_hooks/spdk(bdev_fprint_removal)> git log -1
commit bd6c9f009f1e08f8bb24bda62a81d06b154e6c9b (HEAD ->
bdev_fprint_removal,
bitbucket/bdev_fprint_removal)
Author: Ed Rodriguez <ed.rodriguez@netapp.com>
Date:
Thu May 3 14:36:36 2018 -0400
bdev: Replace fprintf with log message
Signed-off-by: John Meneghini <johnm@netapp.com>
[edwinr@edwinr-mac-0:15]~/p4/SPDK/gerrithub/add_git_hooks/spdk(bdev_fprint_removal)> git push origin HEAD:refs/for/master
Counting objects: 5, done.
Delta compression using up to 8 threads.
Compressing objects: 100% (5/5), done.
Writing objects: 100% (5/5), 489 bytes | 69.00 KiB/s, done.
Total 5 (delta 4), reused 0 (delta 0)
remote: Resolving deltas: 100% (4/4)
remote: Processing changes: refs: 1, done
remote:
remote: ERROR:
In commit bd6c9f009f1e08f8bb24bda62a81d06b154e6c9b
remote: ERROR:
committer email address johnm@netapp.com
remote: ERROR:
does not match your user account and you have no 'forge committer' permission.
remote: ERROR:
remote: ERROR:
The following addresses are currently registered:
remote: ERROR:
ed.rodriguez@netapp.com
remote: ERROR:
edwinr@netapp.com
remote: ERROR:
remote: ERROR:
To register an email address, please visit:
remote: ERROR:
https://review.gerrithub.io/#/settings/contact
remote:
remote:
To https://review.gerrithub.io/spdk/spdk
! [remote rejected]
HEAD -> refs/for/master (invalid committer)
error: failed to push some refs to 'https://review.gerrithub.io/spdk/spdk'
Ed R
From:
SPDK <spdk-bounces@lists.01.org> on behalf of "Harris, James R" <james.r.harris@intel.com>
Reply-To: Storage Performance Development Kit <spdk@lists.01.org>
Date: Thursday, May 3, 2018 at 5:59 PM
To: Storage Performance Development Kit <spdk@lists.01.org>
Subject: Re: [SPDK] GerritHub rejects push
Hi John,
Signed-off-by means that the author/committer/uploader has the rights to submit that work under the associated license. Multiple people may add their own Signed-off-by line. But at least one Signed-off-by line
is required for every patch and GerritHub is enforcing that correctly.
“Signed off” in terms of “I sign off on this patch being ready to go into SPDK” should really be done with Reviewed-by tags.
We have an “SPDK Developers” group within the SPDK GerritHub project – anyone in this group can update patches they did not author. Both you and Ed are now in this list. If anyone else needs to be added, please
ask and the core maintainers will add you (this goes for anyone reading this message – not just John’s team!)
Thanks,
-Jim
On 5/3/18, 2:35 PM, "SPDK on behalf of Meneghini, John" <spdk-bounces@lists.01.org on behalf of
John.Meneghini@netapp.com> wrote:
OK. Thanks Jim.
I think this works now... at least for me.
Note: we might want to eliminate any confusion about what the author, vs. signer, vs. committer, vs. uploader is. I am the author of this change, Ed is the signer, and the uploader.
We're developing a team of people here at NetApp who can code review, sign, and upload each other's changes. As a matter of policy, we'd like:
- Others to sign-off the author's code (we don't want people signing their own changes)
- The author or signer to be able to upload changes
In this example Ed signed my change and uploaded it. This is unusual, normally I will upload my own changes, but we still have many engineers who don't know how to use GerritHub so Ed and I will be signing
and uploading other engineers code changes and it would be good if we have some flexibility.
In the example below I had to sign my own change to get GerritHub to take it. This is a different from the original problem which rejected my change even after I signed it.
If possible, can we fix this problem?
johnm-mac-0:spdk(add_git_hooks) > git push origin HEAD:refs/for/master
Counting objects: 6, done.
Delta compression using up to 8 threads.
Compressing objects: 100% (6/6), done.
Writing objects: 100% (6/6), 2.16 KiB | 2.16 MiB/s, done.
Total 6 (delta 2), reused 0 (delta 0)
remote: Resolving deltas: 100% (2/2)
remote: Processing changes: refs: 1, done
! [remote rejected] HEAD -> refs/for/master (not Signed-off-by author/committer/uploader in commit message footer)
error: failed to push some refs to 'https://review.gerrithub.io/spdk/spdk'
johnm-mac-0:spdk(add_git_hooks) > git commit --amend --sign
[add_git_hooks 4bfdb4fe] git: add .githooks to repository
Date: Mon Feb 5 13:15:54 2018 -0500
3 files changed, 165 insertions(+)
create mode 100755 .githooks/pre-commit
create mode 100755 .githooks/pre-push
johnm-mac-0:spdk(add_git_hooks) > git push origin HEAD:refs/for/master
Counting objects: 6, done.
Delta compression using up to 8 threads.
Compressing objects: 100% (6/6), done.
Writing objects: 100% (6/6), 2.17 KiB | 2.17 MiB/s, done.
Total 6 (delta 2), reused 0 (delta 0)
remote: Resolving deltas: 100% (2/2)
remote: Processing changes: updated: 1, done
remote:
remote: Updated Changes:
remote:
https://review.gerrithub.io/#/c/spdk/spdk/+/409577 git: add .githooks to repository
remote:
* [new branch] HEAD -> refs/for/master
On 5/2/18, 4:49 PM, "SPDK on behalf of Harris, James R" <spdk-bounces@lists.01.org on behalf of
james.r.harris@intel.com> wrote:
John – can you try it again and let me know if it works now?
On 5/2/18, 12:13 PM, "SPDK on behalf of Walker, Benjamin" <spdk-bounces@lists.01.org on behalf of
benjamin.walker@intel.com> wrote:
On Wed, 2018-05-02 at 17:59 +0000, Meneghini, John wrote:
> I’m trying to update change request: 409577
>
> And GerritHub is complaining with the following error.
> ! [remote rejected] HEAD -> refs/for/master (cannot add patch set to
> 409577.)
> error: failed to push some refs to 'https://review.gerrithub.io/spdk/spdk'
>
> Any clues as to what the problem is?
We don't allow reviews to be updated with new patch sets from people that aren't
the original owner. We did allow it originally, but some spammer/bot started
uploading junk over top of my reviews at one point. It is a permission I can
grant (I think) - I just have to figure out how to do it in a reasonable way.
It should work if Ed tries to upload the new revision though.
_______________________________________________
SPDK mailing list
_______________________________________________
SPDK mailing list
_______________________________________________
SPDK mailing list
_______________________________________________
SPDK mailing list