[PATCH 1/1] use a simpler syntax for the nrpe config

From: Michael Scherer m...@zarb.org --- roles/nagios_client/tasks/main.yml | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/roles/nagios_client/tasks/main.yml b/roles/nagios_client/tasks/main.yml index a04e3fb..373cd60 100644 ---

[PATCH] use a simpler syntax for the nrpe config

From: Michael Scherer m...@zarb.org --- roles/nagios_client/tasks/main.yml | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/roles/nagios_client/tasks/main.yml b/roles/nagios_client/tasks/main.yml index a04e3fb..373cd60 100644 ---

[PATCH 1/1] move virthost to a role

From: Michael Scherer m...@zarb.org --- files/virthost/libvirtd.conf | 3 --- handlers/restart_services.yml | 3 --- playbooks/groups/virthost.yml | 2 +- roles/virthost/files/libvirtd.conf | 3 +++ roles/virthost/handlers/main.yml | 5 + roles/virthost/tasks/main.yml

[PATCH 1/2] split memcached into a separate roles from mailman

From: Michael Scherer m...@zarb.org --- roles/mailman/files/memcached.sysconfig | 5 - roles/mailman/handlers/main.yml | 3 +-- roles/mailman/meta/main.yml | 4 roles/mailman/tasks/main.yml | 10 --

[PATCH 2/2] add deps to all software that use memcached to have it installed and managed

From: Michael Scherer m...@zarb.org --- roles/badges/frontend/meta/main.yml | 3 +++ roles/mailman/meta/main.yml | 1 - roles/nuancier/meta/main.yml| 4 roles/pkgdb2/meta/main.yml | 3 +++ 4 files changed, 10 insertions(+), 1 deletion(-) create mode 100644

[PATCH] Replace 'Medium' by 'Moderate' for the security category

From: Michael Scherer The CSI document do not define Medium, but Moderate. --- inventory/group_vars/bodhi-backend-stg | 2 +- inventory/group_vars/bodhi2| 2 +- inventory/group_vars/bodhi2-stg| 2 +- inventory/group_vars/kojipkgs | 2 +-

[PATCH] Remove handlers that do not correspond to any existing playbook

From: Michael Scherer --- handlers/restart_services.yml | 21 - 1 file changed, 21 deletions(-) diff --git a/handlers/restart_services.yml b/handlers/restart_services.yml index 75a805b..67f48ef 100644 --- a/handlers/restart_services.yml +++

[PATCH] Refactor (again) the code deploying zodbot/ursabot

From: Michael Scherer Since there is the same code between staging and production, except for the bot name, using a variable permit to avoid duplication. (this time, it work) --- roles/supybot/tasks/main.yml | 36 roles/supybot/vars/main.yml

[PATCH] Simplify fasclient package installation

From: Michael Scherer Using package, we can merge the yum/dnf duplication, and we do not need is_rhel, since only EL6 will have ansible_distribution_major_version at 6. --- roles/fas_client/tasks/main.yml | 16 +++- 1 file changed, 3 insertions(+), 13 deletions(-)

[PATCH] Move ntp setup in a role

From: Michael Scherer Since base install ntp on all platforms, we can skip the vars and place it by default (next step is to convert the few playbooks duplicating the role) --- roles/base/files/ntp/step-tickers| 7 --- roles/base/meta/main.yml | 1 +

[PATCH] Refactor xinetd in a separate role

From: Michael Scherer --- handlers/restart_services.yml | 3 --- roles/git/server/meta/main.yml | 4 roles/git/server/tasks/main.yml | 5 - roles/rsyncd/meta/main.yml | 3 +++ roles/rsyncd/tasks/main.yml | 7 --- roles/xinetd/handlers/main.yml | 2

[PATCH 1/2] Refactor packages installation for fedmsg-irc, and use package module

From: Michael Scherer --- roles/fedmsg/irc/tasks/main.yml | 13 + 1 file changed, 5 insertions(+), 8 deletions(-) diff --git a/roles/fedmsg/irc/tasks/main.yml b/roles/fedmsg/irc/tasks/main.yml index ac23440..618ab1e 100644 --- a/roles/fedmsg/irc/tasks/main.yml +++

