Skip to content

ci: align the daily integration report with the other SDKs - #93

Merged
mogita merged 2 commits into
masterfrom
fix/cha-5511-ci-parity
Sep 23, 2026
Merged

mogita merged 2 commits into
masterfrom
fix/cha-5511-ci-parity

Conversation

@mogita

@mogita mogita commented Sep 23, 2026

Copy link
Copy Markdown
Collaborator

Ticket

CHA-5511

Problem

Reviews on the later SDKs in this series found gaps in the daily integration workflow that this repo, merged earlier, still has.

Solution

The report fires on any non-success result, not just failure(): a job that hits
timeout-minutes concludes cancelled. It only touches the tracking issue on the default
branch. The cron comment no longer claims java runs at 09:00.

How to verify

actionlint on the changed workflows.

The report fires on any non-success result, not just failure(): a job that hits
timeout-minutes concludes cancelled. It only touches the tracking issue on the default
branch. The cron comment no longer claims java runs at 09:00.
Comment thread .github/workflows/scheduled_test.yml Outdated
…pository

A schedule event carries no repository object, so github.event.repository.default_branch
is empty on the cron run and the default-branch guard was false: a red daily run filed
nothing. A schedule always runs on the default branch, so it passes by event name.
@mogita
mogita merged commit 610529d into master Sep 23, 2026
6 checks passed
@mogita
mogita deleted the fix/cha-5511-ci-parity branch September 23, 2026 14:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant