MY_SUBVERSION is defined in messages.py as a byte string, but here we were comparing this value to the value returned by the RPC. Convert to ensure the types match.

for i in {1..100}
do
  ./test/functional/test_runner.py p2p_add_connections
done