Skip to content

ci: Skip the regression-report job when no acceptance files changed - #7124

Merged
denik merged 5 commits into
mainfrom
denik/regr-early-skip
Oct 20, 2026
Merged

denik merged 5 commits into
mainfrom
denik/regr-early-skip

Conversation

@denik

@denik denik commented Oct 20, 2026

Copy link
Copy Markdown
Contributor

Follow-up to #7105. The tests are selected from changes under acceptance/, but the job set up the whole build environment and compiled the test binary (~200s) just to find nothing to compare. A lightweight changes job on a hosted runner now checks the diff first and skips the job; it also deletes the comment of an earlier commit, which the skipped job can no longer do.

This pull request and its description were written by Isaac.

denik and others added 4 commits October 20, 2026 15:05
Co-authored-by: Isaac <no-reply@databricks.com>
… IP allow list permits

Co-authored-by: Isaac <no-reply@databricks.com>
Co-authored-by: Isaac <no-reply@databricks.com>
Co-authored-by: Isaac <no-reply@databricks.com>
@denik
denik marked this pull request as ready for review October 20, 2026 13:39
@denik
denik requested a review from a team as a code owner October 20, 2026 13:39
@denik
denik requested a review from shreyas-goenka October 20, 2026 13:39
@eng-dev-ecosystem-bot

Copy link
Copy Markdown
Collaborator

Integration test report

Commit: f75f009

Run: 39433953724

Env ✅​pass 🙈​skip Time
✅​ aws linux-2core-8gb 280 15 4:33
✅​ aws-windows-latest-4core-16gb 282 13 5:15
✅​ azure linux-2core-8gb 279 15 4:28
✅​ azure-windows-latest-4core-16gb 281 13 4:03
✅​ gcp linux-2core-8gb 280 15 4:52
✅​ gcp-windows-latest-4core-16gb 282 13 3:32
Top 3 slowest tests (at least 2 minutes):
duration env testname
5:08 aws-windows-latest-4core-16gb TestAccept
3:57 azure-windows-latest-4core-16gb TestAccept
3:26 gcp-windows-latest-4core-16gb TestAccept

@denik
denik added this pull request to the merge queue Oct 20, 2026
Merged via the queue into main with commit 649e21e Oct 20, 2026
28 checks passed
@denik
denik deleted the denik/regr-early-skip branch October 20, 2026 16:24
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.

3 participants