Skip to content

Commit f544b4a

Browse files
authored
chore(ansible): Fix Ansible playbook deprecation warnings (#318)
### Summary Recent Ansible upgrades caused all old-style fact var injection to cause deprecation warnings: https://docs.ansible.com/projects/ansible/devel/porting_guides/porting_guide_13.html\#inject-facts-as-vars Modify playbooks to use the new style. <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **Refactor** * Updated internal deployment configuration to standardise how environment variables are accessed across system setup tasks. <!-- end of auto-generated comment: release notes by coderabbit.ai -->
1 parent 3cbe9ac commit f544b4a

2 files changed

Lines changed: 12 additions & 12 deletions

File tree

infra/ansible/roles/elt/tasks/elt_cron_tasks.yml

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -6,8 +6,8 @@
66
path: "{{ elt_secrets_root_dir }}/{{ warehouse_name }}"
77
state: directory
88
mode: "u=rwx,g=,o="
9-
owner: "{{ ansible_env['USER'] }}"
10-
group: "{{ ansible_env['USER'] }}"
9+
owner: "{{ ansible_facts['env']['USER'] }}"
10+
group: "{{ ansible_facts['env']['USER'] }}"
1111

1212
- name: Generate secrets
1313
become: true
@@ -16,8 +16,8 @@
1616
src: ./secrets/envvars.j2
1717
dest: "{{ elt_secrets_root_dir }}/{{ warehouse_name }}/envvars"
1818
mode: "u=r,g=,o="
19-
owner: "{{ ansible_env['USER'] }}"
20-
group: "{{ ansible_env['USER'] }}"
19+
owner: "{{ ansible_facts['env']['USER'] }}"
20+
group: "{{ ansible_facts['env']['USER'] }}"
2121

2222
- name: Ensure cron logs directory exists
2323
ansible.builtin.file:

infra/ansible/roles/elt/tasks/main.yml

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -28,18 +28,18 @@
2828
# ---------- AWS S3 ----------
2929
- name: Ensure S3 configuration directory exists
3030
ansible.builtin.file:
31-
path: "{{ ansible_env['HOME'] }}/.aws"
31+
path: "{{ ansible_facts['env']['HOME'] }}/.aws"
3232
state: directory
3333
mode: "u=rwx,g=rx,o=rx"
3434

3535
- name: Ensure S3 config file is absent
3636
ansible.builtin.file:
37-
path: "{{ ansible_env['HOME'] }}/.aws/config"
37+
path: "{{ ansible_facts['env']['HOME'] }}/.aws/config"
3838
state: absent
3939

4040
- name: Ensure AWS configuration is present
4141
ansible.builtin.copy:
42-
dest: "{{ ansible_env['HOME'] }}/.aws/credentials"
42+
dest: "{{ ansible_facts['env']['HOME'] }}/.aws/credentials"
4343
content: |+
4444
[default]
4545
request_checksum_calculation = when_required
@@ -48,22 +48,22 @@
4848
# ---------- dlt ----------
4949
- name: Ensure dlt configuration directory exists
5050
ansible.builtin.file:
51-
path: "{{ ansible_env['HOME'] }}/.dlt"
51+
path: "{{ ansible_facts['env']['HOME'] }}/.dlt"
5252
state: directory
5353
mode: "u=rwx,g=rx,o=rx"
5454

5555
- name: Generate .dlt/config.toml
5656
no_log: "{{ not (elt_enable_logs | default(False)) }}"
5757
ansible.builtin.template:
5858
src: dlt/config.toml.j2
59-
dest: "{{ ansible_env['HOME'] }}/.dlt/config.toml"
59+
dest: "{{ ansible_facts['env']['HOME'] }}/.dlt/config.toml"
6060
mode: "u=rwx,g=r,o=r"
6161

6262
- name: Set shared ELT job variables
6363
ansible.builtin.set_fact:
64-
elt_git_clone_dir: "{{ ansible_env['HOME'] }}/var/git"
65-
elt_cron_root_dir: "{{ ansible_env['HOME'] }}/var/cron"
66-
elt_cron_logs_root_dir: "{{ ansible_env['HOME'] }}/var/cron/logs"
64+
elt_git_clone_dir: "{{ ansible_facts['env']['HOME'] }}/var/git"
65+
elt_cron_root_dir: "{{ ansible_facts['env']['HOME'] }}/var/cron"
66+
elt_cron_logs_root_dir: "{{ ansible_facts['env']['HOME'] }}/var/cron/logs"
6767

6868
- name: Ensure cron scripts directory exists
6969
ansible.builtin.file:

0 commit comments

Comments
 (0)