Skip to content

Commit

Permalink
https://github.com/ansible/ansible-lint/pull/2295
Browse files Browse the repository at this point in the history
  • Loading branch information
hswong3i committed Aug 31, 2022
1 parent 3a909d0 commit 55b4ec8
Show file tree
Hide file tree
Showing 7 changed files with 44 additions and 22 deletions.
3 changes: 2 additions & 1 deletion playbooks/50-ceph-reboot.yml
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,8 @@
# See the License for the specific language governing permissions and
# limitations under the License.

- hosts: ceph_mon:ceph_mgr:ceph_osd:ceph_mds:ceph_rgw:!ansible
- name: 50-ceph-reboot | ceph_mon:ceph_mgr:ceph_osd:ceph_mds:ceph_rgw:!ansible
hosts: ceph_mon:ceph_mgr:ceph_osd:ceph_mds:ceph_rgw:!ansible
remote_user: root
become: true
tasks:
Expand Down
3 changes: 2 additions & 1 deletion playbooks/90-ansible-reboot.yml
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,8 @@
# See the License for the specific language governing permissions and
# limitations under the License.

- hosts: ansible
- name: 90-ansible-reboot | ansible
hosts: ansible
remote_user: root
become: true
tasks:
Expand Down
3 changes: 2 additions & 1 deletion playbooks/cleanup.yml
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,8 @@
# See the License for the specific language governing permissions and
# limitations under the License.

- hosts: all
- name: cleanup | all
hosts: all
remote_user: root
become: true
gather_facts: false
Expand Down
21 changes: 14 additions & 7 deletions playbooks/converge.yml
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,8 @@
# See the License for the specific language governing permissions and
# limitations under the License.

- hosts: all
- name: converge | all
hosts: all
remote_user: root
become: true
tasks:
Expand Down Expand Up @@ -88,47 +89,53 @@
ansible.builtin.include_role:
name: sshd

- hosts: ceph_mon:ceph_mgr:ceph_osd:ceph_mds:ceph_rgw
- name: converge | ceph_mon:ceph_mgr:ceph_osd:ceph_mds:ceph_rgw
hosts: ceph_mon:ceph_mgr:ceph_osd:ceph_mds:ceph_rgw
remote_user: root
become: true
tasks:
- name: include role
ansible.builtin.include_role:
name: ceph_common

- hosts: ceph_mon
- name: converge | ceph_mon
hosts: ceph_mon
remote_user: root
become: true
tasks:
- name: include role
ansible.builtin.include_role:
name: ceph_mon

- hosts: ceph_mgr
- name: converge | ceph_mgr
hosts: ceph_mgr
remote_user: root
become: true
tasks:
- name: include role
ansible.builtin.include_role:
name: ceph_mgr

- hosts: ceph_osd
- name: converge | ceph_osd
hosts: ceph_osd
remote_user: root
become: true
tasks:
- name: include role
ansible.builtin.include_role:
name: ceph_osd

- hosts: ceph_mds
- name: converge | ceph_mds
hosts: ceph_mds
remote_user: root
become: true
tasks:
- name: include role
ansible.builtin.include_role:
name: ceph_mds

- hosts: ceph_rgw
- name: converge | ceph_rgw
hosts: ceph_rgw
remote_user: root
become: true
tasks:
Expand Down
6 changes: 4 additions & 2 deletions playbooks/prepare.yml
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,8 @@
# See the License for the specific language governing permissions and
# limitations under the License.

- hosts: all
- name: prepare | all
hosts: all
remote_user: root
become: true
gather_facts: false
Expand Down Expand Up @@ -53,7 +54,8 @@
changed_when: false
failed_when: false

- hosts: all
- name: prepare | all
hosts: all
remote_user: root
become: true
tasks:
Expand Down
3 changes: 2 additions & 1 deletion playbooks/side_effect.yml
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,8 @@
# See the License for the specific language governing permissions and
# limitations under the License.

- hosts: all
- name: side_effect | all
hosts: all
remote_user: root
become: true
tasks:
Expand Down
27 changes: 18 additions & 9 deletions playbooks/verify.yml
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,8 @@
# See the License for the specific language governing permissions and
# limitations under the License.

- hosts: all
- name: verify | all
hosts: all
remote_user: root
become: true
tasks:
Expand Down Expand Up @@ -71,55 +72,62 @@
loop:
- { dest: "/root/.ssh/authorized_keys", mode: "0600" }

- hosts: ceph_mon:ceph_mgr:ceph_osd:ceph_mds:ceph_rgw
- name: verify | ceph_mon:ceph_mgr:ceph_osd:ceph_mds:ceph_rgw
hosts: ceph_mon:ceph_mgr:ceph_osd:ceph_mds:ceph_rgw
remote_user: root
become: true
tasks:
- name: include role
ansible.builtin.include_role:
name: ceph_common

- hosts: ceph_mon
- name: verify | ceph_mon
hosts: ceph_mon
remote_user: root
become: true
tasks:
- name: include role
ansible.builtin.include_role:
name: ceph_mon

- hosts: ceph_mgr
- name: verify | ceph_mgr
hosts: ceph_mgr
remote_user: root
become: true
tasks:
- name: include role
ansible.builtin.include_role:
name: ceph_mgr

- hosts: ceph_osd
- name: verify | ceph_osd
hosts: ceph_osd
remote_user: root
become: true
tasks:
- name: include role
ansible.builtin.include_role:
name: ceph_osd

- hosts: ceph_mds
- name: verify | ceph_mds
hosts: ceph_mds
remote_user: root
become: true
tasks:
- name: include role
ansible.builtin.include_role:
name: ceph_mds

- hosts: ceph_rgw
- name: verify | ceph_rgw
hosts: ceph_rgw
remote_user: root
become: true
tasks:
- name: include role
ansible.builtin.include_role:
name: ceph_rgw

- hosts: ansible
- name: verify | ansible
hosts: ansible
remote_user: root
become: true
tasks:
Expand Down Expand Up @@ -149,7 +157,8 @@
register: base64_decode_result
changed_when: base64_decode_result is not succeeded

- hosts: ceph_mon:ceph_mgr:ceph_osd:ceph_mds:ceph_rgw
- name: verify | ceph_mon:ceph_mgr:ceph_osd:ceph_mds:ceph_rgw
hosts: ceph_mon:ceph_mgr:ceph_osd:ceph_mds:ceph_rgw
remote_user: root
become: true
tasks:
Expand Down

0 comments on commit 55b4ec8

Please sign in to comment.