From 18be272e215f5c95288740fa76caf4a6af1d2ac9 Mon Sep 17 00:00:00 2001
From: Martin Pitt <mpitt@redhat.com>
Date: Tue, 18 Jul 2023 14:41:11 +0200
Subject: [PATCH] XXX debug insights

---
 packit.yaml                   | 20 --------------------
 test/verify/check-system-info |  4 ++++
 2 files changed, 4 insertions(+), 20 deletions(-)

diff --git a/packit.yaml b/packit.yaml
index 318731a1c3fa..063b5c7a065e 100644
--- a/packit.yaml
+++ b/packit.yaml
@@ -23,26 +23,6 @@ srpm_build_deps:
 # use the nicely formatted release NEWS from our upstream release, instead of git shortlog
 copy_upstream_release_description: true
 jobs:
-  - job: tests
-    trigger: pull_request
-    targets:
-      - fedora-37
-      - fedora-38
-      - fedora-latest-aarch64
-      - fedora-development
-      - centos-stream-8-x86_64
-      - centos-stream-9-x86_64
-      - centos-stream-9-aarch64
-
-  # run build/unit tests on some interesting architectures
-  - job: copr_build
-    trigger: pull_request
-    targets:
-      # 32 bit
-      - fedora-development-i386
-      # big-endian
-      - fedora-development-s390x
-
   - job: copr_build
     trigger: release
     owner: "@cockpit"
diff --git a/test/verify/check-system-info b/test/verify/check-system-info
index 6c4a340197da..9fd2c5ad7e8d 100755
--- a/test/verify/check-system-info
+++ b/test/verify/check-system-info
@@ -853,6 +853,8 @@ fi
         m = self.machine
         b = self.browser
 
+        m.verbose = True
+
         # insights-client might get started during boot and might then
         # run concurrently with our explicit "insights-client
         # --register" below.  insights-client is not designed to be
@@ -863,6 +865,7 @@ fi
         self.restore_dir("/etc/insights-client")
         self.restore_dir("/etc/yum")
         self.restore_dir("/var/lib/insights")
+        self.addCleanup(m.execute, "systemctl status insights-client >&2 || true; pgrep -af [i]nsight >&2 || true; pgrep -f [i]nsight | xargs systemctl status >&2 || true")
 
         # Pretend that the Subscriptions page can do Insights stuff
         self.write_file("/etc/cockpit/subscription-manager.override.json", '{ "features": { "insights": true } }')
@@ -888,6 +891,7 @@ password=foobar
 
         # Initially we are not registered
         self.login_and_go('/system')
+        b.cdp.trace = True
         b.wait_text(".system-health-insights a", "Not connected to Insights")
 
         # Enable insights, results should appear automatically