Skip to content

Add final_project_pipeline_dev (VM-only dev integration DAG) - #32

Merged
lassebenni merged 1 commit into
mainfrom
feat/pipeline-dag-dev
Aug 27, 2026
Merged

Add final_project_pipeline_dev (VM-only dev integration DAG)#32
lassebenni merged 1 commit into
mainfrom
feat/pipeline-dag-dev

Conversation

@lassebenni

Copy link
Copy Markdown
Collaborator

What I built

Adds data/airflow/dags/pipeline_dag_dev.py: VM-only final_project_pipeline_dev for manual integration on the team Airflow VM (dev_airflow, job-fp-ingest-dev, analytics_dev). pipeline_dag.py is unchanged.

Why this approach

Rollout for hyf-datatrack#362. HYF maintains this file in a separate module so students keep editing pipeline_dag.py without merge conflicts. Helpers are imported inside tasks only (not at module scope) so CI dag-integrity does not see duplicate final_project_pipeline registrations. Laptops with DATABRICKS_TOKEN in .env skip this file; only the team VM loads it.

Contract impact

None. Dev DAG publishes to analytics_dev only; prod analytics contract unchanged.

How to run

On a team Airflow VM after merge and git pull:

airflow dags unpause final_project_pipeline_dev
airflow dags trigger final_project_pipeline_dev

Locally (Astro): only final_project_pipeline loads.

Self-check

  • ruff + black pass on pipeline_dag_dev.py
  • No secrets, tokens, or connection strings in the diff
  • This pull request does one thing

Made with Cursor

HYF-maintained dev integration DAG for team VMs (dev_airflow,
job-fp-ingest-dev, analytics_dev). Separate from pipeline_dag.py so
student merges stay conflict-free; lazy-imports pipeline_dag helpers so
DagBag does not register the prod DAG twice.

Co-authored-by: Cursor <cursoragent@cursor.com>
Copilot AI lite review requested due to automatic review settings August 27, 2026 05:29

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@lassebenni
lassebenni merged commit 029ddc8 into main Aug 27, 2026
7 of 8 checks passed
@lassebenni
lassebenni deleted the feat/pipeline-dag-dev branch August 27, 2026 05:35
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