<div dir="ltr"><div><span style="font-family:arial,sans-serif;font-size:13px">Hi Mark, </span></div><span style="font-family:arial,sans-serif;font-size:13px"><div><span style="font-family:arial,sans-serif;font-size:13px"><br>
</span></div></span><span style="font-family:arial,sans-serif;font-size:13px">>> Sounds good. Just needs someone willing to implement it.</span><div><span style="font-family:arial,sans-serif;font-size:13px"><br></span></div>
<div><span style="font-family:arial,sans-serif;font-size:13px">This is very interesting for us.</span></div><div><span style="font-family:arial,sans-serif;font-size:13px">What do you think if we create a fake project that use submodules as an example, and then just discuss it? <br>
</span><div><font face="arial, sans-serif"><br></font><div><span style="font-family:arial,sans-serif;font-size:13px"><br></span></div><div><span style="font-family:arial,sans-serif;font-size:13px">Best regards,</span></div>
</div></div><div><span style="font-family:arial,sans-serif;font-size:13px">Boris Pavlovic</span></div><div><font face="arial, sans-serif">--</font></div><div><font face="arial, sans-serif">Mirantis Inc. </font></div><div>
<span style="font-family:arial,sans-serif;font-size:13px"><br></span></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Aug 8, 2013 at 3:21 PM, Mark McLoughlin <span dir="ltr"><<a href="mailto:markmc@redhat.com" target="_blank">markmc@redhat.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 Thu, 2013-08-08 at 15:11 +0400, Boris Pavlovic wrote:<br>
> Mark,<br>
><br>
><br>
> >> What do you mean by "dangerous code merging" in the subject? The<br>
> body of<br>
> >> your mail doesn't make any reference to whatever "danger" you're<br>
> seeing.<br>
><br>
><br>
><br>
> I mean that cut and paste approach is really unsafe. For example some<br>
> new member is able to change oslo code directly during syncing with<br>
> some project,<br>
>  and nobody will be able to catch such things.<br>
><br>
><br>
> I didn't catch any of such situation, but I saw a lot of attempts to<br>
> change openstack/common/* directly.<br>
> (and it is really close situation..)<br>
<br>
</div>Got examples?<br>
<br>
It's a reviewer's responsibility to enforce that openstack/commmon/*<br>
code is just synced from oslo-incubator without modifications, but<br>
there's always an element of trust in reviewing - you can't completely<br>
guard against people doing dumb or nefarious things.<br>
<br>
We could come up with some automation where the oslo-incubator git<br>
commit ID corresponding to each file is included in each project's repo<br>
and a test checks that the file does in fact correspond to that commit<br>
ID. Needs someone willing to implement it.<br>
<div class="im"><br>
> >> The idea of using submodules has come a few times. I don't have a<br>
> >> fundamental objection to it, except any time I've seen submodules<br>
> used<br>
> >> in a project they've been extremely painful for everyone involved.<br>
><br>
><br>
><br>
> oslo-incubator sync util and submodules solves the same problem,<br>
> almost in same way:<br>
> sync util -> copy paste code from <hash><br>
> submodules -> just set <hash> of commit from what to use code<br>
><br>
><br>
> So I think the problem is not in submodules, problem is in approach of<br>
> common code for different projects.<br>
> But IMHO it is much better to have problems around creating common<br>
> code that is used by all projects, then to make<br>
> N different solutions for N different projects doing almost the same<br>
> things.<br>
><br>
><br>
><br>
><br>
> >> I'd be happy to look at a demo of a submodule based system for<br>
> projects<br>
> >> to use code from oslo-incubator.<br>
><br>
><br>
><br>
> Probably we should just try, and analyze what approach is better?<br>
<br>
</div>Sounds good. Just needs someone willing to implement it.<br>
<br>
Cheers,<br>
Mark.<br>
<br>
<br>
</blockquote></div><br></div>