[Yahoo-eng-team] [Bug 1373817] Re: ODL MD should not reraise ODL exceptions as they are

2015-01-27 Thread Cédric OLLIVIER
** Project changed: neutron => networking-odl

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to neutron.
https://bugs.launchpad.net/bugs/1373817

Title:
  ODL MD should not reraise ODL exceptions as they are

Status in OpenDaylight backend controller integration with Neutron:
  Confirmed

Bug description:
  ODL MD reraised errors returned by ODL in single operations and in full 
synchronization.
  In both cases, ODL MD could raise more adapted errors (i.e. server errors 
instead of client errors).

To manage notifications about this bug go to:
https://bugs.launchpad.net/networking-odl/+bug/1373817/+subscriptions

-- 
Mailing list: https://launchpad.net/~yahoo-eng-team
Post to : yahoo-eng-team@lists.launchpad.net
Unsubscribe : https://launchpad.net/~yahoo-eng-team
More help   : https://help.launchpad.net/ListHelp


[Yahoo-eng-team] [Bug 1376169] Re: ODL MD can't reconnect to ODL after it restarts

2015-01-27 Thread Cédric OLLIVIER
** Project changed: neutron => networking-odl

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to neutron.
https://bugs.launchpad.net/bugs/1376169

Title:
  ODL MD can't reconnect to ODL after it restarts

Status in OpenDaylight backend controller integration with Neutron:
  New

Bug description:
  ODL MD doesn't process any traitment when it receives 401 http error 
(Unauthorized) which happens after ODL restarts.
  The only way to recover is to restart neutron. 

  It induces a strongly link between restarts of neutron and ODL.

  To reproduce it:
   - start ODL and neutron
   - create a network
   - restart ODL
   - create another network

  The last command raises an 401 http error and any extra single
  operations will fail.

To manage notifications about this bug go to:
https://bugs.launchpad.net/networking-odl/+bug/1376169/+subscriptions

-- 
Mailing list: https://launchpad.net/~yahoo-eng-team
Post to : yahoo-eng-team@lists.launchpad.net
Unsubscribe : https://launchpad.net/~yahoo-eng-team
More help   : https://help.launchpad.net/ListHelp


[Yahoo-eng-team] [Bug 1371115] Re: fix full synchronization between neutron and ODL

2015-01-27 Thread Cédric OLLIVIER
** Project changed: neutron => networking-odl

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to neutron.
https://bugs.launchpad.net/bugs/1371115

Title:
  fix full synchronization between neutron and ODL

Status in OpenDaylight backend controller integration with Neutron:
  New

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.

To manage notifications about this bug go to:
https://bugs.launchpad.net/networking-odl/+bug/1371115/+subscriptions

-- 
Mailing list: https://launchpad.net/~yahoo-eng-team
Post to : yahoo-eng-team@lists.launchpad.net
Unsubscribe : https://launchpad.net/~yahoo-eng-team
More help   : https://help.launchpad.net/ListHelp


[Yahoo-eng-team] [Bug 1376563] Re: add unit tests covering full-sync to ODL

2015-01-27 Thread Cédric OLLIVIER
** Project changed: neutron => networking-odl

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to neutron.
https://bugs.launchpad.net/bugs/1376563

Title:
  add unit tests covering full-sync to ODL

Status in OpenDaylight backend controller integration with Neutron:
  New

Bug description:
  Full synchronization to OpenDaylight haven't been covered by any unit test.
  Bug #1371115 about "fix full synchronization between neutron and ODL" would 
have been caught.

  It should complete the bug #1325184 focused on covering the single
  operations.

To manage notifications about this bug go to:
https://bugs.launchpad.net/networking-odl/+bug/1376563/+subscriptions

-- 
Mailing list: https://launchpad.net/~yahoo-eng-team
Post to : yahoo-eng-team@lists.launchpad.net
Unsubscribe : https://launchpad.net/~yahoo-eng-team
More help   : https://help.launchpad.net/ListHelp


[Yahoo-eng-team] [Bug 1376169] Re: ODL MD can't reconnect to ODL after it restarts

2015-02-18 Thread Cédric OLLIVIER
** Also affects: neutron
   Importance: Undecided
   Status: New

** Changed in: neutron
 Assignee: (unassigned) => Cédric OLLIVIER (m.col)

** Tags added: juno-backport-potential

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to neutron.
https://bugs.launchpad.net/bugs/1376169

