Implement binding strategy in creating and binding lport#111
Implement binding strategy in creating and binding lport#111huikang wants to merge 2 commits intoovn-org:masterfrom
Conversation
|
Address #95 |
| Overview | ||
| ======== | ||
|
|
||
| In OVN, a logical port can be bind to a local OVS port on any chassis/hypervisor, depending on the VM scheduler (e.g., ``nova-scheduler``). The binding strategy potentially impacts the network performance. That is binding all logical ports in a logical network on a single hypervisor performs differently than distributing the ports on multiple hypervisors. |
There was a problem hiding this comment.
It would be good to linewrap this file at 80 characters. Can you do that please?
There was a problem hiding this comment.
@mestery sorry about the wrong format. Please review the updated PR.
- see the guide: ansible/doc/binding-strategy.rst Signed-off-by: Hui Kang <kangh@us.ibm.com>
huikang
force-pushed
the
add-binding-strategy
branch
from
aaf7a49 to
703f276
Compare
| def __init__(self): | ||
| self.sandboxes = [] | ||
| self.ports_per_network = 0 | ||
|
|
There was a problem hiding this comment.
better to add 'self.lswitch = None' in init(), otherwise get_lswitch() before set_lswitch() may raise a exception
|
For distribute lswithes/ports to sandbox evenly, I run rally-ovs multiple times, each time with a set of sandbox, like: |
|
@l8huang your understanding is correct. The new variable |
|
I will combine the commits into one when the PR is ready for merge. Thanks. |
|
@l8huang Just a gentle reminder: do you mind spending some minutes reviewing this PR? Thanks. |
3 participants
Signed-off-by: Hui Kang kangh@us.ibm.com