From 7646df53d4cc94c820844db94d6bb8448744011d Mon Sep 17 00:00:00 2001 From: Steve McMahon Date: Thu, 14 Jul 2016 14:49:45 -0700 Subject: [PATCH 1/3] Docs for ansible pipelining --- README.md | 10 ++++++++++ 1 file changed, 10 insertions(+) diff --git a/README.md b/README.md index 7718d84..e275262 100644 --- a/README.md +++ b/README.md @@ -44,6 +44,16 @@ You should not need to. But if you do, you should know: Role Variables -------------- +### ansible_ssh_pipelining + + ansible_ssh_pipelining: true + +This role uses ssh pipelining to avoid security errors from Ansible when running operations without superuser rights. +SSH pipelining for this purpose may require the disabling of ‘requiretty’ in /etc/sudoers. +If you get a pipelining error and cannot disable requiretty, set this variable to false and instead turn on allow_world_readable_tmpfiles in your ansible.cfg. +See http://docs.ansible.com/ansible/become.html#becoming-an-unprivileged-user for a discussion. + + ### plone_instance_name plone_instance_name: frodos_site From 3b57040aa9c5bd05d5df324deec97e3a40b3ddec Mon Sep 17 00:00:00 2001 From: Steve McMahon Date: Thu, 14 Jul 2016 15:08:49 -0700 Subject: [PATCH 2/3] Testing: handle role name difference --- .travis.yml | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/.travis.yml b/.travis.yml index b55e5c8..bb96a11 100644 --- a/.travis.yml +++ b/.travis.yml @@ -17,6 +17,10 @@ install: - echo "localhost" > inventory script: + # We have a different canonical role name from our github repo + # in order to accommodate galaxy's naming conventions. + - "ln -s ../ansible.plone_server ../plone.plone_server" + # Check the role/playbook's syntax. - "ansible-playbook -i inventory $SITE --syntax-check" From 5c8e4eba00c120f397a8c12c98564837ce5a14d5 Mon Sep 17 00:00:00 2001 From: Steve McMahon Date: Thu, 14 Jul 2016 15:11:09 -0700 Subject: [PATCH 3/3] Testing: handle role name difference --- .travis.yml | 4 ---- test.yml | 4 ++-- 2 files changed, 2 insertions(+), 6 deletions(-) diff --git a/.travis.yml b/.travis.yml index bb96a11..b55e5c8 100644 --- a/.travis.yml +++ b/.travis.yml @@ -17,10 +17,6 @@ install: - echo "localhost" > inventory script: - # We have a different canonical role name from our github repo - # in order to accommodate galaxy's naming conventions. - - "ln -s ../ansible.plone_server ../plone.plone_server" - # Check the role/playbook's syntax. - "ansible-playbook -i inventory $SITE --syntax-check" diff --git a/test.yml b/test.yml index 25351a6..7aa3554 100644 --- a/test.yml +++ b/test.yml @@ -53,10 +53,10 @@ roles: - - role: plone.plone_server + - role: ansible.plone_server plone_config: "{{ servers[0] }}" - - role: plone.plone_server + - role: ansible.plone_server plone_config: "{{ servers[1] }}" tasks: