-
Notifications
You must be signed in to change notification settings - Fork 29
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
integration test of mininet #577
Comments
Not sure I should add this here or in a different issue, but here it goes... If I add a replace directive in the go.mod to use my local version of onet it works fine. Hope this is enough to identify/fix the bug. |
Another probably more import issue kind of associated with this one. Currently mininet only works for linux version 16.0.4 LTS which is slowly getting deprecated. Is it possible to update this for Bionic 18.0. or even Mint 20.0? Is it also easy to adapt the code to work for other machines other than iccluster (amazon servers for example)? @cgrigis |
Let's shoot for the stars and try 20.04... In the best case, all you need to do is to add a case here: The following should work on all systems, but to be confirmed: What I'm most afraid is a change of mininet version, which would need changes to: When testing, also make sure that it works with multiple servers. So start with one server, then if that works, test with 3 servers. Eventually also update https://github.com/dedis/onet/blob/master/simul/platform/MININET.md - more specifically test that the |
I think this has been fixed in the past. |
Joao commented to me that "it's too bad that often when I come back to using mininet after several months it doesn't work, is that a feature of onet you guys support?"
I told him yes, it is supported, but no, it's not well tested because of the difficulty of reserving servers to test on. He told me that now servers can be reserved instantaneously, which means that we could include a mininet integration test in Jenkins that runs once a day or something.
This issue tracks doing something better than "hope it works" testing for mininet platform testing.
The text was updated successfully, but these errors were encountered: