perry2of5 opened a new pull request, #43385: URL: https://github.com/apache/airflow/pull/43385
The current code throws a KeyError when the environment variable EXCLUDED_PROVIDERS is not defined. This commit adds an empty map as a default to prevent this failure. This was discovered while trying to replicate the issue with microsoft-kiota-http logged here: https://github.com/microsoftgraph/msgraph-sdk-python-core/issues/706 > breeze testing tests --force-lowest-dependencies --test-type "Providers[microsoft.azure]" ### Before: Forcing dependencies to lowest versions for provider: [microsoft-azure] Traceback (most recent call last): File "/opt/airflow/scripts/in_container/is_provider_excluded.py", line 26, in <module> excluded_providers = json.loads(os.environ["EXCLUDED_PROVIDERS"]) File "/usr/local/lib/python3.9/os.py", line 679, in __getitem__ raise KeyError(key) from None KeyError: 'EXCLUDED_PROVIDERS' ### After: Forcing dependencies to lowest versions for provider: [microsoft-azure] Check if provider is excluded in {} Provider is not excluded. -- 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]