Title:
  ODL MD can't reconnect to ODL after it restarts

Status in OpenDaylight backend controller integration with Neutron:
  Fix Committed
Status in OpenStack Neutron (virtual network service):
  New

Bug description:
  ODL MD doesn't process any traitment when it receives 401 http error 
(Unauthorized) which happens after ODL restarts.
  The only way to recover is to restart neutron. 

  It induces a strongly link between restarts of neutron and ODL.

  To reproduce it:
   - start ODL and neutron
   - create a network
   - restart ODL
   - create another network

  The last command raises an 401 http error and any extra single
  operations will fail.

To manage notifications about this bug go to:
https://bugs.launchpad.net/networking-odl/+bug/1376169/+subscriptions

-- 
Mailing list: https://launchpad.net/~yahoo-eng-team
Post to : yahoo-eng-team@lists.launchpad.net
Unsubscribe : https://launchpad.net/~yahoo-eng-team
More help   : https://help.launchpad.net/ListHelp


[Yahoo-eng-team] [Bug 1361540] [NEW] no mechanism driver calls for gateway port removal

2014-08-26 Thread Cédric OLLIVIER
Public bug reported:

MechanismDriver.delete_port_* (gateway port) are not called when the router is 
being removed.
For instance, it remains the network:router_gateway ports in OpenDaylight as 
its Mechanism Driver is not correctly called.

To reproduce it:
- create a router and set the gateway
- delete this router (without clearing the gateway)

It works well if the gateway port is cleared before the router.

** Affects: neutron
 Importance: Undecided
 Status: New


** Tags: icehouse-backport-potential l3-ipam-dhcp ml2

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to neutron.
https://bugs.launchpad.net/bugs/1361540

Title:
  no mechanism driver calls for gateway port removal

Status in OpenStack Neutron (virtual network service):
  New

Bug description:
  MechanismDriver.delete_port_* (gateway port) are not called when the router 
is being removed.
  For instance, it remains the network:router_gateway ports in OpenDaylight as 
its Mechanism Driver is not correctly called.

  To reproduce it:
  - create a router and set the gateway
  - delete this router (without clearing the gateway)

  It works well if the gateway port is cleared before the router.

To manage notifications about this bug go to:
https://bugs.launchpad.net/neutron/+bug/1361540/+subscriptions

-- 
Mailing list: https://launchpad.net/~yahoo-eng-team
Post to : yahoo-eng-team@lists.launchpad.net
Unsubscribe : https://launchpad.net/~yahoo-eng-team
More help   : https://help.launchpad.net/ListHelp


[Yahoo-eng-team] [Bug 1324450] [NEW] add delete operations for the ODL MechanismDriver

2014-05-29 Thread Cédric OLLIVIER
Public bug reported:

The delete operations (networks, subnets and ports) haven't been managed since 
the 12th review of the initial support.
It seems sync_single_resource only implements create and update operations.

** Affects: neutron
 Importance: Undecided
 Status: New

** Patch added: "mechanism_odl.py_delete.diff"
   
https://bugs.launchpad.net/bugs/1324450/+attachment/4122020/+files/mechanism_odl.py_delete.diff

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to neutron.
https://bugs.launchpad.net/bugs/1324450

Title:
  add delete operations for the ODL MechanismDriver

Status in OpenStack Neutron (virtual network service):
  New

Bug description:
  The delete operations (networks, subnets and ports) haven't been managed 
since the 12th review of the initial support.
  It seems sync_single_resource only implements create and update operations.

To manage notifications about this bug go to:
https://bugs.launchpad.net/neutron/+bug/1324450/+subscriptions

-- 
Mailing list: https://launchpad.net/~yahoo-eng-team
Post to : yahoo-eng-team@lists.launchpad.net
Unsubscribe : https://launchpad.net/~yahoo-eng-team
More help   : https://help.launchpad.net/ListHelp


[Yahoo-eng-team] [Bug 1325184] [NEW] add unit tests for the ODL MechanismDriver

2014-05-31 Thread Cédric OLLIVIER
Public bug reported:

All the operations (create, update or delete) haven't been covered by unit 
tests.
Bug #1324450 about the delete operations would have been caught.

** Affects: neutron
 Importance: Undecided
 Status: New

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to neutron.
https://bugs.launchpad.net/bugs/1325184

Title:
  add unit tests for the ODL MechanismDriver

Status in OpenStack Neutron (virtual network service):
  New

