ci: Only use one artifact instruction for logs.

Having the different types of logs in artifacts instruction makes it quicker
to navigate between them. Either that didn't use to be possible, or I just did
it wrong. I saw that / how it works in a patch by Justin Pryzby.

Also remove a few unnecessary uses of **, suggested by Justin Pryzby.

Discussion: https://postgr.es/m/20220203195718[email protected]

diff --git a/.cirrus.yml b/.cirrus.yml
index 8c84ff0b00dfe2c378748569662f56424960f44c..ff3f4306ade500380790b1232854f52bf1bd4ff1 100644 (file)
--- a/.cirrus.yml
+++ b/.cirrus.yml
@@ -30,15 +30,11 @@ env:
 # What files to preserve in case tests fail
 on_failure: &on_failure
   log_artifacts:
-    path: "**/**.log"
+    paths:
+      - "**/*.log"
+      - "**/.diffs"
+      - "**/regress_log_*"
     type: text/plain
-  regress_diffs_artifacts:
-    path: "**/**.diffs"
-    type: text/plain
-  tap_artifacts:
-    path: "**/regress_log_*"
-    type: text/plain
-
 
 task:
   name: FreeBSD - 13
@@ -440,7 +436,7 @@ task:
   on_failure:
     <<: *on_failure
     crashlog_artifacts:
-      path: "crashlog-**.txt"
+      path: "crashlog-*.txt"
       type: text/plain