This is a mechanically generated change to replace openstack.org
git:// URLs with https:// equivalents.
This is in aid of a planned future move of the git hosting
infrastructure to a self-hosted instance of gitea (https://gitea.io),
which does not support the git wire protocol at this stage.
This update should result in no functional change.
For more information see the thread at
http://lists.openstack.org/pipermail/openstack-discuss/2019-March/003825.html
Change-Id: Iabaa13ebc5ff9c8bb842c89a62fdbac75acd774c
get_local_ports_to_ofport_mapping will be called when process port.
It will return all information of chassises and logical ports in
current node. However, only one chassis or one port will be used.
When the topology is huge, for example, thousands of chassises and ports,
it will be time/resource consuming to call the method.
This patch makes the method just return the wanted information.
Related-Bug: #1622879
Change-Id: Ifb51575de444fa28e48a6330debdb72ba37a565c
These two deprecation warning can be observed, when run UT for
test_l3_router_plugin
DeprecationWarning: neutron.db.l3_db.Router: moved to
neutron.db.models.l3.Router
DeprecationWarning: neutron.db.l3_db.FloatingIP: moved to
neutron.db.models.l3.FloatingIP
Change-Id: I9aeda537aea405e86dfcbda2e0245be8ca151b8c
The line endings were CRLF (dos/windows style). The standard is to use
UNIX style endings.
No other change was made to the file.
This is a result of following command in dragonflow folder.
find ./ -name '*.rst' -type f -exec dos2unix {} \;
Change-Id: Idb2f6f3d4d32044b3d095dfcf27e31a04741d071
The sample local.conf of controller has been set to USE_ML2_PLUGIN=True.
However, the sample local.conf of compute node has not been set.
In a real deployment, it will fail if user uses sample local.conf for
both controller and compute node.
Since it is end of N release, it would be safe to just change the sameple
local.conf for now. Changing USE_ML2_PLUGIN=True as default value will
be performed in O release.
Change-Id: I5143adef0bc8f09ee35616908b600f2fa70c08ea
Partial-Bug: #1618792
The logical port might already been deleted due to concurrent operation.
DF mech_driver should check its existence before updating it.
Change-Id: I110b4d3adcd5c169a65917422c1f2c8fc3bac993
Close-Bug: #1628408
Current OvsdbSwitchApi provides ovs command sometimes, and outputs result
directly sometimes. It makes it hard to use. Also, OvsdbSwitchApi uses
neutron framework to execute ovs command sometimes, and uses idlutils
directly sometimes. Using idlutils directly will not reconnect ovsdb,
if timeout.
This patch does following things:
1) Create a dedicated ovsdb api, which is a sub class of OvsdbIdl from
neutron
2) Make OvsdbSwitchApi execute ovs command, and remove explicit execute()
call from consumers
3) Remove directly using idlutils in OvsdbSwitchApi
4) Remove obselete methods in OvsdbSwitchApi
5) Remove duplicated command class, which already have defination in
neutron
Change-Id: I7e9a7f9337c703673b7817ad47de1223c1b01ec0
Closes-Bug: #1622879
Following OpenStack Style Guidelines[1]:
http://docs.openstack.org/developer/hacking/#unit-tests-and-assertraises
[H203] Unit test assertions tend to give better messages for more specific assertions.
As a result, assertIsNone(...) is preferred over assertEqual(None, ...) and assertIs(..,None)
Change-Id: I2ab46cc724665ed5362cee865c8c0dfa0e93a0ae
As a sub-class of api_nb.NbApi, ovsdb_nb_impl.OvsdbNbApi is not used
anywhere. And the logic in ovsdb_nb_impl is out of date somehow. For
example, OVN_Southbound listens to 6642, and OVN_Northbound listens to
6641 by default, however, ovsdb_nb_impl still uses port 6640 to connect
both OVN_Southbound and OVN_Northbound DB.
Besides, ovsdb_nb_impl will require installing ovn, which might cause
confusion to user, as ovn is another distributed sdn controller, compared
to dragonflow.
Besides, using ovn db to store nb db might cause conflicts, as ovn db
change will cause ovn-controller to download OpenFlow, if there is any
ovn-controller.
This patch just remove ovsdb_nb_impl.
Change-Id: I7ddc583146989bcd598b06c49c8190a1c28597f3
Related-Bug: #1622879