ovn-tester: Group multiple ctl commands into transactions about ovn-heater HOT 4 CLOSED

While I agree that this might be easier to implement than other approaches, probably the most realistic test would be to not use SSH commands at all and instead use IDL clients connecting to NB/SB/OVSDB on the remote hosts. In the end, it's also the way a real CMS should configure OVN.

That being said, if we can easily batch some SSH commands together without complicating the code too much that's fine by me. However, if the effort to do that is significant, it might make more sense to invest time into looking at using the Python OVSDB IDL directly.

@putnopvut what do you think?

from ovn-heater.

I opened this issue with the idea that the unit under test is OVN, rather than the OVSDB client. In other words, the thing that generates the JSON requests for the OVSDB server isn't very important, as long as the content of those JSON requests is in keeping with what a real CMS would send.

There are only three reasons I can think of to switch to the python IDL

1. If the importance of the python IDL client starts to increase, then we may decide that it is important to specifically test the performance of the python IDL client.
2. It eliminates the overhead of an SSH connection and removes a dependency from ovn-tester.
3. It gives us the opportunity to exercise more stream types with OVSDB.

I think that switching to the python IDL is going to be more difficult than augmenting SSH, especially once the asyncio changes go in.

from ovn-heater.

Ok, thanks for the follow up.

from ovn-heater.

We switched to the python IDL (ovsdbapp) since #136. Closing this issue.

from ovn-heater.