diff options
| author | Egor Tensin <Egor.Tensin@gmail.com> | 2023-08-20 18:38:17 +0200 |
|---|---|---|
| committer | Egor Tensin <Egor.Tensin@gmail.com> | 2023-08-20 21:43:59 +0200 |
| commit | 97b930c6edc7973497f469ae859fa2258cbea4d6 (patch) | |
| tree | 822382f2275cc25cb7d6d280cc37a17d0c7e1214 /roles/digitalocean_volume | |
| parent | v0.0.16 (diff) | |
| download | infra-ansible-97b930c6edc7973497f469ae859fa2258cbea4d6.tar.gz infra-ansible-97b930c6edc7973497f469ae859fa2258cbea4d6.zip | |
use variables instead facts mostly everywhere
set_fact is stupid; they persist through multiple role executions; for
example, you cannot do this:
set_fact:
foo: '{{ foo | default("bar") }}'
If somebody calls the role and defines foo, it will always be set to
that value forever, even for subsequent role calls.
Diffstat (limited to 'roles/digitalocean_volume')
| -rw-r--r-- | roles/digitalocean_volume/tasks/main.yml | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/roles/digitalocean_volume/tasks/main.yml b/roles/digitalocean_volume/tasks/main.yml index c23da76..07d6c74 100644 --- a/roles/digitalocean_volume/tasks/main.yml +++ b/roles/digitalocean_volume/tasks/main.yml @@ -1,11 +1,7 @@ -- name: Set default mount point - ansible.builtin.set_fact: - volume_dir: "{{ volume_dir | default('/mnt/{{ volume_name }}') }}" - - name: 'Mount volume: {{ volume_name }}' become: true ansible.posix.mount: - path: '{{ volume_dir }}' + path: "{{ volume_dir | default('/mnt/{{ volume_name }}') }}" src: '/dev/disk/by-id/scsi-0DO_Volume_{{ volume_name }}' state: mounted fstype: '{{ volume_fs }}' |