Skip to content
This repository has been archived by the owner on Mar 31, 2023. It is now read-only.

[Enhancement] convert execute_ovsdb_command shell calls to ovsdb protocol #122

Open
er1cthe0ne opened this issue Aug 13, 2020 · 0 comments
Labels
help wanted Extra attention is needed

Comments

@er1cthe0ne
Copy link
Contributor

er1cthe0ne commented Aug 13, 2020

10/06/2020 update: execute_openflow_command implementation using openflow protocol is completed with PR#137.

Done: The execute_openflow_command conversion to openflow protocol, similar to what ovs-ofctl is doing should be done first. The implementation and testing should only take days.

execute_ovsdb_command and execute_openflow_command shell calls was used as a quick way to manipulate ovs database, with the current ability of openflow controller in ACA, we should switch to use openflow protocol. This will improve performance and failure diagnosis. This should also help the performance of parallel neighbor creation with data below:

1 Neighbor Create:
Neighbor State(0) took: 15069514 nanoseconds or 15 milliseconds
Average L2 Neighbor Create of 1 took: 15069514 nanoseconds or 15 milliseconds
[TEST METRICS] Elapsed time for message total operation took: 23627628 nanoseconds or 23 milliseconds
 
2 Neighbors Create:
Neighbor State(0) took: 15858309 nanoseconds or 15 milliseconds
Neighbor State(1) took: 16741936 nanoseconds or 16 milliseconds
Average L2 Neighbor Create of 2 took: 16300122 nanoseconds or 16 milliseconds
[TEST METRICS] Elapsed time for message total operation took: 25486427 nanoseconds or 25 milliseconds
 
5 Neighbors Create:
Neighbor State(0) took: 16456704 nanoseconds or 16 milliseconds
Neighbor State(1) took: 17646185 nanoseconds or 17 milliseconds
Neighbor State(2) took: 18333706 nanoseconds or 18 milliseconds
Neighbor State(3) took: 19331644 nanoseconds or 19 milliseconds
Neighbor State(4) took: 19333761 nanoseconds or 19 milliseconds
Average L2 Neighbor Create of 5 took: 18220400 nanoseconds or 18 milliseconds
[TEST METRICS] Elapsed time for message total operation took: 29158475 nanoseconds or 29 milliseconds
 
10 Neighbors Create:
Average L2 Neighbor Create of 10 took: 24590829 nanoseconds or 24 milliseconds
[TEST METRICS] Elapsed time for message total operation took: 39429835 nanoseconds or 39 milliseconds
 
25 Neighbors Create:
Average L2 Neighbor Create of 25 took: 41843695 nanoseconds or 41 milliseconds
[TEST METRICS] Elapsed time for message total operation took: 81601841 nanoseconds or 81 milliseconds
 
50 Neighbors Create:
Average L2 Neighbor Create of 50 took: 89949436 nanoseconds or 89 milliseconds
[TEST METRICS] Elapsed time for message total operation took: 201807854 nanoseconds or 201 milliseconds
 
100 Neighbors Create:
Average L2 Neighbor Create of 100 took: 239373127 nanoseconds or 239 milliseconds
[TEST METRICS] Elapsed time for message total operation took: 737647040 nanoseconds or 737 milliseconds
 
150 Neighbors Create:
Average L2 Neighbor Create of 150 took: 906120768 nanoseconds or 906 milliseconds
[TEST METRICS] Elapsed time for message total operation took: 3210279786 nanoseconds or 3210 milliseconds
 
200 Neighbors Create: Failed
Average L2 Neighbor Create of 200 took: 812249789 nanoseconds or 812 milliseconds
[TEST METRICS] Elapsed time for message total operation took: 1135719911 nanoseconds or 1135 milliseconds
@er1cthe0ne er1cthe0ne added the help wanted Extra attention is needed label Aug 13, 2020
@er1cthe0ne er1cthe0ne changed the title [Enhancement] covert execute_ovsdb_command and execute_openflow_command shell calls to openflow protocol [Enhancement] convert execute_ovsdb_command and execute_openflow_command shell calls to openflow protocol Aug 20, 2020
@er1cthe0ne er1cthe0ne changed the title [Enhancement] convert execute_ovsdb_command and execute_openflow_command shell calls to openflow protocol [Enhancement] convert execute_ovsdb_command shell calls to openflow protocol Oct 7, 2020
@er1cthe0ne er1cthe0ne changed the title [Enhancement] convert execute_ovsdb_command shell calls to openflow protocol [Enhancement] convert execute_ovsdb_command shell calls to ovsdb protocol Nov 3, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

1 participant