Skip to content

fix(ci): skip timing refresh without runs#1596

Open
sd-db wants to merge 1 commit into
mainfrom
sd-db/codex/fix-refresh-test-timings
Open

fix(ci): skip timing refresh without runs#1596
sd-db wants to merge 1 commit into
mainfrom
sd-db/codex/fix-refresh-test-timings

Conversation

@sd-db

@sd-db sd-db commented Jul 17, 2026

Copy link
Copy Markdown
Collaborator

Resolves #

Description

A scheduled timing refresh can legitimately find no eligible integration-test artifacts while the workflow is waiting for a complete sample. Previously, that condition returned a failure and stopped the refresh workflow.

Automatic discovery now exits successfully when no runs yield timing data, allowing the next scheduled refresh to retry. Explicit --run-ids requests still fail when they produce no timings, preserving their validation behavior. The change adds regression coverage for the automatic zero-run path.

Checklist

  • I have run this code in development and it appears to resolve the stated issue
  • This PR includes tests, or tests are not required/relevant for this PR
  • I have updated the CHANGELOG.md and added information about my change to the "dbt-databricks next" section. (not applicable: CI-only timing refresh behavior)

Verification

  • hatch run pytest tests/unit/test_regenerate_timings.py -v
  • hatch run pre-commit run --all-files

@sd-db
sd-db requested a review from jprakash-db as a code owner July 17, 2026 10:28
@github-actions

Copy link
Copy Markdown

Coverage report

This PR does not seem to contain any modification to coverable code.

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