Re: [openstack-dev] Revert Pass instance host-id to Quantum using port bindings extension.

2013-07-23 Thread Vishvananda Ishaya
On Jul 19, 2013, at 3:37 PM, Ian Wells wrote: >> [arosen] - sure, in this case though then we'll have to add even more >> queries between nova-compute and quantum as nova-compute will need to query >> quantum for ports matching the device_id to see if the port was already >> created and if not t

Re: [openstack-dev] Revert Pass instance host-id to Quantum using port bindings extension.

2013-07-19 Thread Kyle Mestery (kmestery)
On Jul 19, 2013, at 6:01 PM, Aaron Rosen wrote: > > On Fri, Jul 19, 2013 at 3:37 PM, Ian Wells wrote: > > [arosen] - sure, in this case though then we'll have to add even more > > queries between nova-compute and quantum as nova-compute will need to query > > quantum for ports matching the devic

Re: [openstack-dev] Revert Pass instance host-id to Quantum using port bindings extension.

2013-07-19 Thread Aaron Rosen
On Fri, Jul 19, 2013 at 3:37 PM, Ian Wells wrote: > > [arosen] - sure, in this case though then we'll have to add even more > > queries between nova-compute and quantum as nova-compute will need to > query > > quantum for ports matching the device_id to see if the port was already > > created and

Re: [openstack-dev] Revert Pass instance host-id to Quantum using port bindings extension.

2013-07-19 Thread Ian Wells
> [arosen] - sure, in this case though then we'll have to add even more > queries between nova-compute and quantum as nova-compute will need to query > quantum for ports matching the device_id to see if the port was already > created and if not try to create them. The cleanup job doesn't look like

Re: [openstack-dev] Revert Pass instance host-id to Quantum using port bindings extension.

2013-07-19 Thread Aaron Rosen
On Fri, Jul 19, 2013 at 8:47 AM, Kyle Mestery (kmestery) wrote: > On Jul 18, 2013, at 5:16 PM, Aaron Rosen wrote: > > > > Hi, > > > > I wanted to raise another design failure of why creating the port on > nova-compute is bad. Previously, we have encountered this bug ( > https://bugs.launchpad.ne

Re: [openstack-dev] Revert Pass instance host-id to Quantum using port bindings extension.

2013-07-19 Thread Kyle Mestery (kmestery)
On Jul 19, 2013, at 1:58 PM, Aaron Rosen wrote: > > > > > On Fri, Jul 19, 2013 at 8:47 AM, Kyle Mestery (kmestery) > wrote: > On Jul 18, 2013, at 5:16 PM, Aaron Rosen wrote: > > > > Hi, > > > > I wanted to raise another design failure of why creating the port on > > nova-compute is bad. Pr

Re: [openstack-dev] Revert Pass instance host-id to Quantum using port bindings extension.

2013-07-19 Thread Aaron Rosen
On Fri, Jul 19, 2013 at 1:11 PM, Kyle Mestery (kmestery) wrote: > On Jul 19, 2013, at 1:58 PM, Aaron Rosen wrote: > > > > > > > > > > On Fri, Jul 19, 2013 at 8:47 AM, Kyle Mestery (kmestery) < > kmest...@cisco.com> wrote: > > On Jul 18, 2013, at 5:16 PM, Aaron Rosen wrote: > > > > > > Hi, > > >

Re: [openstack-dev] Revert Pass instance host-id to Quantum using port bindings extension.

2013-07-19 Thread Kyle Mestery (kmestery)
On Jul 18, 2013, at 5:16 PM, Aaron Rosen wrote: > > Hi, > > I wanted to raise another design failure of why creating the port on > nova-compute is bad. Previously, we have encountered this bug > (https://bugs.launchpad.net/neutron/+bug/1160442). What was causing the issue > was that when nov

Re: [openstack-dev] Revert Pass instance host-id to Quantum using port bindings extension.

2013-07-19 Thread Ian Wells
> I wanted to raise another design failure of why creating the port on > nova-compute is bad. Previously, we have encountered this bug > (https://bugs.launchpad.net/neutron/+bug/1160442). What was causing the > issue was that when nova-compute calls into quantum to create the port; > quantum create

Re: [openstack-dev] Revert Pass instance host-id to Quantum using port bindings extension.

2013-07-18 Thread Aaron Rosen
Hi, I wanted to raise another design failure of why creating the port on nova-compute is bad. Previously, we have encountered this bug ( https://bugs.launchpad.net/neutron/+bug/1160442). What was causing the issue was that when nova-compute calls into quantum to create the port; quantum creates th

Re: [openstack-dev] Revert Pass instance host-id to Quantum using port bindings extension.

2013-07-15 Thread Aaron Rosen
On Mon, Jul 15, 2013 at 1:26 PM, Robert Kukura wrote: > On 07/15/2013 03:54 PM, Aaron Rosen wrote: > > > > > > > > On Sun, Jul 14, 2013 at 6:48 PM, Robert Kukura > > wrote: > > > > On 07/12/2013 04:17 PM, Aaron Rosen wrote: > > > Hi, > > > > > > > >

Re: [openstack-dev] Revert Pass instance host-id to Quantum using port bindings extension.

2013-07-15 Thread Robert Kukura
On 07/15/2013 03:54 PM, Aaron Rosen wrote: > > > > On Sun, Jul 14, 2013 at 6:48 PM, Robert Kukura > wrote: > > On 07/12/2013 04:17 PM, Aaron Rosen wrote: > > Hi, > > > > > > On Fri, Jul 12, 2013 at 6:47 AM, Robert Kukura

Re: [openstack-dev] Revert Pass instance host-id to Quantum using port bindings extension.

2013-07-15 Thread Aaron Rosen
On Sun, Jul 14, 2013 at 6:48 PM, Robert Kukura wrote: > On 07/12/2013 04:17 PM, Aaron Rosen wrote: > > Hi, > > > > > > On Fri, Jul 12, 2013 at 6:47 AM, Robert Kukura > > wrote: > > > > On 07/11/2013 04:30 PM, Aaron Rosen wrote: > > > Hi, > > > > > > I t

Re: [openstack-dev] Revert Pass instance host-id to Quantum using port bindings extension.

2013-07-14 Thread Robert Kukura
On 07/12/2013 04:17 PM, Aaron Rosen wrote: > Hi, > > > On Fri, Jul 12, 2013 at 6:47 AM, Robert Kukura > wrote: > > On 07/11/2013 04:30 PM, Aaron Rosen wrote: > > Hi, > > > > I think we should revert this patch that was added here > > (https://revi

Re: [openstack-dev] Revert Pass instance host-id to Quantum using port bindings extension.

2013-07-12 Thread Aaron Rosen
Hi, On Fri, Jul 12, 2013 at 6:47 AM, Robert Kukura wrote: > On 07/11/2013 04:30 PM, Aaron Rosen wrote: > > Hi, > > > > I think we should revert this patch that was added here > > (https://review.openstack.org/#/c/29767/). What this patch does is when > > nova-compute calls into quantum to creat

Re: [openstack-dev] Revert Pass instance host-id to Quantum using port bindings extension.

2013-07-12 Thread Robert Kukura
On 07/11/2013 04:30 PM, Aaron Rosen wrote: > Hi, > > I think we should revert this patch that was added here > (https://review.openstack.org/#/c/29767/). What this patch does is when > nova-compute calls into quantum to create the port it passes in the > hostname on which the instance was booted

Re: [openstack-dev] Revert Pass instance host-id to Quantum using port bindings extension.

2013-07-11 Thread Sumit Naiksatam
I agree with Andre and Kyle here. I am not sure that the polling option is even going to work for certain use cases where the host_id information is required when creating the port (for instance, to decide the VIF type). Thanks, ~Sumit. On Thu, Jul 11, 2013 at 7:27 PM, Kyle Mestery (kmestery) wr

Re: [openstack-dev] Revert Pass instance host-id to Quantum using port bindings extension.

2013-07-11 Thread Kyle Mestery (kmestery)
I agree with Andre's concerns around the implications of polling in what Aaron is proposing, and in fact, this is one reason the existing change is so nice. The ML2 sub-team talked about this at a recent meeting, and we liked the approach which Yong had taken with the patch. But as Andre says, w

Re: [openstack-dev] Revert Pass instance host-id to Quantum using port bindings extension.

2013-07-11 Thread Andre Pech
Hey Aaron, As an interested party in the change, figured I'd take a stab at responding. I've talked with people at BigSwitch and Cisco about this change, so I know others are interested in this as well, but I'll let them give their perspective. At a high level, our goal at Arista is similar to wh

[openstack-dev] Revert Pass instance host-id to Quantum using port bindings extension.

2013-07-11 Thread Aaron Rosen
Hi, I think we should revert this patch that was added here ( https://review.openstack.org/#/c/29767/). What this patch does is when nova-compute calls into quantum to create the port it passes in the hostname on which the instance was booted on. The idea of the patch was that providing this infor