Re: [openstack-dev] [nova][ceilometer] FloatingIp pollster spamming n-api logs (bug 1328694)

2014-06-16 Thread Joe Gordon
On Sat, Jun 14, 2014 at 7:33 AM, Eoghan Glynn wrote: > > > - Original Message - > > On 11 June 2014 20:07, Joe Gordon wrote: > > > On Wed, Jun 11, 2014 at 11:38 AM, Matt Riedemann > > > wrote: > > >> On 6/11/2014 10:01 AM, Eoghan Glynn wrote: > > >>> Thanks for bringing this to the list

Re: [openstack-dev] [nova][ceilometer] FloatingIp pollster spamming n-api logs (bug 1328694)

2014-06-14 Thread Eoghan Glynn
- Original Message - > On 11 June 2014 20:07, Joe Gordon wrote: > > On Wed, Jun 11, 2014 at 11:38 AM, Matt Riedemann > > wrote: > >> On 6/11/2014 10:01 AM, Eoghan Glynn wrote: > >>> Thanks for bringing this to the list Matt, comments inline ... > >>> > tl;dr: some pervasive changes

Re: [openstack-dev] [nova][ceilometer] FloatingIp pollster spamming n-api logs (bug 1328694)

2014-06-14 Thread Eoghan Glynn
> On 6/12/2014 10:31 AM, John Garbutt wrote: > > On 11 June 2014 20:07, Joe Gordon wrote: > >> On Wed, Jun 11, 2014 at 11:38 AM, Matt Riedemann > >> wrote: > >>> On 6/11/2014 10:01 AM, Eoghan Glynn wrote: > Thanks for bringing this to the list Matt, comments inline ... > > > tl;dr

Re: [openstack-dev] [nova][ceilometer] FloatingIp pollster spamming n-api logs (bug 1328694)

2014-06-13 Thread Matt Riedemann
On 6/12/2014 10:31 AM, John Garbutt wrote: On 11 June 2014 20:07, Joe Gordon wrote: On Wed, Jun 11, 2014 at 11:38 AM, Matt Riedemann wrote: On 6/11/2014 10:01 AM, Eoghan Glynn wrote: Thanks for bringing this to the list Matt, comments inline ... tl;dr: some pervasive changes were made to

Re: [openstack-dev] [nova][ceilometer] FloatingIp pollster spamming n-api logs (bug 1328694)

2014-06-12 Thread John Garbutt
On 11 June 2014 20:07, Joe Gordon wrote: > On Wed, Jun 11, 2014 at 11:38 AM, Matt Riedemann > wrote: >> On 6/11/2014 10:01 AM, Eoghan Glynn wrote: >>> Thanks for bringing this to the list Matt, comments inline ... >>> tl;dr: some pervasive changes were made to nova to enable polling in

Re: [openstack-dev] [nova][ceilometer] FloatingIp pollster spamming n-api logs (bug 1328694)

2014-06-12 Thread liusheng
Matt, Eoghan, thanks Firstly , sorry for the effection, the direct reason of the bug is an issue in nova-network scenario, it is my mistake when commit patch https://review.openstack.org/#/c/81429/ to fix the bug 1262124. with agreement of Matt's view, to dissipate the load of nova API, so it

Re: [openstack-dev] [nova][ceilometer] FloatingIp pollster spamming n-api logs (bug 1328694)

2014-06-11 Thread Eoghan Glynn
> > Thanks for bringing this to the list Matt, comments inline ... > > > >> tl;dr: some pervasive changes were made to nova to enable polling in > >> ceilometer which broke some things and in my opinion shouldn't have been > >> merged as a bug fix but rather should have been a blueprint. > >> > >>

Re: [openstack-dev] [nova][ceilometer] FloatingIp pollster spamming n-api logs (bug 1328694)

2014-06-11 Thread Joe Gordon
On Wed, Jun 11, 2014 at 11:38 AM, Matt Riedemann wrote: > > > On 6/11/2014 10:01 AM, Eoghan Glynn wrote: > >> >> >> Thanks for bringing this to the list Matt, comments inline ... >> >> tl;dr: some pervasive changes were made to nova to enable polling in >>> ceilometer which broke some things and

Re: [openstack-dev] [nova][ceilometer] FloatingIp pollster spamming n-api logs (bug 1328694)

2014-06-11 Thread Matt Riedemann
On 6/11/2014 10:01 AM, Eoghan Glynn wrote: Thanks for bringing this to the list Matt, comments inline ... tl;dr: some pervasive changes were made to nova to enable polling in ceilometer which broke some things and in my opinion shouldn't have been merged as a bug fix but rather should have

Re: [openstack-dev] [nova][ceilometer] FloatingIp pollster spamming n-api logs (bug 1328694)

2014-06-11 Thread Eoghan Glynn
Thanks for bringing this to the list Matt, comments inline ... > tl;dr: some pervasive changes were made to nova to enable polling in > ceilometer which broke some things and in my opinion shouldn't have been > merged as a bug fix but rather should have been a blueprint. > > === > > The detail

[openstack-dev] [nova][ceilometer] FloatingIp pollster spamming n-api logs (bug 1328694)

2014-06-11 Thread Matt Riedemann
tl;dr: some pervasive changes were made to nova to enable polling in ceilometer which broke some things and in my opinion shouldn't have been merged as a bug fix but rather should have been a blueprint. === The detailed version: I opened bug 1328694 [1] yesterday and found that came back to s