Skip to content

Migrate dbt Cloud connection UI metadata to YAML - #62579

Merged
amoghrajesh merged 2 commits into
apache:mainfrom
josh-fell:dbt-cloud-conn-form
Mar 11, 2026
Merged

Migrate dbt Cloud connection UI metadata to YAML#62579
amoghrajesh merged 2 commits into
apache:mainfrom
josh-fell:dbt-cloud-conn-form

Conversation

@josh-fell

@josh-fell josh-fell commented Feb 27, 2026

Copy link
Copy Markdown
Contributor

This change adds connection UI fields explicitly as provider-related configuration for the dbt Cloud provider.

Related: #62345

@josh-fell josh-fell changed the title Migrate dbt Cloud connection UI fields to YAML config Migrate dbt Cloud connection UI metadata to YAML Feb 27, 2026
@josh-fell

josh-fell commented Feb 27, 2026

Copy link
Copy Markdown
Contributor Author

@amoghrajesh Should we remove the legacy methods from the hook now or is the intention for the removal to be a "fast-follow" once a new version of the provider is release with the YAML config?

Nevermind, you mention leaving the methods in the hook in #60410 and eventually deprecating them.

@josh-fell
josh-fell force-pushed the dbt-cloud-conn-form branch from 19e6851 to d75423a Compare March 9, 2026 14:34
@amoghrajesh amoghrajesh added the full tests needed We need to run full set of tests for this PR to merge label Mar 10, 2026
@amoghrajesh amoghrajesh reopened this Mar 10, 2026
This change adds the corresponding connection configuration for UI
fields directly in the provider.yaml file.
@josh-fell
josh-fell force-pushed the dbt-cloud-conn-form branch from d75423a to f52414d Compare March 10, 2026 13:13
@amoghrajesh

Copy link
Copy Markdown
Contributor

@josh-fell I see some failures in the CI, seems unrelated but can you confirm?

FAILED airflow-core/tests/unit/core/test_configuration.py::TestDeprecatedConf::test_conf_as_dict_when_deprecated_value_in_secrets[True] - AssertionError: assert equals failed
  None                (                  
                        'postgresql://', 
                        'secret',        
                      )
FAILED airflow-core/tests/unit/core/test_configuration.py::TestDeprecatedConf::test_conf_as_dict_when_deprecated_value_in_secrets[False] - AssertionError: assert equals failed
  None             'postgresql://'

@amoghrajesh
amoghrajesh merged commit 13789d0 into apache:main Mar 11, 2026
133 checks passed
dominikhei pushed a commit to dominikhei/airflow that referenced this pull request Mar 11, 2026
This change adds the corresponding connection configuration for UI
fields directly in the provider.yaml file.

Co-authored-by: Amogh Desai <amoghrajesh1999@gmail.com>
Pyasma pushed a commit to Pyasma/airflow that referenced this pull request Mar 13, 2026
This change adds the corresponding connection configuration for UI
fields directly in the provider.yaml file.

Co-authored-by: Amogh Desai <amoghrajesh1999@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:providers full tests needed We need to run full set of tests for this PR to merge provider:dbt-cloud

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants