Skip to content

ci: retry gh api in scheduled-tests Slack notification#1898

Merged
vdusek merged 1 commit into
masterfrom
ci/notify-slack-retry-gh-api
May 15, 2026
Merged

ci: retry gh api in scheduled-tests Slack notification#1898
vdusek merged 1 commit into
masterfrom
ci/notify-slack-retry-gh-api

Conversation

@vdusek
Copy link
Copy Markdown
Collaborator

@vdusek vdusek commented May 15, 2026

The Build Slack payload step aborted on a transient HTTP 502 from gh api, so no Slack notification was sent. Retry the call (up to 5 attempts, linear backoff) and fall back to a placeholder message on persistent failure.

Failing run: https://github.com/apify/crawlee-python/actions/runs/25905726146/job/76143180252

The `Build Slack payload` step fetches the failed-job list with `gh api`
and aborted on a transient HTTP 502, leaving us without a Slack
notification. Retry up to 5 times with linear backoff and fall back to a
placeholder message if the API stays unreachable.
@vdusek vdusek added t-tooling Issues with this label are in the ownership of the tooling team. adhoc Ad-hoc unplanned task added during the sprint. labels May 15, 2026
@vdusek vdusek self-assigned this May 15, 2026
@vdusek vdusek requested a review from Mantisus May 15, 2026 10:35
@github-actions github-actions Bot added this to the 140th sprint - Tooling team milestone May 15, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented May 15, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 92.83%. Comparing base (722c96b) to head (d0ae223).

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1898   +/-   ##
=======================================
  Coverage   92.83%   92.83%           
=======================================
  Files         167      167           
  Lines       11699    11699           
=======================================
  Hits        10861    10861           
  Misses        838      838           
Flag Coverage Δ
unit 92.83% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@vdusek vdusek merged commit 41cd17e into master May 15, 2026
32 checks passed
@vdusek vdusek deleted the ci/notify-slack-retry-gh-api branch May 15, 2026 10:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

adhoc Ad-hoc unplanned task added during the sprint. t-tooling Issues with this label are in the ownership of the tooling team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants