Skip to content

ci: arm64-only failures do not trigger Slack notification (matrix fan-in needed) #288

Description

@coderabbitai

Summary

In the current matrix CI workflow (PR #287), the Slack notification steps (Set Slack notify status / Notify Slack) are gated on matrix.arch == 'amd64' and evaluated entirely within the amd64 matrix leg. Because failure() in GitHub Actions is job-scoped, an arm64-only failure leaves the amd64 context unaffected — no Slack alert fires.

Expected behavior

Failures in any matrix architecture (amd64 or arm64) should trigger a Slack notification.

Suggested fix

Add a dedicated fan-in job (e.g., notify-slack) that:

  1. needs the full matrix job (build-test-push).
  2. Runs with if: always().
  3. Inspects needs.build-test-push.result (or needs.*.result) to detect any failed architecture.
  4. Passes the aggregated publish/failure status to the Slack notify script once, replacing the per-leg notification logic.

References

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions