From fab12fe57aa196014655ddbcc7aba827eb4d16b8 Mon Sep 17 00:00:00 2001 From: Markos Chandras Date: Fri, 15 Jun 2018 13:08:39 +0100 Subject: [PATCH] playbooks: roles: Drop explicit 'delegate_to' options Bifrost instructions suggest to use 'connection: local' when calling any of the roles. However, there are scenarios which the roles can be delegated to some other host. The explicit 'delegate_to' statements prevent us from doing that and they are not necessary if we use 'connection: local' anyway. As such we drop the explicit statements to make the roles a bit more flexible. Change-Id: Ifc983992125d203899d74b8bf997f9f58f528978 --- playbooks/roles/bifrost-deploy-nodes-dynamic/tasks/main.yml | 1 - playbooks/roles/bifrost-unprovision-node-dynamic/tasks/main.yml | 1 - playbooks/roles/ironic-delete-dynamic/tasks/main.yml | 1 - 3 files changed, 3 deletions(-) diff --git a/playbooks/roles/bifrost-deploy-nodes-dynamic/tasks/main.yml b/playbooks/roles/bifrost-deploy-nodes-dynamic/tasks/main.yml index 456b066da..ee65a2edd 100644 --- a/playbooks/roles/bifrost-deploy-nodes-dynamic/tasks/main.yml +++ b/playbooks/roles/bifrost-deploy-nodes-dynamic/tasks/main.yml @@ -100,5 +100,4 @@ root_gb: 10 wait: "{{ wait_for_node_deploy }}" timeout: " {{ wait_timeout | default(1800) }}" - delegate_to: localhost when: instance_info is not defined or ( instance_info is defined and instance_info | to_json == '{}' ) diff --git a/playbooks/roles/bifrost-unprovision-node-dynamic/tasks/main.yml b/playbooks/roles/bifrost-unprovision-node-dynamic/tasks/main.yml index 39acb3d14..e23ce7288 100644 --- a/playbooks/roles/bifrost-unprovision-node-dynamic/tasks/main.yml +++ b/playbooks/roles/bifrost-unprovision-node-dynamic/tasks/main.yml @@ -43,4 +43,3 @@ name: "{{ name | default() }}" state: absent instance_info: "{}" - delegate_to: localhost diff --git a/playbooks/roles/ironic-delete-dynamic/tasks/main.yml b/playbooks/roles/ironic-delete-dynamic/tasks/main.yml index 29fa7e2d9..4ff83c519 100644 --- a/playbooks/roles/ironic-delete-dynamic/tasks/main.yml +++ b/playbooks/roles/ironic-delete-dynamic/tasks/main.yml @@ -45,4 +45,3 @@ state: absent nics: "{{ nics }}" driver_info: "{}" - delegate_to: localhost