Skip to content

Commit 294b8c4

Browse files
jeremylenzclaude
authored andcommitted
Move systemd_target before postgresql to avoid failed_when
Move the systemd_target role before postgresql in deploy-dev so foreman.target exists when the upgrade task needs to stop it. This eliminates the need for failed_when: false. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent b7887fc commit 294b8c4

2 files changed

Lines changed: 1 addition & 2 deletions

File tree

development/playbooks/deploy-dev/deploy-dev.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -51,13 +51,13 @@
5151
foreman_development_enabled_plugins: "{{ foreman_development_enabled_plugins + ['foreman_ansible'] }}"
5252
roles:
5353
- role: pre_install
54+
- role: systemd_target
5455
- role: certificates
5556
- role: postgresql
5657
- role: valkey
5758
- role: candlepin
5859
- role: httpd
5960
- role: pulp
60-
- role: systemd_target
6161
- role: foreman_development
6262
vars:
6363
foreman_development_oauth_consumer_key: "{{ foreman_oauth_consumer_key }}"

src/roles/postgresql/tasks/upgrade.yaml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,6 @@
2525
ansible.builtin.systemd:
2626
name: foreman.target
2727
state: stopped
28-
failed_when: false
2928

3029
- name: Wait for port 5432 to be completely freed by Podman
3130
ansible.builtin.wait_for:

0 commit comments

Comments
 (0)