From 58f80b4ec7c5053266292c201636b3ae3f03e9d5 Mon Sep 17 00:00:00 2001
From: Slawek Kaplonski <skaplons@redhat.com>
Date: Mon, 5 Sep 2022 12:56:30 +0200
Subject: [PATCH] Remove "fetch_journal_log" role as it's not needed

We don't need to run that ansible role in the functional and fullstack
jobs as devstack already stores archive with journal log from the test
node.
It is available in the "devstack.journal.gz" file in logs.

Change-Id: Id527e7b5240240f5299a6f163bc124002c64fa18
---
 playbooks/post_functional_job.yaml         |  1 -
 roles/fetch_journal_log/README.rst         | 16 ----------------
 roles/fetch_journal_log/defaults/main.yaml |  2 --
 roles/fetch_journal_log/tasks/main.yaml    | 22 ----------------------
 4 files changed, 41 deletions(-)
 delete mode 100644 roles/fetch_journal_log/README.rst
 delete mode 100644 roles/fetch_journal_log/defaults/main.yaml
 delete mode 100644 roles/fetch_journal_log/tasks/main.yaml

diff --git a/playbooks/post_functional_job.yaml b/playbooks/post_functional_job.yaml
index 2537f1cbd9a..7f0cb198245 100644
--- a/playbooks/post_functional_job.yaml
+++ b/playbooks/post_functional_job.yaml
@@ -1,5 +1,4 @@
 - hosts: all
   roles:
-    - fetch_journal_log
     - fetch-tox-output
     - fetch-subunit-output
diff --git a/roles/fetch_journal_log/README.rst b/roles/fetch_journal_log/README.rst
deleted file mode 100644
index 71276c3f6a4..00000000000
--- a/roles/fetch_journal_log/README.rst
+++ /dev/null
@@ -1,16 +0,0 @@
-Collect journal log from test run
-
-By default, this stores journal log into log file and store it in
-"journal_log_file_path"
-
-**Role Variables**
-
-.. zuul:rolevar:: journal_log_path
-   :default: {{ ansible_user_dir }}/workspace/logs
-
-   Path where journal log file will be stored on job's node.
-
-.. zuul:rolevar:: journal_log_file_name
-   :default: {{ journal_log_path }}/journal.log
-
-   Name of journal log file.
diff --git a/roles/fetch_journal_log/defaults/main.yaml b/roles/fetch_journal_log/defaults/main.yaml
deleted file mode 100644
index dfc540ba9d1..00000000000
--- a/roles/fetch_journal_log/defaults/main.yaml
+++ /dev/null
@@ -1,2 +0,0 @@
-journal_log_path: "{{ ansible_user_dir }}/logs"
-journal_log_file_name: "{{ journal_log_path }}/journal.log"
diff --git a/roles/fetch_journal_log/tasks/main.yaml b/roles/fetch_journal_log/tasks/main.yaml
deleted file mode 100644
index 107417e714d..00000000000
--- a/roles/fetch_journal_log/tasks/main.yaml
+++ /dev/null
@@ -1,22 +0,0 @@
-- name: Ensure {{ journal_log_path }} exists
-  become: yes
-  file:
-    path: "{{ journal_log_path }}"
-    state: directory
-    owner: "{{ ansible_user }}"
-    group: "{{ ansible_user }}"
-    mode: 0775
-
-- name: Store journal logs in {{ journal_log_file_name }}
-  become: yes
-  shell:
-    cmd: |
-      /bin/journalctl -a > {{ journal_log_file_name }}
-
-- name: Set journal.log file permissions
-  become: yes
-  file:
-    path: '{{ journal_log_file_name }}'
-    owner: '{{ ansible_user }}'
-    group: '{{ ansible_user }}'
-    mode: 0644