Networking: smarter-network-cache-update

Registered by Aaron Rosen on 2013-12-12

Currently, we update the network_cache whether we need to or not. We should add more logic to only update the cache if needed. This is important as the periodic healer is always updating the cache.

Note that this blueprint has no end user or documentation impact.

Blueprint information

Status:
Started
Approver:
Russell Bryant
Priority:
Medium
Drafter:
None
Direction:
Needs approval
Assignee:
Aaron Rosen
Definition:
Review
Series goal:
None
Implementation:
Needs Code Review
Milestone target:
milestone icon next
Started by
Russell Bryant on 2013-12-17

Related branches

Sprints

Whiteboard

Seems like this is all up for review, setting to Needs Code Review, not sure how this interacts with the info_cache callback, it seems complimentary, at a quick glance --johnthetubaguy

Gerrit topic: https://review.openstack.org/#q,topic:bp/smarter-network-cache-update,n,z

Addressed by: https://review.openstack.org/62104
    Correct network_model tests and __eq__ operator

Addressed by: https://review.openstack.org/61871
    Remove unused variables in neutron api interface and neutron tests

Addressed by: https://review.openstack.org/60396
    Remove unneeded call to conductor in network interface

Addressed by: https://review.openstack.org/60592
    Make network_cache more robust with neutron

Addressed by: https://review.openstack.org/62108
    Make network_cache more smarter (WIP)

Gerrit topic: https://review.openstack.org/#q,topic:bp/nova-api-quantum-create-port,n,z

Addressed by: https://review.openstack.org/64769
    Garbage collect neutron ports not in nw_cache

Addressed by: https://review.openstack.org/73422
    Remove unneeded call to conductor in network interface

Gerrit topic: https://review.openstack.org/#q,topic:master,n,z

This is really a bug fixing, moving to medium so it sits next to the other exceptions --johnthetubaugy

Addressed by: https://review.openstack.org/78552
    Make @refresh_cache decorator take update_cells argument

Merged:
======

https://review.openstack.org/#/c/61871/
https://review.openstack.org/#/c/60396/
https://review.openstack.org/#/c/60592/
https://review.openstack.org/#/c/62104/
https://review.openstack.org/#/c/73422/

Pending:
=======

https://review.openstack.org/#/c/62108/
https://review.openstack.org/#/c/64769/
https://review.openstack.org/#/c/78552/

Gerrit topic: https://review.openstack.org/#q,topic:bug/1252410,n,z

Unapproved - please re-submit via nova-spec --johnthetubagy (20th March 2014)

(?)

Work Items

This blueprint contains Public information 
Everyone can see this information.