-
Notifications
You must be signed in to change notification settings - Fork 55
Add proxy support to Kubernetes client configuration in get_client #2291
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
myakove
merged 34 commits into
RedHatQE:main
from
sbahar619:CNV-46351/add-proxy-support-to-k8s-client-configuration
Jan 30, 2025
Merged
Changes from 27 commits
Commits
Show all changes
34 commits
Select commit
Hold shift + click to select a range
0ed97d6
Add proxy support to Kubernetes client configuration in get_client
sbahar619 e58e6c8
Simplify docstring by removing redundant default value explanation
sbahar619 705372c
Update proxy configuration logic and README documentation
sbahar619 0f89b00
Simplify use_proxy docstring in get_client method
sbahar619 0fb9702
Add support for enabling proxy with use_proxy flag
sbahar619 a9b4c6f
Remove client_configuration from get_client and initialize only for p…
sbahar619 479f42f
Refactor proxy handling in get_client function
sbahar619 251eee6
Refactor docstring to clarify proxy behavior
sbahar619 6d43194
Simplify proxy configuration logic in get_client
sbahar619 6bf30f6
Improve error message for missing proxy configuration in get_client
sbahar619 948fac0
Remove redundant dot in README
sbahar619 90d5bbb
Remove unnecessary check for client_configuration in get_client
sbahar619 595cea7
Update README to clarify proxy enablement instructions
sbahar619 6bea2c9
Clarify proxy enablement in README
sbahar619 84b9023
Clarify use_proxy argument type in README
sbahar619 1898f5c
Retrieve client_configuration from kwargs if provided
sbahar619 59367f4
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] 776c447
Ensure environment variable overrides use_proxy
sbahar619 4a92fbb
Remove redundant proxy configuration log
sbahar619 f6673b2
Preserve client_configuration from kwargs when setting proxy
sbahar619 f375dad
Update docstring to clarify proxy conf behavior with env var
sbahar619 040a1c9
Refactor proxy configuration logic for clarity
sbahar619 3f86433
Update README with proxy enablement details
sbahar619 10ee9f2
Refactor client_configuration retrieval in get_client
sbahar619 cbf9f0d
Refactor proxy configuration in get_client for improved clarity
sbahar619 cae73aa
Refactor use_proxy argument in get_client to support None as default …
sbahar619 e3e958c
Merge remote-tracking branch 'upstream/main' into CNV-46351/add-proxy…
sbahar619 b010e7b
Remove use_proxy argument from get_client and update README
sbahar619 148d834
Ensure proxy settings are always applied when enabled
sbahar619 aa63891
Check for proxy conflicts in client_configuration and environment var…
sbahar619 de9c6a8
Add test to verify proxy conflict raises ValueError for mismatched pr…
sbahar619 85965ca
Refactor test for proxy conflict in get_client method
sbahar619 b617c53
Refactor test for proxy conflict to use pytest.raises match exp
sbahar619 798ef9e
Add test for proxy enabled without proxy env variables set
sbahar619 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
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.
Uh oh!
There was an error while loading. Please reload this page.