osrg / bgperf

Apache License 2.0
86 stars 30 forks source link

doctor: better handling of docker version #10

Closed pierky closed 7 years ago

pierky commented 7 years ago

With Docker 17.03.0-ce I had this error:

$ ./bgperf.py doctor
Traceback (most recent call last):
  File "./bgperf.py", line 400, in <module>
    args.func(args)
  File "./bgperf.py", line 67, in doctor
    ok = int(''.join(ver.split('.'))) >= 190
ValueError: invalid literal for int() with base 10: '17030-ce'
$ docker version
Client:
 Version:      17.03.0-ce
 API version:  1.26
 Go version:   go1.7.5
 Git commit:   60ccb22
 Built:        Thu Feb 23 10:57:47 2017
 OS/Arch:      linux/amd64

Server:
 Version:      17.03.0-ce
 API version:  1.26 (minimum version 1.12)
 Go version:   go1.7.5
 Git commit:   60ccb22
 Built:        Thu Feb 23 10:57:47 2017
 OS/Arch:      linux/amd64
 Experimental: false
ishidawataru commented 7 years ago

thanks!