[PATCH 2/2] Start fedmsg-irc once the config is setup

From: Michael Scherer Also, do not notify the service to restart once it was started by service, that's not needed --- roles/fedmsg/irc/tasks/main.yml | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/roles/fedmsg/irc/tasks/main.yml

[PATCH] Move yumrepos to a role

From: Michael Scherer Since the role is added a dependencie of base, I also removed it from all playbooks, since all of them will pull yumrepos. Bonus point, it will be pulled first, avoiding ordering issues (since tasks are run after roles, and several roles likely need to have

[PATCH] Remove the old sks.conf file

From: Michael Scherer It was a left over of the migration to a role --- files/keyserver/sks.conf | 83 1 file changed, 83 deletions(-) delete mode 100644 files/keyserver/sks.conf diff --git a/files/keyserver/sks.conf

[PATCH] Fix the playbook, since hosts was move to a role

From: Michael Scherer --- playbooks/manual/qadevel.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/playbooks/manual/qadevel.yml b/playbooks/manual/qadevel.yml index 6105d36..f229d4f 100644 --- a/playbooks/manual/qadevel.yml +++

[PATCH] Move netapp_* vars to the iscsi role

From: Michael Scherer Since that's the only role where they are used, it make sense to keep it close and not "pollute" the global namespace --- roles/iscsi_client/vars/main.yml | 5 + vars/global.yml | 4 2 files changed, 5 insertions(+), 4 deletions(-)

[PATCH] Move file from gnome_backup to the role

From: Michael Scherer --- files/gnome/backup.sh| 41 files/gnome/ssh_config | 8 --- roles/gnome_backups/files/backup.sh | 41 roles/gnome_backups/files/ssh_config | 8

[PATCH] Split postfix in a separate file for clarity

From: Michael Scherer --- roles/base/tasks/main.yml| 51 ++-- roles/base/tasks/postfix.yml | 51 2 files changed, 53 insertions(+), 49 deletions(-) create mode 100644

[PATCH] Refactor the copy of keys and certs for unbound

From: Michael Scherer Also restart the server if the key is changed, and tags the task --- roles/unbound/tasks/main.yml | 22 +++--- 1 file changed, 11 insertions(+), 11 deletions(-) diff --git a/roles/unbound/tasks/main.yml b/roles/unbound/tasks/main.yml index

[PATCH] Move ntp setup in a role

From: Michael Scherer Since base install ntp on all platforms, we can skip the vars and place it by default (next step is to convert the few playbooks duplicating the role) --- roles/base/files/ntp/step-tickers| 7 --- roles/base/meta/main.yml | 1 +

[PATCH] Move koshei handlers to the role

From: Michael Scherer --- handlers/restart_services.yml | 12 roles/koschei/handlers/main.yml | 11 +++ 2 files changed, 11 insertions(+), 12 deletions(-) create mode 100644 roles/koschei/handlers/main.yml diff --git a/handlers/restart_services.yml

[PATCH] Move the unbound related handlers to the role

From: Michael Scherer --- handlers/restart_services.yml | 3 --- handlers/semanage.yml | 8 playbooks/groups/blockerbugs.yml | 1 - playbooks/groups/dhcp.yml

[PATCH] Remove duplicate handlers for update-ca-trust

From: Michael Scherer Since there is a already existing update ca-trust in handlers/main.yml doing the same thing as 'update ca trust', better use 1 single copy. --- handlers/update_ca_trust.yml | 6 -- tasks/osbs_certs.yml | 2 +- 2 files changed, 1 insertion(+), 7

[PATCH 1/2] Move service start to the end (after we did set it up)

From: Michael Scherer Also remove notify for handler for service, since the service will be started by service, no need to notify a handler to restart it. --- roles/fedmsg/gateway/tasks/main.yml | 14 ++ 1 file changed, 6 insertions(+), 8 deletions(-) diff --git

[PATCH 2/2] Do not override the complete systemd config

From: Michael Scherer using the drop-in config snipper system, we can add configuration to systemd without replacing it, in case the systemd file is changed upstream. --- roles/fedmsg/gateway/files/fedmsg-gateway.service | 15 ---

[PATCH 2/3] Use package module, so we can simplify the package installation

From: Michael Scherer --- roles/base/tasks/watchdog.yml | 14 ++ 1 file changed, 2 insertions(+), 12 deletions(-) diff --git a/roles/base/tasks/watchdog.yml b/roles/base/tasks/watchdog.yml index 6ae0d54..feebcfc 100644 --- a/roles/base/tasks/watchdog.yml +++

[PATCH 3/3] Refactor the condition to setup the watchdog with a block

From: Michael Scherer --- roles/base/tasks/watchdog.yml | 52 +-- 1 file changed, 26 insertions(+), 26 deletions(-) diff --git a/roles/base/tasks/watchdog.yml b/roles/base/tasks/watchdog.yml index feebcfc..148c856 100644 ---

[PATCH 1/3] Split watchdog related setup in a separate file

From: Michael Scherer --- roles/base/tasks/main.yml | 46 ++- roles/base/tasks/watchdog.yml | 44 + 2 files changed, 46 insertions(+), 44 deletions(-) create mode 100644

[PATCH] Fix typo in doc

From: Michael Scherer --- CONVENTIONS | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/CONVENTIONS b/CONVENTIONS index f6c2eec..f6b37fe 100644 --- a/CONVENTIONS +++ b/CONVENTIONS @@ -9,7 +9,7 @@ Playbook naming === The top level playbooks

[PATCH] Remove old file from some openshift v2 deployment

From: Michael Scherer Nothing seems to use them in the current repo --- files/openshift/Gemfile| 50 --- files/openshift/jenkins.repo | 5 -- files/openshift/mcollective-client.cfg | 18 --- files/openshift/mongo-acct.sh |

[PATCH] Use a file instead of a command for removing postfix pid

From: Michael Scherer Avoid a uneeded fork, and avoid repeating the filename --- playbooks/death_to_postfix.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/playbooks/death_to_postfix.yml b/playbooks/death_to_postfix.yml index 46a1f33..8e0c2a8 100644 ---

[PATCH] Remove old unused handlers in the current codebase

From: Michael Scherer --- handlers/restart_services.yml | 18 -- 1 file changed, 18 deletions(-) diff --git a/handlers/restart_services.yml b/handlers/restart_services.yml index a2e9669..24bfcc7 100644 --- a/handlers/restart_services.yml +++

[PATCH] Move a few handlers to the only role using them

From: Michael Scherer --- handlers/restart_services.yml | 15 --- roles/copr/backend/handlers/main.yml | 3 +++ roles/haproxy/handlers/main.yml| 3 +++ roles/koji_builder/handlers/main.yml | 3 +++ roles/kojipkgs/handlers/main.yml | 3

[PATCH] Fix edafebe7dca01, varnish file was missing

From: Michael Scherer --- roles/varnish/handlers/main.yml | 2 ++ 1 file changed, 2 insertions(+) create mode 100644 roles/varnish/handlers/main.yml diff --git a/roles/varnish/handlers/main.yml b/roles/varnish/handlers/main.yml new file mode 100644 index 000..ce6018b ---

[PATCH] Refactor the code deploying zodbot/ursabot

From: Michael Scherer Since there is the same code between staging and production, except for the bot name, using a variable permit to avoid duplication. --- roles/supybot/tasks/main.yml | 36 roles/supybot/vars/main.yml | 3 +++ 2 files

[PATCH] Fix small errors in sshaccess.rst

From: Michael Scherer --- sshaccess.rst | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/sshaccess.rst b/sshaccess.rst index 6bfcebe..b73043b 100644 --- a/sshaccess.rst +++ b/sshaccess.rst @@ -50,7 +50,7 @@ First of all: (on your local machine)::