fix full synchronization between neutron and ODL

Bug #1371115 reported by Cédric OLLIVIER
16
This bug affects 3 people
Affects Status Importance Assigned to Milestone
networking-odl
Invalid
High
Isaku Yamahata

Bug Description

ODL MD doesn't treat the pending operation in sync_full().
It induces a desynchronization in case of an update and a delete operation triggering the full sync.
To reproduce this desynchronisation:
  - create a network
  - restart neutron
  - delete or update this network

When neutron and ODL aren't synchronized, all the update and delete operations are also lost (from the ODL point of view)
To reproduce the test case:
  - create a network
  - set wrong credentials in ml2_conf_odl.ini and restart neutron
  - delete or update this network
  - set the correct credentials in ml2_conf_odl.ini and restart neutron
  - trigger the full synchronization via any operation

ODL MD will also send POST requests with empty data (ie. {'networks': []}) to ODL if all resources (UUID) previously exist in ODL.

Cédric OLLIVIER (m.col)
Changed in neutron:
assignee: nobody → Cédric OLLIVIER (m.col)
Changed in neutron:
importance: Undecided → Medium
Cédric OLLIVIER (m.col)
tags: added: juno-rc-potential
Thierry Carrez (ttx)
tags: added: juno-backport-potential
removed: juno-rc-potential
Cédric OLLIVIER (m.col)
affects: neutron → networking-odl
Kyle Mestery (mestery)
Changed in networking-odl:
milestone: none → 0.1.1
Cédric OLLIVIER (m.col)
affects: networking-odl → neutron
Changed in neutron:
milestone: 0.1.1 → none
affects: neutron → networking-odl
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to networking-odl (master)

Related fix proposed to branch: master
Review: https://review.openstack.org/202834

Kyle Mestery (mestery)
Changed in networking-odl:
milestone: none → 1.0.0
importance: Medium → High
status: New → In Progress
assignee: Cédric OLLIVIER (m.col) → Isaku Yamahata (yamahata)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to networking-odl (master)

Reviewed: https://review.openstack.org/202834
Committed: https://git.openstack.org/cgit/openstack/networking-odl/commit/?id=6537676b899192b7fa96789869adce72ec8f7986
Submitter: Jenkins
Branch: master

commit 6537676b899192b7fa96789869adce72ec8f7986
Author: Isaku Yamahata <email address hidden>
Date: Thu Jul 16 15:08:32 2015 -0700

    mech_driver: full_sync uses unrelated context for resources

    context passed to full sync is unrelated to the resource to be synced.
    When necessary, the context needs to be created for the resource

    Change-Id: I3d1842c2c0c95bdbaa689727a36c66e210f6273a
    Related-Bug: #1371115
    Closes-Bug: #1475450

Kyle Mestery (mestery)
Changed in networking-odl:
milestone: 1.0.0 → none
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to networking-odl (stable/kilo)

Related fix proposed to branch: stable/kilo
Review: https://review.openstack.org/225191

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to networking-odl (stable/kilo)

Reviewed: https://review.openstack.org/225191
Committed: https://git.openstack.org/cgit/openstack/networking-odl/commit/?id=869d97a11ed80059ae5048b6e7acf06fea231031
Submitter: Jenkins
Branch: stable/kilo

commit 869d97a11ed80059ae5048b6e7acf06fea231031
Author: Isaku Yamahata <email address hidden>
Date: Thu Jul 16 15:08:32 2015 -0700

    mech_driver: full_sync uses unrelated context for resources

    context passed to full sync is unrelated to the resource to be synced.
    When necessary, the context needs to be created for the resource

    Change-Id: I3d1842c2c0c95bdbaa689727a36c66e210f6273a
    Related-Bug: #1371115
    Closes-Bug: #1475450
    (cherry picked from commit 6537676b899192b7fa96789869adce72ec8f7986)

tags: added: in-stable-kilo
Revision history for this message
Isaku Yamahata (yamahata) wrote :

This issue is being addressed by v2driver. and the legacy existing driver is being deprecated.
So marking invalid.

Changed in networking-odl:
status: In Progress → Invalid
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.