<div dir="ltr">Kyle,<div><br></div><div style>I actually meant that this problem might occur if patchset 2, that you're trying to push, is a rebase of patchset 1 on top of another patch in order to make the commit dependent on another one. If that is the case, gerrit won't see any difference between patchset2 and patchset1, as git diff HEAD~1 would be the same in both cases.<br>
</div><div style><br></div><div style>If this however is not your problem just disregard this post.</div><div style><br></div><div style>Salvatore</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On 2 July 2013 23:29, Kyle Mestery (kmestery) <span dir="ltr"><<a href="mailto:kmestery@cisco.com" target="_blank">kmestery@cisco.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On Jul 2, 2013, at 4:18 PM, Salvatore Orlando <<a href="mailto:sorlando@nicira.com">sorlando@nicira.com</a>> wrote:<br>

><br>
> Kyle,<br>
><br>
> is this commit basically a rebase on top of 91e0850?<br>
> In that case the diff with the previous patchset would be empty.<br>
> I recall I had a similar issue; I just tweaked a comment line in my commit to let gerrit think it was a different patchset.<br>
><br>
> Salvatore<br>
><br>
</div>Hi Salvatore:<br>
<br>
Actually, no. 91e0850 is actually the implementation for bp/ml2-gre,<br>
and my commit implements bp/ml2-vxlan. Since there was some<br>
shared code, the first commit implemented that, thus requiring my<br>
commit to be dependent on this other commit.<br>
<br>
Thanks,<br>
Kyle<br>
<div class="HOEnZb"><div class="h5"><br>
><br>
> On 2 July 2013 23:05, Kyle Mestery (kmestery) <<a href="mailto:kmestery@cisco.com">kmestery@cisco.com</a>> wrote:<br>
> On Jul 2, 2013, at 3:52 PM, Jeremy Stanley <<a href="mailto:fungi@yuggoth.org">fungi@yuggoth.org</a>><br>
>  wrote:<br>
> > On 2013-07-02 20:14:35 +0000 (+0000), Kyle Mestery (kmestery) wrote:<br>
> >> I'm trying to submit a gerrit review for a commit which is<br>
> >> dependent on another person's commit [1].<br>
> > [...]<br>
> >> ! [remote rejected] HEAD -> refs/publish/master/bp/ml2-vxlan (no changes made)<br>
> > [...]<br>
> ><br>
> > I want to say I've seen this recently when someone copy-pasted from<br>
> > one commit message to another and included an existing Change-Id<br>
> > header, so you might amend and clear that line from the top commit<br>
> > before reviewing again (it'll get regenerated fresh).<br>
> ><br>
> Thanks Jeremy, this was it! For some reason, the top commit had the<br>
> same Change-ID as the bottom commit. Once I cleared it, I was able to<br>
> push, though with the issue seen here:<br>
><br>
> [kmestery@fedora-mac quantum]$ git review -R<br>
> You have more than one commit that you are about to submit.<br>
> The outstanding commits are:<br>
><br>
> 6cca2cf (HEAD, bp/ml2-vxlan) Add VXLAN tunneling support for the ML2 plugin<br>
> 91e0850 Add gre tunneling support for the ML2 plugin<br>
><br>
> Is this really what you meant to do?<br>
> Type 'yes' to confirm: yes<br>
> Enter passphrase for key '/home/kmestery/.ssh/id_rsa':<br>
> remote: Resolving deltas: 100% (32/32)<br>
> remote: Processing changes: new: 1, updated: 1, refs: 1, done<br>
> remote:<br>
> remote: New Changes:<br>
> remote:   <a href="https://review.openstack.org/35384" target="_blank">https://review.openstack.org/35384</a><br>
> remote:<br>
> To ssh://<a href="http://mestery@review.openstack.org:29418/openstack/quantum.git" target="_blank">mestery@review.openstack.org:29418/openstack/quantum.git</a><br>
>  ! [remote rejected] HEAD -> refs/publish/master/bp/ml2-vxlan (no changes made)<br>
> error: failed to push some refs to 'ssh://<a href="http://mestery@review.openstack.org:29418/openstack/quantum.git" target="_blank">mestery@review.openstack.org:29418/openstack/quantum.git</a>'<br>
> [kmestery@fedora-mac quantum]$<br>
><br>
> In addition, my commit seen at the review URL above does not show<br>
> the dependancy. Any ideas now?<br>
><br>
> Thanks!<br>
> Kyle<br>
><br>
> > If that doesn't work, could you push your working branch to<br>
> > somewhere I could pull from so I can test it myself? Feel free to<br>
> > follow up with me in private or open a bug against git-review on<br>
> > Launchpad if you don't want to run through troubleshooting<br>
> > back-and-forth on the list.o<br>
><br>
><br>
><br>
><br>
> _______________________________________________<br>
> OpenStack-dev mailing list<br>
> <a href="mailto:OpenStack-dev@lists.openstack.org">OpenStack-dev@lists.openstack.org</a><br>
> <a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
><br>
> _______________________________________________<br>
> OpenStack-dev mailing list<br>
> <a href="mailto:OpenStack-dev@lists.openstack.org">OpenStack-dev@lists.openstack.org</a><br>
> <a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
<br>
<br>
<br>
_______________________________________________<br>
OpenStack-dev mailing list<br>
<a href="mailto:OpenStack-dev@lists.openstack.org">OpenStack-dev@lists.openstack.org</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
</div></div></blockquote></div><br></div>