Skip to content
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

Docker version check failed #260

Open
nadgowdas opened this issue Mar 4, 2017 · 1 comment
Open

Docker version check failed #260

nadgowdas opened this issue Mar 4, 2017 · 1 comment
Assignees
Labels

Comments

@nadgowdas
Copy link
Contributor

Description

Crawler docker version check fails for rc build versions of docker

How to Reproduce

Install any rc build docker version and run crawler with it.

Log Output

Traceback (most recent call last):
 File "crawler.py", line 239, in <module>
   main()
 File "crawler.py", line 233, in main
   worker.run()
 File "/crawler/worker.py", line 83, in run
   self.iterate(time_to_sleep)
 File "/crawler/worker.py", line 46, in iterate
   for frame in self.crawler.crawl():
 File "/crawler/containers_crawler.py", line 85, in crawl
   host_namespace=self.host_namespace)
 File "/crawler/containers.py", line 83, in get_containers
   for _container in containers_list:
 File "/crawler/containers.py", line 26, in list_all_containers
   user_list=user_list):
 File "/crawler/dockercontainer.py", line 59, in get_docker_containers
   host_namespace=host_namespace)
 File "/crawler/dockercontainer.py", line 171, in __init__
   self.root_fs = get_docker_container_rootfs_path(self.long_id)
 File "/crawler/utils/dockerutils.py", line 359, in get_docker_container_rootfs_path
   rootfs_path = _get_container_rootfs_path_aufs(long_id, inspect)
 File "/crawler/utils/dockerutils.py", line 277, in _get_container_rootfs_path_aufs
   if VERSION_SPEC.match(semantic_version.Version(server_version)):
 File "/usr/local/lib/python2.7/site-packages/semantic_version/base.py", line 75, in __init__
   major, minor, patch, prerelease, build = self.parse(version_string, partial)
 File "/usr/local/lib/python2.7/site-packages/semantic_version/base.py", line 208, in parse
   raise ValueError("Invalid leading zero in minor: %r" % version_string)
ValueError: Invalid leading zero in minor: u'17.03.0-ce'

Docker info

sudo docker info
Containers: 13
Running: 7
Paused: 0
Stopped: 6
Images: 7
Server Version: 17.03.0-ce

Solution

Stefan mentioned, with semantic_version=2.2, it work. But we are using sematic_version 2.5.
Fixed this issue and add a new docker version in the travis tests

@nadgowdas nadgowdas added the bug label Mar 4, 2017
@nadgowdas
Copy link
Contributor Author

nadgowdas commented Mar 7, 2017

Docker has decided to have monthly release cycle and follow YY.MM version format.
https://github.com/docker/docker/blob/master/CHANGELOG.md

So docker release is not going to be semantic anymore..

We need long term fix for this by changing semantic_version

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants