Feature/add vendor data - #285
Merged
Merged
Conversation
Contributor
🧪 dbt Workflow Status
✅ All dbt validations passed |
* infra(snowcap): declare the Claude MCP OAuth security integration
Bring CLAUDE_MCP_OAUTH under snowcap management (OAUTH_CLIENT = CUSTOM). Pre-authorizes
account_monitor -- the role the connector's token binds to -- not the retired z_monitor
from the original DDL. snowcap reconciles the existing integration with ALTER, never a
CREATE OR REPLACE, so the client_id/client_secret and live refresh tokens are preserved.
* chore(airflow): land example DAGs that only lived on airflow_development
airflow_development had not merged main since 2026-01-03 and was
rebuilt from it. These five files were the only content genuinely
unique to the old branch, so they land on main here rather than staying
on a long-lived branch where they can be lost the next time it is
rebuilt.
- other_examples/dbt_templated_overrides_dag.py: decorator overrides
example (was only on airflow_development)
- other_examples/test_oom.py: OOM alerting test DAG, renamed from
test_oom_fernando.py and moved in alongside the other examples
- dbt_dag.yml: task-level retries/execution_timeout example. Set in
the yml because dbt_dag.py is generated from it; regenerate the py
to pick it up
- retry_dbt_failures.py: owner_email
daily_loan_run runs in two Datacoves clusters whose Airbyte connection
ids differ. The test cluster used to carry a local edit to
daily_loan_run.py that reverted on every sync; it now has its own DAG,
daily_loan_run__dc_test_cluster, and daily_loan_run is unchanged.
* fix(airflow): drop node-level execution_timeout from dbt_dag.yml
dbt-coves resolves the !py tag only for DAG-level keys. On a node it
falls through the isinstance(value, str) branch in generate_task_output
and is emitted quoted, so the generated DAG got
execution_timeout="pendulum.duration(minutes=15)"
which Airflow rejects -- execution_timeout must be a timedelta. retries: 0
generates correctly, so keep that and drop the timeout until dbt-coves
honours RawExpr at node level.
* Update dags to match airflow_development
* add snowcap account defaults
* chore: empty commit to retrigger CI
Contributor
|
Contributor
|
Contributor
🎯 Airflow Workflow Status
✅ All Airflow validations passed |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.