FloatingIp pollster spamming n-api logs

Bug #1328694 reported by Matt Riedemann on 2014-06-10
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ceilometer
Fix Released
Undecided
Matt Riedemann
OpenStack Compute (nova)
Medium
Matt Riedemann

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

Changed in ceilometer:
assignee: nobody → Matt Riedemann (mriedem)
status: New → In Progress
Liusheng (liusheng) on 2014-06-11
affects: ceilometer → nova-project
no longer affects: ceilometer
affects: nova-project → ceilometer
Matt Riedemann (mriedem) wrote :
Changed in nova:
assignee: nobody → Liusheng (liusheng)
status: New → In Progress
Michael Still (mikal) on 2014-06-12
tags: added: network
Changed in nova:
importance: Undecided → Medium

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

Changed in ceilometer:
status: In Progress → Fix Committed
Thierry Carrez (ttx) on 2014-06-12
Changed in ceilometer:
milestone: none → juno-1
status: Fix Committed → Fix Released
Matt Riedemann (mriedem) wrote :

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

Changed in nova:
assignee: Liusheng (liusheng) → Matt Riedemann (mriedem)
milestone: none → juno-2

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

Changed in nova:
status: In Progress → Fix Committed

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

Changed in nova:
status: Fix Committed → Fix Released
Thierry Carrez (ttx) on 2014-10-16
Changed in nova:
milestone: juno-2 → 2014.2
Thierry Carrez (ttx) on 2014-10-16
Changed in ceilometer:
milestone: juno-1 → 2014.2
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers