diff options
| author | Russell Teague <rteague@redhat.com> | 2016-11-18 14:26:02 -0500 |
|---|---|---|
| committer | Russell Teague <rteague@redhat.com> | 2016-11-28 11:11:44 -0500 |
| commit | ec9c22ca6a376b7d1139a384dcb78f3f5bb5b607 (patch) | |
| tree | a8c3897b9765b9b6dc02582a851b704be8f0828e /roles/openshift_storage_nfs_lvm | |
| parent | b45db4c7e0b278623f7531cee50127aa24686139 (diff) | |
| download | openshift-ec9c22ca6a376b7d1139a384dcb78f3f5bb5b607.tar.gz openshift-ec9c22ca6a376b7d1139a384dcb78f3f5bb5b607.tar.bz2 openshift-ec9c22ca6a376b7d1139a384dcb78f3f5bb5b607.tar.xz openshift-ec9c22ca6a376b7d1139a384dcb78f3f5bb5b607.zip | |
Refactored to use Ansible systemd module
* Ansible systemd module used in place of service module
* Refactored command tasks which are no longer necessary
* Applying rules from openshift-ansible Best Practices Guide
Diffstat (limited to 'roles/openshift_storage_nfs_lvm')
| -rw-r--r-- | roles/openshift_storage_nfs_lvm/README.md | 5 | ||||
| -rw-r--r-- | roles/openshift_storage_nfs_lvm/handlers/main.yml | 2 | ||||
| -rw-r--r-- | roles/openshift_storage_nfs_lvm/meta/main.yml | 2 | ||||
| -rw-r--r-- | roles/openshift_storage_nfs_lvm/tasks/nfs.yml | 15 |
4 files changed, 16 insertions, 8 deletions
diff --git a/roles/openshift_storage_nfs_lvm/README.md b/roles/openshift_storage_nfs_lvm/README.md index 3680ef5b5..8b8471745 100644 --- a/roles/openshift_storage_nfs_lvm/README.md +++ b/roles/openshift_storage_nfs_lvm/README.md @@ -8,10 +8,9 @@ create persistent volumes. ## Requirements -* NFS server with NFS, iptables, and everything setup. - +* Ansible 2.2 +* NFS server with NFS, iptables, and everything setup * A lvm volume group created on the nfs server (default: openshiftvg) - * The lvm volume needs to have as much free space as you are allocating ## Role Variables diff --git a/roles/openshift_storage_nfs_lvm/handlers/main.yml b/roles/openshift_storage_nfs_lvm/handlers/main.yml index 52f3ceffe..9ce8b783d 100644 --- a/roles/openshift_storage_nfs_lvm/handlers/main.yml +++ b/roles/openshift_storage_nfs_lvm/handlers/main.yml @@ -1,3 +1,3 @@ --- - name: restart nfs - service: name=nfs-server state=restarted + systemd: name=nfs-server state=restarted diff --git a/roles/openshift_storage_nfs_lvm/meta/main.yml b/roles/openshift_storage_nfs_lvm/meta/main.yml index 62ea54883..bed1216f8 100644 --- a/roles/openshift_storage_nfs_lvm/meta/main.yml +++ b/roles/openshift_storage_nfs_lvm/meta/main.yml @@ -4,7 +4,7 @@ galaxy_info: description: Create LVM volumes and use them as openshift persistent volumes. company: Red Hat, Inc. license: license (Apache) - min_ansible_version: 1.4 + min_ansible_version: 2.2 platforms: - name: EL versions: diff --git a/roles/openshift_storage_nfs_lvm/tasks/nfs.yml b/roles/openshift_storage_nfs_lvm/tasks/nfs.yml index e0be9f0b7..03f4fcec0 100644 --- a/roles/openshift_storage_nfs_lvm/tasks/nfs.yml +++ b/roles/openshift_storage_nfs_lvm/tasks/nfs.yml @@ -4,14 +4,23 @@ when: not openshift.common.is_containerized | bool - name: Start rpcbind - service: name=rpcbind state=started enabled=yes + systemd: + name: rpcbind + state: started + enabled: yes - name: Start nfs - service: name=nfs-server state=started enabled=yes + systemd: + name: nfs-server + state: started + enabled: yes - name: Export the directories lineinfile: dest=/etc/exports regexp="^{{ osnl_mount_dir }}/{{ item }} " line="{{ osnl_mount_dir }}/{{ item }} {{osnl_nfs_export_options}}" - with_sequence: start={{osnl_volume_num_start}} count={{osnl_number_of_volumes}} format={{osnl_volume_prefix}}{{osnl_volume_size}}g%04d + with_sequence: + start: "{{osnl_volume_num_start}}" + count: "{{osnl_number_of_volumes}}" + format: "{{osnl_volume_prefix}}{{osnl_volume_size}}g%04d" notify: restart nfs |