Open andreaseno opened 1 year ago
@andreaseno I merged #327 so now you will need to resolve conflicts.
@andreaseno @xwedea I added a couple more changes. If you can fix them sometime in the next few weeks - great! If not I'll get another student to help out later.
Replaced the server.conf file inside the load balancer nf with a json file called server.json. Because the config file parser was written for a plaintext file rather than a json file, I had to rewrite the backend config file parse function. It now uses cjson to parse the config file.
Summary:
Usage:
Merging notes:
TODO before merging :
Test Plan:
I tested the changes by doing a full setup of a 2 server ONVM experiment, made sure the arp_response and load balancer nf's ran correctly without error, and used iperf to verify that the packets would still send correctly. The only other necessary testing I can think of off the top of my head is verifying it works with more than 2 servers.
Review:
(optional) << @-mention people who should review these changes >>
(optional) Subscribers: << @-mention people who probably care about these changes >>