From 6e06bee66ff71382177cb880e50b9343b1cce1fb Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Roman=20Kr=C4=8Dek?= <roman.krcek@tietoevry.com>
Date: Wed, 15 May 2024 10:17:06 +0200
Subject: [PATCH] Fix: Update Skyline stop task
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Update Sykline stop task to use the
service-stop role to symplify the task
and make sure it is using kolla_container.

Authored-By: Roman Krček <roman.krcek@tietoevry.com>
Change-Id: I7b11359cee931273a058364160b64fe1fb606b5e
---
 ansible/roles/skyline/tasks/stop.yml                | 13 ++++---------
 .../notes/skyline-stop-7eb0d8770335a9ca.yaml        |  6 ++++++
 2 files changed, 10 insertions(+), 9 deletions(-)
 create mode 100644 releasenotes/notes/skyline-stop-7eb0d8770335a9ca.yaml

diff --git a/ansible/roles/skyline/tasks/stop.yml b/ansible/roles/skyline/tasks/stop.yml
index a8e927807d..965ab35b97 100644
--- a/ansible/roles/skyline/tasks/stop.yml
+++ b/ansible/roles/skyline/tasks/stop.yml
@@ -1,11 +1,6 @@
 ---
-- name: "Stopping skyline containers"
+- import_role:
+    name: service-stop
   vars:
-    service: "{{ item.value }}"
-  docker_container:
-    name: "{{ service.container_name }}"
-    state: stopped
-  when:
-    - service.enabled | bool
-    - service.container_name not in skip_stop_containers
-  with_dict: "{{ skyline_services }}"
+    project_services: "{{ skyline_services }}"
+    service_name: "{{ project_name }}"
diff --git a/releasenotes/notes/skyline-stop-7eb0d8770335a9ca.yaml b/releasenotes/notes/skyline-stop-7eb0d8770335a9ca.yaml
new file mode 100644
index 0000000000..16524fc902
--- /dev/null
+++ b/releasenotes/notes/skyline-stop-7eb0d8770335a9ca.yaml
@@ -0,0 +1,6 @@
+---
+fixes:
+  - |
+    Fixes skyline's old format of stop task.
+    It used docker_container which would cause
+    problems with podman deployments.