Merge pull request #202 from whyscream/better-workflows

Better workflows
whyscream · Jul 4, 2024 · 3982b33 · 3982b33
2 parents 5dc376a + c13b5da
commit 3982b33
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 5 deletions.
diff --git a/.github/workflows/test_config_syntax.yml b/.github/workflows/test_config_syntax.yml
@@ -1,13 +1,20 @@
 name: Validate configuration syntax
-on: [push]
+on: [push, workflow_dispatch]
 jobs:
   test-syntax:
     runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        logstash-version: ['8.14.1', '7.17.22']
     steps:
-      - uses: actions/checkout@v4
-      - run: |
+      - name: Checkout code
+        uses: actions/checkout@v4
+      - name: Validate configuration syntax
+        env:
+          LOGSTASH_VERSION: ${{matrix.logstash-version}}
+        run: |
           docker run --rm \
           --volume ./postfix.grok:/etc/logstash/patterns.d/postfix.grok \
           --volume ./50-filter-postfix.conf:/usr/share/logstash/pipeline/50-filter-postfix.conf \
-          logstash:8.12.0 \
+          logstash:${LOGSTASH_VERSION} \
           logstash --config.test_and_exit -f /usr/share/logstash/pipeline/50-filter-postfix.conf
diff --git a/.github/workflows/test_grok_patterns.yml b/.github/workflows/test_grok_patterns.yml
@@ -1,5 +1,5 @@
 name: Test grok patterns
-on: [push]
+on: [push, workflow_dispatch]
 jobs:
   test-patterns:
     runs-on: ubuntu-latest