Skip to content

ci: run docker image workflow cron once daily at 07:00 IST - #42170

Merged
subrata71 merged 1 commit into
releasefrom
ci/app-15858
Aug 28, 2026
Merged

ci: run docker image workflow cron once daily at 07:00 IST#42170
subrata71 merged 1 commit into
releasefrom
ci/app-15858

Conversation

@subrata71

@subrata71 subrata71 commented Aug 28, 2026

Copy link
Copy Markdown
Collaborator

Description

Reduce the weekday schedule for Test, build and push Docker Image from three runs (07:00, 11:00, 15:00 IST) to a single 07:00 IST run.

  • Before: 30 1-11/4 * * 1-5 → 01:30, 05:30, 09:30 UTC
  • After: 30 1 * * 1-5 → 01:30 UTC = 07:00 IST, weekdays

CE-only change. The hourly sync will carry the same cron into EE. GitHub scheduled workflows run from the default branch (release).

Why a single 07:00 IST run

  • Each TBP is a full Cypress suite (~60 shards). Typical duration is ~67 min (CE) / ~79 min (EE); retries stretch to 3–5 hours. Three weekday runs are expensive for little extra signal.
  • Scheduled TBP failure rate is high (CE ~33%, EE ~45% in recent runs). Later slots often contradict the morning result, which is flake, not additional coverage.
  • The 11:00 IST slot starts in the same window as auto-promotion (30 5 * * 1-5 = 11:00 IST). GitHub often delays TBP until 11:10–11:47, so it is still running when promotion needs a completed SHA-matching run.
  • A flaky 11:00 run on the same SHA can become the latest completed TBP and block promotion of an already-green 07:00 run.
  • The 15:00 IST run is after promotion and cannot unblock that day's promotion.

Fixes https://linear.app/appsmith/issue/APP-15858

Automation

/ok-to-test tags=""

🔍 Cypress test results

Caution

If you modify the content in this section, you are likely to disrupt the CI result for your PR.

Communication

Should the DevRel and Marketing teams inform users about this change?

  • Yes
  • No

Summary by CodeRabbit

  • Chores
    • Updated the automated build check schedule to run once each weekday morning instead of three times daily.

Drop the 11:00 and 15:00 IST weekday runs so the scheduled Test, build and push Docker Image workflow fires only at 01:30 UTC.
@linear-code

linear-code Bot commented Aug 28, 2026

Copy link
Copy Markdown

APP-15858

@coderabbitai

coderabbitai Bot commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: efae9442-eea4-458a-9c71-5fea5b3caed4

📥 Commits

Reviewing files that changed from the base of the PR and between 4d8d6a3 and f8e4aa5.

📒 Files selected for processing (1)
  • .github/workflows/test-build-docker-image.yml

Included review availability: 4 reviews are currently available. Your included PR review attempts over the past 7 days set your current allowance at 5 reviews per hour.


Walkthrough

The scheduled Docker image test workflow now runs once at 7:00 AM IST on weekdays instead of three times daily.

Changes

Docker workflow schedule

Layer / File(s) Summary
Update weekday execution schedule
.github/workflows/test-build-docker-image.yml
The weekday cron schedule now runs at 7:00 AM IST only.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Merge Risk: ⚪ Minimal · up to f8e4a

This change reduces the Docker image workflow to one weekday run at 07:00 IST. No actionable merge-blocking risk remains beyond normal checks and review.

Suggested reviewers: salevine, amelia-c0n

Poem

One morning run takes flight,
The Docker checks align just right.
No noon bell rings,
No afternoon springs,
Weekdays begin with build-light.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely describes the main change: reducing the Docker image workflow cron to one weekday run at 07:00 IST.
Description check ✅ Passed The description is complete and relevant. It explains the schedule change, motivation, timing details, issue reference, automation section, test-results section, and communication decision.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch ci/app-15858

Warning

Your free Security trial is over. An organization admin can activate Security or dismiss this notice.


Comment @coderabbitai help to get the list of available commands.

@github-actions github-actions Bot added skip-changelog Adding this label to a PR prevents it from being listed in the changelog CI labels Aug 28, 2026
@subrata71
subrata71 marked this pull request as ready for review August 28, 2026 08:44
@subrata71
subrata71 requested a review from a team as a code owner August 28, 2026 08:44
@subrata71
subrata71 merged commit acb91d7 into release Aug 28, 2026
21 checks passed
@subrata71
subrata71 deleted the ci/app-15858 branch August 28, 2026 09:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI skip-changelog Adding this label to a PR prevents it from being listed in the changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants