Address review feedback: remove redundant service stops

Remove the deploy-dev pre-task that stopped foreman.target before
deployment - the roles should handle idempotent re-deploys.

Remove the explicit postgresql stop from the upgrade path since
postgresql is PartOf=foreman.target and stopping the target
cascades to it.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: jeremylenz

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

@@ -15,12 +15,6 @@
     - "../../../src/vars/foreman.yml"
     - "../../../src/vars/base.yaml"
   pre_tasks:
-    - name: Stop foreman target
-      ansible.builtin.systemd:
-        name: foreman.target
-        state: stopped
-      failed_when: false
-
     - name: Set development postgresql databases
       ansible.builtin.set_fact:
         postgresql_databases: >-

src/roles/postgresql/tasks/upgrade.yaml

@@ -27,12 +27,6 @@
         state: stopped
       failed_when: false
 
-    - name: Stop postgresql service
-      ansible.builtin.systemd:
-        name: "{{ postgresql_container_name }}"
-        state: stopped
-      failed_when: false
-
     - name: Wait for port 5432 to be completely freed by Podman
       ansible.builtin.wait_for:
         port: 5432