CI: Set explicit retention days for uploaded assets; name fixes

Default retention period is 90 days which is excessive for large
artifacts, especially now that this can be controlled.

Do not retain the packages built for post-install testing. We limit to
the minimum of 1 day. These assets are large and we don't want people to
use them.

diff --git a/.github/workflows/ci-deb.yml b/.github/workflows/ci-deb.yml
index 5846ffd1dcbbd46e988654fe832e3b0d85677038..0cc5b37d0cf236261b95849b06ad8f69848ba5db 100644 (file)
--- a/.github/workflows/ci-deb.yml
+++ b/.github/workflows/ci-deb.yml
@@ -102,6 +102,7 @@ jobs:
       with:
         name: debs-${{ matrix.env.NAME }}
         path: debs
+        retention-days: 1
 
   deb-test:
 
@@ -179,7 +180,9 @@ jobs:
       if: ${{ failure() }}
       uses: actions/upload-artifact@v2
       with:
-        name: radius-logs-${{ matrix.env.NAME }}.tgz
+        name: radius-logs-${{ matrix.env.NAME }}
         path: |
           /var/log/freeradius
           freeradius/build/tests/eapol_test
+        retention-days: 30
+

diff --git a/.github/workflows/ci-rpm.yml b/.github/workflows/ci-rpm.yml
index 68909f8b352c6b0edfb703deb24826da95f6243c..5518236f32ca8efda09ee5377e227e0e267eee9d 100644 (file)
--- a/.github/workflows/ci-rpm.yml
+++ b/.github/workflows/ci-rpm.yml
@@ -151,6 +151,7 @@ jobs:
       with:
         name: rpms-${{ matrix.env.NAME }}
         path: rpms
+        retention-days: 1
 
   rpm-test:
 
@@ -255,7 +256,8 @@ jobs:
       if: ${{ failure() }}
       uses: actions/upload-artifact@v2
       with:
-        name: radius-logs-${{ matrix.env.NAME }}.tgz
+        name: radius-logs-${{ matrix.env.NAME }}
         path: |
           /var/log/radius
           freeradius/build/tests/eapol_test
+        retention-days: 30

diff --git a/.github/workflows/ci-scheduled-fuzzing.yml b/.github/workflows/ci-scheduled-fuzzing.yml
index fce69ae8ba6e12a1d649ee0923af3ad38b0ef1fd..b4e5f04b4b1a999e1c02a0e1fd38140e7a99b553 100644 (file)
--- a/.github/workflows/ci-scheduled-fuzzing.yml
+++ b/.github/workflows/ci-scheduled-fuzzing.yml
@@ -144,4 +144,5 @@ jobs:
       with:
         name: fuzzer-${{ matrix.env.PROTOCOL }}-${{ steps.pick_commit.outputs.commit_id }}
         path: build/fuzzer
+        retention_days: 30
       if: ${{ failure() }}

diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
index 95c830a861a2574f4214236dc6d37895df52aee0..1a7168643122ec07b3e6811202ef54e07fc85e82 100644 (file)
--- a/.github/workflows/ci.yml
+++ b/.github/workflows/ci.yml
@@ -251,6 +251,7 @@ jobs:
       with:
         name: clang-scan.tgz
         path: build/plist/**/*.html
+        retention-days: 30
       if: ${{ matrix.env.CC == 'clang' && failure() }}
 
     - name: Install fixture (redis)
@@ -336,6 +337,7 @@ jobs:
       with:
         name: clang-fuzzer
         path: build/fuzzer
+        retention-days: 30
       if: ${{ matrix.env.TEST_TYPE == 'fuzzing' && failure() }}
 
     #