Infra/claude mcp oauth - #286
Merged
Merged
Conversation
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.
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.
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.
Contributor
🎯 Airflow Workflow Status
❌ Branch validation failed |
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
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.