search

ovn-org / ovn-scale-test

Apache License 2.0
18 stars 22 forks source link

Add DB clustering support for ovn-fake-multinode deployments. #195

Closed dceara closed 4 years ago

dceara commented 4 years ago

CC: @numansiddique

dceara commented 4 years ago

There are bunch of places where self.controller_client("ovn-nbctl") called by rally_ovs plugin, but only few of them actually calls ovn_nbctl.set_daemon_socket(self.context.get("daemon_socket", None)) to configure socket files. This will cause test failures if NB DB leader changed for whatever reason.

@igsilya thanks for spotting this! I added a commit to address all cases and call _get_ovn_controller() instead of explicitly creating ovn-nbctl connections.