Fix: Feed notification (#117)

* Fix: Change notification channel to TOD:FeedDeployment:Notification * Fix: Remove startWith(github.ref ...) from all workflows using notify-mattermost* * Fix: Add input parameter for notification channel * Fix: Forward notifcation-channel through container-build-push* workflows * Fix: Set default channel to todfeeddeploymentnotification * Update .github/workflows/container-build-push-2nd-gen.yml Co-authored-by: Jaspar Stach <[email protected]> * Fix: Remove notify from container-build-push-feed * Fix: Channel name * Fix: Readd notify * Update .github/workflows/notify-mattermost-2nd-gen.yml Co-authored-by: Jaspar Stach <[email protected]> * Update container-build-push-feed.yml * Update .github/workflows/container-build-push-feed.yml Co-authored-by: Jaspar Stach <[email protected]> --------- Co-authored-by: Robert Schardt <[email protected]> Co-authored-by: Jaspar Stach <[email protected]> Co-authored-by: Robert Schardt <[email protected]>
greenbone · Oct 22, 2024 · 1b1d879 · 1b1d879
1 parent c559da7
commit 1b1d879
Show file tree

Hide file tree

Showing 4 changed files with 16 additions and 2 deletions.
diff --git a/.github/workflows/container-build-push-2nd-gen.yml b/.github/workflows/container-build-push-2nd-gen.yml
@@ -38,6 +38,10 @@ on:
         description: "The artifact path to extract the artifact into."
         required: false
         type: string
+      notification-channel:
+        description: "The name of the channel where the notification happens."
+        type: string
+        default: "pd2ndgendeployment"
 
     secrets:
       COSIGN_KEY_OPENSIGHT:
@@ -272,4 +276,5 @@ jobs:
     uses: greenbone/workflows/.github/workflows/notify-mattermost-2nd-gen.yml@main
     with:
       status: ${{ contains(needs.*.result, 'failure') && 'failure' || 'success' }}
+      channel: ${{ inputs.notification-channel }}
     secrets: inherit
diff --git a/.github/workflows/container-build-push-feed.yml b/.github/workflows/container-build-push-feed.yml
@@ -42,6 +42,10 @@ on:
         description: "Is this the vts image?"
         default: "false"
         type: string
+      notification-channel:
+        description: "The name of the channel where the notification happens."
+        type: string
+        default: "pdfeeddeploymentnotification"
 
     secrets:
       COSIGN_KEY_OPENSIGHT:
@@ -207,4 +211,5 @@ jobs:
     uses: greenbone/workflows/.github/workflows/notify-mattermost-2nd-gen.yml@main
     with:
       status: ${{ contains(needs.*.result, 'failure') && 'failure' || 'success' }}
+      channel: ${{ inputs.notification-channel }}
     secrets: inherit
diff --git a/.github/workflows/helm-container-build-push-3rd-gen.yml b/.github/workflows/helm-container-build-push-3rd-gen.yml
@@ -248,7 +248,7 @@ jobs:
       - building-service-chart
       - building-product-chart
       - building-product-compose
-    if: ${{ !cancelled() && startsWith(github.ref, 'refs/tags/v')  && startsWith(inputs.notify, 'true') }}
+    if: ${{ !cancelled()  && startsWith(inputs.notify, 'true') }}
     uses: greenbone/workflows/.github/workflows/notify-mattermost-3rd-gen.yml@main
     with:
       status: ${{ contains(needs.*.result, 'failure') && 'failure' || 'success' }}

diff --git a/.github/workflows/notify-mattermost-2nd-gen.yml b/.github/workflows/notify-mattermost-2nd-gen.yml
@@ -19,6 +19,10 @@ on:
         description: "The monitored job, job status."
         type: string
         required: true
+      channel:
+        description: "The name of the channel where the notification happens."
+        type: string
+        default: pd2ndgendeployment
     # Dependabot don't have this secrets and on PR's this secrets are not needed.
     secrets:
       MATTERMOST_WEBHOOK_URL:
@@ -33,7 +37,7 @@ jobs:
         uses: greenbone/actions/mattermost-notify@v3
         with:
           url: ${{ secrets.MATTERMOST_WEBHOOK_URL }}
-          channel: "pd2ndgendeployment"
+          channel: ${{ inputs.channel }}
           highlight: ${{ inputs.highlight }}
           branch: ${{ github.ref_name }}
           commit: ${{ inputs.commit }}