Skip to content

Document sourceCodeLocation facet requirement for dbt Core CI/CD#38327

Open
himakolavennu wants to merge 1 commit into
masterfrom
himakolavennu/dbt-ci-sourcecodelocation-facet
Open

Document sourceCodeLocation facet requirement for dbt Core CI/CD#38327
himakolavennu wants to merge 1 commit into
masterfrom
himakolavennu/dbt-ci-sourcecodelocation-facet

Conversation

@himakolavennu

Copy link
Copy Markdown
Contributor

What does this PR do? What is the motivation?

Drift Detection for dbt Core requires the sourceCodeLocation facet (carrying the pull request number) on the OpenLineage events sent from CI. That facet is opt-in in openlineage-dbt >= 1.46.0 (disabled=true by default), so without configuration no facet is emitted and drift checks never run. The docs didn't mention this.

This PR documents:

  • The required OPENLINEAGE__FACETS__SOURCE_CODE_LOCATION__DISABLED=false environment variable (and the >= 1.46.0 version requirement) in the dbt Core setup.
  • How the pull request number is resolved: auto-detected from GITHUB_REF (GitHub Actions PR workflows) or CI_MERGE_REQUEST_IID (GitLab MR pipelines), or set explicitly via OPENLINEAGE__FACETS__SOURCE_CODE_LOCATION__PULL_REQUEST_NUMBER.
  • A note in the CI/CD Drift Detection section pointing dbt Core users to the requirement.

Merge readiness

  • Ready for merge

AI assistance

Investigation, root-cause analysis, and initial draft with Claude Code; reviewed and edited manually.

Drift Detection for dbt Core requires the sourceCodeLocation facet (with the pull request number) on OpenLineage events. This facet is opt-in in openlineage-dbt >= 1.46.0, so document the required OPENLINEAGE__FACETS__SOURCE_CODE_LOCATION__DISABLED=false env var and how the PR number is detected (GITHUB_REF / CI_MERGE_REQUEST_IID) or set explicitly.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@github-actions

Copy link
Copy Markdown
Contributor

@himakolavennu
himakolavennu marked this pull request as ready for review July 20, 2026 18:41
@himakolavennu
himakolavennu requested a review from a team as a code owner July 20, 2026 18:41
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.

2 participants