Skip to content

ci(workflow-alerts): Remove release-team webhook calls for slack reports#24059

Merged
rbarker-dev merged 1 commit intomainfrom
24058-ci-workflow-alerts-update-slack-alerts
Mar 6, 2026
Merged

ci(workflow-alerts): Remove release-team webhook calls for slack reports#24059
rbarker-dev merged 1 commit intomainfrom
24058-ci-workflow-alerts-update-slack-alerts

Conversation

@rbarker-dev
Copy link
Member

Description

This pull request removes the step that reports failures to the release team Slack channel from several GitHub Actions workflow files. The Slack notification for failures directed at the release team is no longer sent in any of the affected workflows.

Removed Slack failure notifications from workflows:

  • Node application workflows:

    • Removed the "Report failure (slack release-team)" step from .github/workflows/node-flow-build-application.yaml
    • Removed the "Report failure (slack release-team)" step from .github/workflows/node-flow-deploy-release-artifact.yaml
  • Test and performance workflows:

    • Removed the "Report failure (slack release-team)" step from .github/workflows/zxcron-extended-test-suite.yaml
    • Removed the "Report failure (slack release-team)" step from .github/workflows/zxf-merge-queue-performance-test-controller.yaml
    • Removed the "Report failure (slack release-team)" step from .github/workflows/zxf-merge-queue-performance-test-controller-adhoc.yaml
    • Removed the "Report failure (slack release-team)" step from .github/workflows/zxf-single-day-longevity-test-controller.yaml
    • Removed the "Report failure (slack release-team)" step from .github/workflows/zxf-single-day-performance-test-controller.yaml

Related Issue(s)

Closes #24058

@rbarker-dev rbarker-dev self-assigned this Mar 5, 2026
@rbarker-dev rbarker-dev requested a review from a team as a code owner March 5, 2026 23:25
@rbarker-dev rbarker-dev added the github_actions Pull requests that update Github Actions code. label Mar 5, 2026
@rbarker-dev rbarker-dev linked an issue Mar 5, 2026 that may be closed by this pull request
@trunk-io
Copy link

trunk-io bot commented Mar 5, 2026

Merging to main in this repository is managed by Trunk.

  • To merge this pull request, check the box to the left or comment /trunk merge below.

@lfdt-bot
Copy link

lfdt-bot commented Mar 5, 2026

Snyk checks have passed. No issues have been found so far.

Status Scanner Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@codacy-production
Copy link

Coverage summary from Codacy

See diff coverage on Codacy

Coverage variation Diff coverage
+0.00% (target: -1.00%)
Coverage variation details
Coverable lines Covered lines Coverage
Common ancestor commit (16910e1) 92861 76407 82.28%
Head commit (44e4e28) 92861 (+0) 76407 (+0) 82.28% (+0.00%)

Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch: <coverage of head commit> - <coverage of common ancestor commit>

Diff coverage details
Coverable lines Covered lines Diff coverage
Pull request (#24059) 0 0 ∅ (not applicable)

Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: <covered lines added or modified>/<coverable lines added or modified> * 100%

See your quality gate settings    Change summary preferences

@codecov
Copy link

codecov bot commented Mar 6, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

Impacted file tree graph

@@            Coverage Diff            @@
##               main   #24059   +/-   ##
=========================================
  Coverage     78.28%   78.28%           
  Complexity    11656    11656           
=========================================
  Files          2461     2461           
  Lines         92958    92958           
  Branches       9965     9965           
=========================================
  Hits          72772    72772           
  Misses        16503    16503           
  Partials       3683     3683           

Impacted file tree graph

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@rbarker-dev rbarker-dev merged commit 20ffe51 into main Mar 6, 2026
164 of 173 checks passed
@rbarker-dev rbarker-dev deleted the 24058-ci-workflow-alerts-update-slack-alerts branch March 6, 2026 15:05
aderevets pushed a commit that referenced this pull request Mar 6, 2026
aderevets pushed a commit that referenced this pull request Mar 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

github_actions Pull requests that update Github Actions code.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ci(workflow-alerts): Update slack alerts

3 participants