neronsoda commented on PR #60074: URL: https://github.com/apache/airflow/pull/60074#issuecomment-3707954214
Hi 👋 The commit for this change is already prepared, and before commit, I’d like to ask for a confirmation on the proposed approach. While running `breeze testing providers-tests --test-type "Providers[cncf.kubernetes]"`, I observed multiple failures caused by SDK `conf` accessing provider configuration values at import time. The root cause appears to be that SDK `conf` does not include `provider_config_fallback_defaults.cfg` in its lookup chain. To address this, I added explicit `fallback` values (aligned with existing provider defaults) to the relevant `conf.get*()` calls. With this change, all Kubernetes provider tests are passing. Before commit, I’d like to confirm whether adding explicit fallbacks in provider code is considered an acceptable approach. Once confirmed, I’ll proceed with the commit. Thanks! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
