Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[CT-2613] TestFastFailingDuringRun::test_fail_fast_run is flaky #7744

Closed
Tracked by #7808
MichelleArk opened this issue May 31, 2023 · 0 comments · Fixed by #7767
Closed
Tracked by #7808

[CT-2613] TestFastFailingDuringRun::test_fail_fast_run is flaky #7744

MichelleArk opened this issue May 31, 2023 · 0 comments · Fixed by #7767
Assignees
Labels
flaky_test tech_debt Behind-the-scenes changes, with little direct impact on end-user functionality

Comments

@MichelleArk
Copy link
Contributor

Example failing CI run: https://github.com/dbt-labs/dbt-core/actions/runs/5126607618/jobs/9236277629?pr=7739

13:43:46  Database Error in model two (models\two.sql)
13:43:46    column "failed" does not exist
13:43:46    LINE 7:     select 1 /failed
...
FAILED tests/functional/fail_fast/test_fail_fast_run.py::TestFastFailingDuringRun::test_fail_fast_run - IndexError: list index out of range

Test passes on other builds + main.

Looking closer at the failing test, I don't really see a reason why the /failed statement is necessary in "two.sql" - unless there is a dependency between the two models -- "two.sql" could run first occasionally and cause the test to fail.

@github-actions github-actions bot changed the title TestFastFailingDuringRun::test_fail_fast_run is flaky [CT-2613] TestFastFailingDuringRun::test_fail_fast_run is flaky May 31, 2023
@jtcohen6 jtcohen6 mentioned this issue Jun 5, 2023
6 tasks
@MichelleArk MichelleArk added the tech_debt Behind-the-scenes changes, with little direct impact on end-user functionality label Jun 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
flaky_test tech_debt Behind-the-scenes changes, with little direct impact on end-user functionality
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant