FloatingIp pollster spamming n-api logs

[Matt Riedemann]

Noticed this here:

http://logs.openstack.org/02/99002/1/check/check-tempest-dsvm-full/5fca6a7/logs/screen-n-api.txt.gz?level=TRACE

Showing up a ton since 6/8:

http://logstash.openstack.org/#eyJzZWFyY2giOiJtZXNzYWdlOlwiQ2F1Z2h0IGVycm9yOiBQYXJlbnQgaW5zdGFuY2VcIiBBTkQgbWVzc2FnZTpcImlzIG5vdCBib3VuZCB0byBhIFNlc3Npb247IGxhenkgbG9hZCBvcGVyYXRpb24gb2YgYXR0cmlidXRlICdmaXhlZF9pcCcgY2Fubm90IHByb2NlZWRcIiBBTkQgdGFnczpcInNjcmVlbi1uLWFwaS50eHRcIiIsImZpZWxkcyI6W10sIm9mZnNldCI6MCwidGltZWZyYW1lIjoiNjA0ODAwIiwiZ3JhcGhtb2RlIjoiY291bnQiLCJ0aW1lIjp7InVzZXJfaW50ZXJ2YWwiOjB9LCJzdGFtcCI6MTQwMjQzNDU3MTcwNX0=

Assuming this is the culprit given it merged on 6/8 and is related to floating ip's which is what shows up in that n-api log mess.

https://review.openstack.org/83676

[Matt Riedemann (mriedem)]

I looked at the n-api logs of a change before https://review.openstack.org/83676 merged and the spam isn't there:

http://logs.openstack.org/30/95830/1/check/check-tempest-dsvm-full/51812b2/logs/screen-n-api.txt.gz?level=TRACE

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.openstack.org/99212

[Matt Riedemann (mriedem)]

Nova patch is here: https://review.openstack.org/#/c/99251/

[Matt Riedemann (mriedem)]

Mailing list thread on this issue:

http://lists.openstack.org/pipermail/openstack-dev/2014-June/037304.html

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/99212
Committed: https://git.openstack.org/cgit/openstack/ceilometer/commit/?id=e7bd5eb0d401903944bb9ca3fa60e4874e3ad7a4
Submitter: Jenkins
Branch: master

commit e7bd5eb0d401903944bb9ca3fa60e4874e3ad7a4
Author: Matt Riedemann <email address hidden>
Date: Tue Jun 10 21:26:47 2014 +0000

    Revert "Fix the floatingip pollster"

    This reverts commit 8057b66c0b7ac87863f175d850414bc0ed260ab2.

    This is spamming the nova-api logs in tempest runs.

    Change-Id: Iafe20bbab45bff62470374eba2a6e0e4fa3be611
    Closes-Bug: #1328694

[Matt Riedemann (mriedem)]

Nova API change revert is here: https://review.openstack.org/#/c/99938/

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/99938
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=801a37de197cdd98f264caa2bbe9c6ebddec070b
Submitter: Jenkins
Branch: master

commit 801a37de197cdd98f264caa2bbe9c6ebddec070b
Author: Matt Riedemann <email address hidden>
Date: Fri Jun 13 13:30:17 2014 +0000

    Revert "Allow admin user to get all tenant's floating IPs"

    This reverts commit 6478554f531f6ee2fa86226fbc79dd31e556bc06.

    This API change merged as a bug fix and introduced bugs and our API
    change policy is more or less to go through the blueprint/nova-specs
    review process:

    http://lists.openstack.org/pipermail/openstack-dev/2014-June/037536.html

    So reverting this so it can go through the blueprint/nova-specs
    review process.

    There should be minimal impacts to removing this (broken) API since
    it merged not too long ago and was added strictly for Ceilometer's
    usage, not an external deployer that would be (more) broken by this
    API removal.

    Closes-Bug: #1328694

    Change-Id: I22a711795ea616aabc155a71703c9c794e84a5dc

[OpenStack Infra (hudson-openstack)]

Change abandoned by liusheng (<email address hidden>) on branch: master
Review: https://review.openstack.org/99251