Bug description:
  All the operations (create, update or delete) haven't been covered by unit 
tests.
  Bug #1324450 about the delete operations would have been caught.

To manage notifications about this bug go to:
https://bugs.launchpad.net/neutron/+bug/1325184/+subscriptions

-- 
Mailing list: https://launchpad.net/~yahoo-eng-team
Post to : yahoo-eng-team@lists.launchpad.net
Unsubscribe : https://launchpad.net/~yahoo-eng-team
More help   : https://help.launchpad.net/ListHelp


[Yahoo-eng-team] [Bug 1369239] [NEW] OpenDaylight MD should not ignore 400 errors

2014-09-14 Thread Cédric OLLIVIER
Public bug reported:

400 (Bad Request) errors are ignored in every create or update operation to 
OpenDaylight.
Referring to the comment, it protects against conflicts with already existing 
resources.

In case of update operations, it seems irrelevant and masks "real" bad requests.
It could also be removed in create operations.

** Affects: neutron
 Importance: Undecided
 Status: New


** Tags: icehouse-backport-potential opendaylight

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to neutron.
https://bugs.launchpad.net/bugs/1369239

Title:
  OpenDaylight MD should not ignore 400 errors

Status in OpenStack Neutron (virtual network service):
  New

Bug description:
  400 (Bad Request) errors are ignored in every create or update operation to 
OpenDaylight.
  Referring to the comment, it protects against conflicts with already existing 
resources.

  In case of update operations, it seems irrelevant and masks "real" bad 
requests.
  It could also be removed in create operations.

To manage notifications about this bug go to:
https://bugs.launchpad.net/neutron/+bug/1369239/+subscriptions

-- 
Mailing list: https://launchpad.net/~yahoo-eng-team
Post to : yahoo-eng-team@lists.launchpad.net
Unsubscribe : https://launchpad.net/~yahoo-eng-team
More help   : https://help.launchpad.net/ListHelp


[Yahoo-eng-team] [Bug 1371115] [NEW] fix full synchronization between neutron and ODL

2014-09-18 Thread Cédric OLLIVIER
Public bug reported:

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.

** Affects: neutron
 Importance: Undecided
     Assignee: Cédric OLLIVIER (m.col)
 Status: New


** Tags: icehouse-backport-potential opendaylight

** Changed in: neutron
 Assignee: (unassigned) => Cédric OLLIVIER (m.col)

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to neutron.
https://bugs.launchpad.net/bugs/1371115

Title:
  fix full synchronization between neutron and ODL

Status in OpenStack Neutron (virtual network service):
  New

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.

To manage notifications about this bug go to:
https://bugs.launchpad.net/neutron/+bug/1371115/+subscriptions

-- 
Mailing list: https://launchpad.net/~yahoo-eng-team
Post to : yahoo-eng-team@lists.launchpad.net
Unsubscribe : https://launchpad.net/~yahoo-eng-team
More help   : https://help.launchpad.net/ListHelp


[Yahoo-eng-team] [Bug 1373817] [NEW] ODL MD should not reraise ODL exceptions as they are

2014-09-25 Thread Cédric OLLIVIER
Public bug reported:

ODL MD reraised errors returned by ODL in single operations and in full 
synchronization.
In both cases, ODL MD could raise more adapted errors (i.e. server errors 
instead of client errors).

** Affects: neutron
 Importance: Undecided
 Status: New


** Tags: icehouse-backport-potential opendaylight

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to neutron.
https://bugs.launchpad.net/bugs/1373817

Title:
  ODL MD should not reraise ODL exceptions as they are

Status in OpenStack Neutron (virtual network service):
  New

Bug description:
  ODL MD reraised errors returned by ODL in single operations and in full 
synchronization.
  In both cases, ODL MD could raise more adapted errors (i.e. server errors 
instead of client errors).

To manage notifications about this bug go to:
https://bugs.launchpad.net/neutron/+bug/1373817/+subscriptions

-- 
Mailing list: https://launchpad.net/~yahoo-eng-team
Post to : yahoo-eng-team@lists.launchpad.net
Unsubscribe : https://launchpad.net/~yahoo-eng-team
More help   : https://help.launchpad.net/ListHelp


[Yahoo-eng-team] [Bug 1376169] [NEW] ODL MD can't reconnect to ODL after it restarts

2014-10-01 Thread Cédric OLLIVIER
Public bug reported:

