<div dir="ltr">Hi,<br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Dec 3, 2013 at 7:21 PM, Robert Collins <span dir="ltr"><<a href="mailto:robertc@robertcollins.net" target="_blank">robertc@robertcollins.net</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">No, you need to manually arrange to land your changes in first one<br>
repo then the other.<br>
<br>
-Rob<br>
<div><div class="h5"><br>
On 3 December 2013 19:17, wu jiang <<a href="mailto:wingwj@gmail.com">wingwj@gmail.com</a>> wrote:<br>
> Hi all,<br>
><br>
> Recently, I found a bug at API layer in Cinder, but the modifications relate<br>
> to CinderClient & Tempest.<br>
> So, I'm confused how to commit it. Can 'git --dependence' cross different<br>
> Repo?<br>
</div></div></blockquote><div><br><br></div><div>Just to expand on what Rob mentioned - for situations like this its pretty common<br></div><div>not to be able to just say make a change in cinder first because the tempest test will<br>
fail. And at the same time you can't make the final change in tempest first because the<br>cinder change hasn't applied yet. If this is your situation what I'd suggest you do is:<br><br></div><div>- Submit the cinder change, it will fail tempest, that's ok, you might want to leave a comment<br>
saying you are waiting on a tempest change to land first.<br></div><div><br>- Submit a tempest change which disables the tempest tests which fail because of your change<br>and in the commit message reference the gerrit url for the cinder change. Its very helpful, but not<br>
always necessary if you can get a cinder core to +1 this patch so the tempest cores know that the<br>cinder team is happy with what is going on.<br></div><div><br></div><div>- Once the tempest change has merged, the cinder change should be able to merge.<br>
<br></div><div>- Submit a tempest change enabling the disabled test(s) and modifying it for the expected behaviour.<br><br></div><div>Yes, it would be nice to be able to have cross project dependent patches :)<br></div><div>
<br></div><div>Chris<br></div><div> <br></div></div></div></div>