antonio-mello-ai opened a new pull request, #63478:
URL: https://github.com/apache/airflow/pull/63478

   ## Summary
   
   - Fix `KubernetesHook` so that `disable_verify_ssl=True` actually disables 
SSL verification on the returned `ApiClient`
   - The root cause: `load_kube_config()` overwrites 
`Configuration.set_default()` with a fresh `Configuration(verify_ssl=True)`, so 
the earlier `_disable_verify_ssl()` call is lost
   - Fix: re-apply `verify_ssl=False` after each config load, and pass the 
configuration explicitly to `_TimeoutK8sApiClient`
   
   ## Root Cause
   
   `_disable_verify_ssl()` calls `Configuration.set_default()` to register 
`verify_ssl=False` globally. However, `load_kube_config()` (called right after) 
creates a new `Configuration` instance with `verify_ssl=True` by default and 
overwrites the global default. The `_TimeoutK8sApiClient()` then gets 
`verify_ssl=True` from `get_default_copy()`.
   
   ## Test Plan
   
   - [x] New parametrized test covering all 4 config paths (`kube_config_path`, 
`kube_config`, `config_dict`, `default_client`)
   - [x] All 138 tests pass
   - [x] All pre-commit hooks pass (`prek`)
   
   Closes #56432
   
   🤖 Generated with [Claude Code](https://claude.com/claude-code)
   
   Co-Authored-By: Claude Opus 4.6 <[email protected]>


-- 
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]

Reply via email to