ODL MD doesn't process any traitment when it receives 401 http error 
(Unauthorized) which happens after ODL restarts.
The only way to recover is to restart neutron. 

It induces a strongly link between restarts of neutron and ODL.

To reproduce it:
 - start ODL and neutron
 - create a network
 - restart ODL
 - create another network

The last command raises an 401 http error and any extra single
operations will fail.

** Affects: neutron
 Importance: Undecided
 Assignee: Cédric OLLIVIER (m.col)
 Status: New


** Tags: icehouse-backport-potential opendaylight

** Tags added: icehouse-backport-potential opendaylight

** Changed in: neutron
 Assignee: (unassigned) => Cédric OLLIVIER (m.col)

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to neutron.
https://bugs.launchpad.net/bugs/1376169

Title:
  ODL MD can't reconnect to ODL after it restarts

Status in OpenStack Neutron (virtual network service):
  New

Bug description:
  ODL MD doesn't process any traitment when it receives 401 http error 
(Unauthorized) which happens after ODL restarts.
  The only way to recover is to restart neutron. 

  It induces a strongly link between restarts of neutron and ODL.

  To reproduce it:
   - start ODL and neutron
   - create a network
   - restart ODL
   - create another network

  The last command raises an 401 http error and any extra single
  operations will fail.

To manage notifications about this bug go to:
https://bugs.launchpad.net/neutron/+bug/1376169/+subscriptions

-- 
Mailing list: https://launchpad.net/~yahoo-eng-team
Post to : yahoo-eng-team@lists.launchpad.net
Unsubscribe : https://launchpad.net/~yahoo-eng-team
More help   : https://help.launchpad.net/ListHelp


[Yahoo-eng-team] [Bug 1376563] [NEW] add unit tests covering full-sync to ODL

2014-10-01 Thread Cédric OLLIVIER
Public bug reported:

Full synchronization to OpenDaylight haven't been covered by any unit test.
Bug #1371115 about "fix full synchronization between neutron and ODL" would 
have been caught.

It should complete the bug #1325184 focused on covering the single
operations.

** Affects: neutron
 Importance: Undecided
 Assignee: Cédric OLLIVIER (m.col)
 Status: New


** Tags: icehouse-backport-potential opendaylight unittest

** Changed in: neutron
 Assignee: (unassigned) => Cédric OLLIVIER (m.col)

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to neutron.
https://bugs.launchpad.net/bugs/1376563

Title:
  add unit tests covering full-sync to ODL

Status in OpenStack Neutron (virtual network service):
  New

Bug description:
  Full synchronization to OpenDaylight haven't been covered by any unit test.
  Bug #1371115 about "fix full synchronization between neutron and ODL" would 
have been caught.

  It should complete the bug #1325184 focused on covering the single
  operations.

To manage notifications about this bug go to:
https://bugs.launchpad.net/neutron/+bug/1376563/+subscriptions

-- 
Mailing list: https://launchpad.net/~yahoo-eng-team
Post to : yahoo-eng-team@lists.launchpad.net
Unsubscribe : https://launchpad.net/~yahoo-eng-team
More help   : https://help.launchpad.net/ListHelp


[Yahoo-eng-team] [Bug 1863707] [NEW] [neutron-tempest-plugin] test_trunk.TrunkTestInheritJSONBase.test_add_subport fails if unordered

2020-02-17 Thread Cédric OLLIVIER
Public bug reported:

Tested vs OpenStack master + OVN:
Traceback (most recent call last):
  File 
"/usr/lib/python3.8/site-packages/neutron_tempest_plugin/api/test_trunk.py", 
line 238, in test_add_subport
self.assertEqual(expected_subports, trunk['sub_ports'])
  File "/usr/lib/python3.8/site-packages/testtools/testcase.py", line 411, in 
assertEqual
self.assertThat(observed, matcher, message)
  File "/usr/lib/python3.8/site-packages/testtools/testcase.py", line 498, in 
assertThat
raise mismatch_error
testtools.matchers._impl.MismatchError: !=:
reference = [{'port_id': '6eb624a9-368c-472c-a3ce-26608e77d2e7',
  'segmentation_id': 3,
  'segmentation_type': 'vlan'},
 {'port_id': '7a06845b-f379-4143-90f7-e664a8a602ec',
  'segmentation_id': 2,
  'segmentation_type': 'vlan'}]
