<div dir="ltr">On Mon, May 2, 2016 at 12:38 PM, Steven Dake (stdake) <span dir="ltr"><<a href="mailto:stdake@cisco.com" target="_blank">stdake@cisco.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">Yup but that didn't happen with kolla-mesos and I didn't catch it until 2<br>
weeks after it was locked in stone.  At that point I asked for the ABI to<br>
be unified to which I got a "shrug" and no action.<br>
<br>
If it has been in one repo, everyone would have seen the multiple ABIs and<br>
rejected the patch in the first place.<br>
<br>
FWIW I am totally open to extending the ABI however is necessary to make<br>
Kolla containers be the reference that other projects use for their<br>
container deployment technology tooling.  In this case the ABI was<br>
extended without consultation and without repair after the problem was<br>
noticed.</blockquote><div><br></div><div><div><div>ABI has been mentioned a lot in either this thread or the spec code review.</div><div>Does it refer to container image only, or does it cover other part like jinja2</div><div>template for config generation as well?</div><div><br></div><div>That is the part I think need more clarification. Because even though we treat</div><div>Kubernetes as just another deployment tool, but if it still relies on Ansible to</div><div>generate configuations (as proposed in the spec[1]), then there's no clean way</div><div>to centralize all Kube related stuff in separate repo.</div><div><br></div><div>If we're going to re-use Kolla's jinja2 templates and ini merging (which is</div><div>heavily depends on Ansible module as of now), I think practically it is easiser</div><div>to bootstrap Kubernetes stuff in the same Kolla repo. But other than that, I'm</div><div>in favor of separate Kolla-kubernetes repo.</div><div><br></div><div>[1] <a href="https://review.openstack.org/#/c/304182">https://review.openstack.org/#/c/304182</a></div></div></div><div><br></div><div>QY</div><div><br></div></div></div></div>