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-2892] [Bug] CI fails on PRs from forks #8257

Closed
2 tasks done
emmyoop opened this issue Jul 31, 2023 · 0 comments · Fixed by #8258
Closed
2 tasks done

[CT-2892] [Bug] CI fails on PRs from forks #8257

emmyoop opened this issue Jul 31, 2023 · 0 comments · Fixed by #8258
Assignees
Labels
bug Something isn't working

Comments

@emmyoop
Copy link
Member

emmyoop commented Jul 31, 2023

Is this a new bug in dbt-core?

  • I believe this is a new bug in dbt-core
  • I have searched the existing issues, and I could not find an existing issue for this bug

Current Behavior

CI fails because it cannot find the value for how many splits to make. It's set as a variable here.

Example failed run: https://github.com/dbt-labs/dbt-core/actions/runs/5711486491/job/15473190823?pr=8222

Expected Behavior

CI runs.

Steps To Reproduce

Open a PR from a fork.

Relevant log output

No response

Environment

n/a

Which database adapter are you using with dbt?

No response

Additional Context

We can either hardcode the value or we can change the trigger to on_pull_request_target

open discussion

@emmyoop emmyoop added bug Something isn't working triage labels Jul 31, 2023
@emmyoop emmyoop self-assigned this Jul 31, 2023
@github-actions github-actions bot changed the title [Bug] CI fails on PRs from forks [CT-2892] [Bug] CI fails on PRs from forks Jul 31, 2023
@emmyoop emmyoop mentioned this issue Jul 31, 2023
4 tasks
@dbeatty10 dbeatty10 removed the triage label Jul 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants