Skip to content

Commit

Permalink
Fix crash caused by incorrectly appended volume strings
Browse files Browse the repository at this point in the history
Docker volumes are supposed to be a list in the form of
[ '<path>:<bind>', '<path>:<bind>' ]

Current parsing code parses this correctly but zeros out self.volumes
and initializes it again in the form of [ '<path>', '<path>', '<path' ]

This breaks the parser if volumes is parsed again as it is expecting
 <path>:<bind> (a specific example is rebuild)
  • Loading branch information
anish committed Mar 24, 2016
1 parent 738588c commit 369da64
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion master/buildbot/worker/docker.py
Original file line number Diff line number Diff line change
Expand Up @@ -80,7 +80,7 @@ def __init__(self, name, password, docker_host, image=None, command=None,
config.error("Invalid volume definition for docker "
"%s. Skipping..." % volume_string)
continue
self.volumes.append(volume)
self.volumes.append(volume_string)

ro = False
if bind.endswith(':ro') or bind.endswith(':rw'):
Expand Down

0 comments on commit 369da64

Please sign in to comment.