Last active
August 29, 2015 14:05
-
-
Save sgoings/c1a7f0a79010980e8252 to your computer and use it in GitHub Desktop.
playbook
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
--- | |
- hosts: all | |
tasks: | |
- name: "This task works" | |
docker: image="readytalk/tomcat-native:8.0.9" | |
state="running" | |
env="CATALINA_OPTS=-Ddb_max_active='5' | |
-Ddb_initial_pool_size='1' | |
-Ddb_time_bewtween_eviction='600000'" | |
- name: "Try to inject commas in env strings" | |
docker: image="readytalk/tomcat-native:8.0.9" | |
state="running" | |
env='CATALINA_OPTS=-Dconfig.zookeepers="zookeeper1,zookeeper2,zookeeper3"' | |
################## | |
OUTPUT ERROR (on second task): | |
invalid output was: Traceback (most recent call last): | |
File "/home/sgoings/.ansible/tmp/ansible-tmp-1408550577.4-10076568041997/docker", line 2130, in <module> | |
main() | |
File "/home/sgoings/.ansible/tmp/ansible-tmp-1408550577.4-10076568041997/docker", line 700, in main | |
manager = DockerManager(module) | |
File "/home/sgoings/.ansible/tmp/ansible-tmp-1408550577.4-10076568041997/docker", line 394, in __init__ | |
self.env = dict(map(lambda x: x.split("=", 1), self.module.params.get('env'))) | |
OUTPUT with devel branch: | |
File "/home/sgoings/.ansible/tmp/ansible-tmp-1408553134.84-195115593098547/docker", line 2163, in <module> | |
main() | |
File "/home/sgoings/.ansible/tmp/ansible-tmp-1408553134.84-195115593098547/docker", line 724, in main | |
net = dict(default=None) | |
File "/home/sgoings/.ansible/tmp/ansible-tmp-1408553134.84-195115593098547/docker", line 1086, in __init__ | |
self._check_argument_types() | |
File "/home/sgoings/.ansible/tmp/ansible-tmp-1408553134.84-195115593098547/docker", line 1623, in _check_argument_types | |
self.params[k] = dict([x.strip().split("=", 1) for x in value.split(",")]) | |
ValueError: dictionary update sequence element #1 has length 1; 2 is required |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment