<div dir="ltr">Check out my comments on the review. Only Neutron knows whether or not an instance needs to do manual tagging based on the plugin/driver loaded.<div><br></div><div>For example, Ironic/bare metal ports can be bound by neutron with a correct driver so they shouldn't get the VLAN information at the instance level in those cases. Nova has no way to know whether Neutron is configured this way so Neutron should have an explicit response in the port binding information indicating that an instance needs to tag.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jul 17, 2015 at 9:51 AM, Jim Rollenhagen <span dir="ltr"><<a href="mailto:jim@jimrollenhagen.com" target="_blank">jim@jimrollenhagen.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="HOEnZb"><div class="h5">On Fri, Jul 17, 2015 at 01:06:46PM +0100, John Garbutt wrote:<br>
> On 17 July 2015 at 11:23, Sean Dague <<a href="mailto:sean@dague.net">sean@dague.net</a>> wrote:<br>
> > On 07/16/2015 06:06 PM, Sean M. Collins wrote:<br>
> >> On Thu, Jul 16, 2015 at 01:23:29PM PDT, Mathieu Gagné wrote:<br>
> >>> So it looks like there is a missing part in this feature. There should<br>
> >>> be a way to "hide" this information if the instance does not require to<br>
> >>> configure vlan interfaces to make network functional.<br>
> >><br>
> >> I just commented on the review, but the provider network API extension<br>
> >> is admin only, most likely for the reasons that I think someone has<br>
> >> already mentioned, that it exposes details of the phyiscal network<br>
> >> layout that should not be exposed to tenants.<br>
> ><br>
> > So, clearly, under some circumstances the network operator wants to<br>
> > expose this information, because there was the request for that feature.<br>
> > The question in my mind is what circumstances are those, and what<br>
> > additional information needs to be provided here.<br>
> ><br>
> > There is always a balance between the private cloud case which wants to<br>
> > enable more self service from users (and where the users are often also<br>
> > the operators), and the public cloud case where the users are outsiders<br>
> > and we want to hide as much as possible from them.<br>
> ><br>
> > For instance, would an additional attribute on a provider network that<br>
> > says "this is cool to tell people about" be an acceptable approach? Is<br>
> > there some other creative way to tell our infrastructure that these<br>
> > artifacts are meant to be exposed in this installation?<br>
> ><br>
> > Just kicking around ideas, because I know a pile of gate hardware for<br>
> > everyone to use is at the other side of answers to these questions. And<br>
> > given that we've been running full capacity for days now, keeping this<br>
> > ball moving forward would be great.<br>
><br>
> Maybe we just need to add policy around who gets to see that extra<br>
> detail, and maybe hide it by default?<br>
><br>
> Would that deal with the concerns here?<br>
<br>
</div></div>I'm not so sure. There are certain Neutron plugins that work with<br>
certain virt drivers (Ironic) that require this information to be passed<br>
to all instances built by that virt driver. However, it doesn't (and<br>
probably shouldn't, as to not confuse cloud-init/etc) need to be passed<br>
to other instances. I think the conditional for passing this as metadata<br>
is going to need to be some combination of operator config, Neutron<br>
config/driver, and virt driver.<br>
<br>
I know we don't like networking things to be conditional on the virt<br>
driver, but Ironic is working on feature parity with virt for<br>
networking, and baremetal networking is vastly different than virt<br>
networking. I think we're going to have to accept that.<br>
<span class="HOEnZb"><font color="#888888"><br>
// jim<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
><br>
> Thanks,<br>
> John<br>
><br>
> __________________________________________________________________________<br>
> OpenStack Development Mailing List (not for usage questions)<br>
> Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br>
> <a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
<br>
__________________________________________________________________________<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div>Kevin Benton</div></div>
</div>