From f034012d78a07f7c1a057c54903b3854267edfbc Mon Sep 17 00:00:00 2001 From: gokul-das-ext Date: Fri, 28 Aug 2026 01:49:31 +0530 Subject: [PATCH] feat(DO-7930): stamp deployment_id into config.yaml for per-attempt notification dedupe ArgoCD's notification oncePer key is moving from app.status.sync.revision to app.metadata.annotations.app_commit to stop unrelated GitOps-repo commits from re-arming notifications for idle apps. That fix breaks rollback: replaying a previously-deployed commit reuses an app_commit value that was already notified, silently suppressing the rollback's own status. Adding a deployment-id input, stamped into config.yaml alongside app_commit, gives the dedupe key a value that's unique per deploy attempt instead of per code state - covering rollback and same-SHA CI re-runs without reintroducing the original over-posting bug. Backward compatible: new input is optional with an empty default, existing callers are unaffected until they pass it explicitly. --- action.yml | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/action.yml b/action.yml index 46a9404..40f895f 100644 --- a/action.yml +++ b/action.yml @@ -117,7 +117,11 @@ inputs: kube-version: description: "Kubernetes version for helm/helmfile rendering (e.g. `1.28`). When provided, skips SSM metadata lookup via chamber." required: false - default: "" + default: "" + deployment-id: + description: "Unique identifier for this specific deploy attempt (e.g. run ID + attempt), used to dedupe ArgoCD notifications per attempt rather than per commit" + required: false + default: '' outputs: webapp-url: description: "Web Application url" @@ -386,6 +390,7 @@ runs: name: ${{ inputs.namespace }}.${{ inputs.application }} namespace: ${{ inputs.namespace }} manifests: ${{ steps.config.outputs.path }}/manifests + deployment_id: ${{ inputs.deployment-id }} - name: Push to Github uses: nick-fields/retry@v4