actual= [{'port_id': '7a06845b-f379-4143-90f7-e664a8a602ec',
  'segmentation_id': 2,
  'segmentation_type': 'vlan'},
 {'port_id': '6eb624a9-368c-472c-a3ce-26608e77d2e7',
  'segmentation_id': 3,
  'segmentation_type': 'vlan'}]

A straightforward proposal would be:
diff --git a/neutron_tempest_plugin/api/test_trunk.py 
b/neutron_tempest_plugin/api/test_trunk.py
index 823a95d..bc8ec82 100644
--- a/neutron_tempest_plugin/api/test_trunk.py
+++ b/neutron_tempest_plugin/api/test_trunk.py
@@ -235,7 +235,8 @@ class TrunkTestInheritJSONBase(TrunkTestJSONBase):
   'segmentation_id': segmentation_id2}]
 
 # Validate that subport got segmentation details from the network
-    self.assertEqual(expected_subports, trunk['sub_ports'])
+self.assertin(expected_subports[0], trunk['sub_ports'])
+self.assertin(expected_subports[1], trunk['sub_ports'])

** Affects: neutron
 Importance: Undecided
 Assignee: Cédric OLLIVIER (m.col)
 Status: In Progress

** Summary changed:

- [neutron-tempest-plugin] test_trunk.TrunkTestInheritJSONBase.test_add_subport 
forces ordering
+ [neutron-tempest-plugin] test_trunk.TrunkTestInheritJSONBase.test_add_subport 
fails if unordered list

** Summary changed:

- [neutron-tempest-plugin] test_trunk.TrunkTestInheritJSONBase.test_add_subport 
fails if unordered list
+ [neutron-tempest-plugin] test_trunk.TrunkTestInheritJSONBase.test_add_subport 
fails if unordered

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to neutron.
https://bugs.launchpad.net/bugs/1863707

Title:
  [neutron-tempest-plugin]
  test_trunk.TrunkTestInheritJSONBase.test_add_subport fails if
  unordered

Status in neutron:
  In Progress

Bug description:
  Tested vs OpenStack master + OVN:
  Traceback (most recent call last):
File 
"/usr/lib/python3.8/site-packages/neutron_tempest_plugin/api/test_trunk.py", 
line 238, in test_add_subport
  self.assertEqual(expected_subports, trunk['sub_ports'])
File "/usr/lib/python3.8/site-packages/testtools/testcase.py", line 411, in 
assertEqual
  self.assertThat(observed, matcher, message)
File "/usr/lib/python3.8/site-packages/testtools/testcase.py", line 498, in 
assertThat
  raise mismatch_error
  testtools.matchers._impl.MismatchError: !=:
  reference = [{'port_id': '6eb624a9-368c-472c-a3ce-26608e77d2e7',
'segmentation_id': 3,
'segmentation_type': 'vlan'},
   {'port_id': '7a06845b-f379-4143-90f7-e664a8a602ec',
'segmentation_id': 2,
'segmentation_type': 'vlan'}]
  actual= [{'port_id': '7a06845b-f379-4143-90f7-e664a8a602ec',
'segmentation_id': 2,
'segmentation_type': 'vlan'},
   {'port_id': '6eb624a9-368c-472c-a3ce-26608e77d2e7',
'segmentation_id': 3,
'segmentation_type': 'vlan'}]

  A straightforward proposal would be:
  diff --git a/neutron_tempest_plugin/api/test_trunk.py 
b/neutron_tempest_plugin/api/test_trunk.py
  index 823a95d..bc8ec82 100644
  --- a/neutron_tempest_plugin/api/test_trunk.py
  +++ b/neutron_tempest_plugin/api/test_trunk.py
  @@ -235,7 +235,8 @@ class TrunkTestInheritJSONBase(TrunkTestJSONBase):
 'segmentation_id': segmentation_id2}]
   
   # Validate that subport got segmentation details from the network
  -self.assertEqual(expected_subports, trunk['sub_ports'])
  +self.assertin(expected_subports[0], trunk['sub_ports'])
  +self.assertin(expected_subports[1], trunk['sub_ports'])

To manage notifications about this bug go to:
https://bugs.launchpad.net/neutron/+bug/1863707/+subscriptions

-- 
Mailing list: https://launchpad.net/~yahoo-eng-team
Post to : yahoo-eng-team@lists.launchpad.net
Unsubscribe : https://launchpad.net/~yahoo-eng-team
More help   : https://help.launchpad.net/ListHelp