python icon indicating copy to clipboard operation
python copied to clipboard

Changes made in configuration.py to accept environmental variables

Open p172913 opened this issue 7 months ago • 15 comments

What type of PR is this? /kind bug

What this PR does / why we need it: This PRs will read environment variables assigned for proxy and no_proxy.

Which issue(s) this PR fixes: Fixes https://github.com/kubernetes-client/python/issues/2321

Special notes for your reviewer: Unit test is added kubernetes\base\stream\ws_client_test.py

Does this PR introduce a user-facing change? None

Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.: None

p172913 avatar Apr 29 '25 06:04 p172913

@roycaihw , @fabianvf Please approve my PR

p172913 avatar Apr 29 '25 18:04 p172913

@roycaihw , @fabianvf Please review my PR...

p172913 avatar May 01 '25 19:05 p172913

@roycaihw , @fabianvf Please review my PR...

p172913 avatar May 05 '25 15:05 p172913

/assign

roycaihw avatar May 07 '25 20:05 roycaihw

Please remove the "Merge branch 'kubernetes-client:master' into master" commit to make the commits clear. You may have to start from a new branch to do so.

This commit is reverted back.

p172913 avatar May 28 '25 10:05 p172913

I added a bash script file. This will edit configuration.py file if we create it using new opneapi generator.

p172913 avatar May 28 '25 10:05 p172913

@roycaihw I have been working get this upstream: https://github.com/OpenAPITools/openapi-generator/pull/21394

Edit: I ceased working on it for the time being.

bachorp avatar Jun 08 '25 07:06 bachorp

@roycaihw I have been working get this upstream: OpenAPITools/openapi-generator#21394

Oh ok. Then @roycaihw I will remove statement to execute insert_proxy_config.sh from scripts/release.sh. So, that we can execute insert_proxy_config.sh only if needed instead of running always....

p172913 avatar Jun 08 '25 10:06 p172913

@roycaihw I have been working get this upstream: OpenAPITools/openapi-generator#21394

Oh ok. Then @roycaihw I will remove statement to execute insert_proxy_config.sh from scripts/release.sh. So, that we can execute insert_proxy_config.sh only if needed instead of running always....

@roycaihw What do you think...

p172913 avatar Jun 09 '25 16:06 p172913

Could you squash the commits? Otherwise LGTM. https://www.kubernetes.dev/docs/guide/github-workflow/#squash-commits

roycaihw avatar Jun 09 '25 23:06 roycaihw

Could you squash the commits? Otherwise LGTM. https://www.kubernetes.dev/docs/guide/github-workflow/#squash-commits

@roycaihw squashed commits as per instructions.

p172913 avatar Jun 10 '25 11:06 p172913

Could you squash the commits? Otherwise LGTM. https://www.kubernetes.dev/docs/guide/github-workflow/#squash-commits

@roycaihw squashed commits as per instructions.

@roycaihw please approve my commit.

p172913 avatar Jun 12 '25 16:06 p172913

Could you squash the commits? Otherwise LGTM. https://www.kubernetes.dev/docs/guide/github-workflow/#squash-commits

@roycaihw squashed commits as per instructions.

@roycaihw please approve my commit.

@roycaihw Do i need to add anything to this...?

p172913 avatar Jun 16 '25 16:06 p172913

@roycaihw Can you please look into my PR. I squashed my commits as you mentioned. Please approve my Pull request....

p172913 avatar Jun 23 '25 15:06 p172913

/lgtm /approve

Thank you!

roycaihw avatar Jun 24 '25 19:06 roycaihw

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: p172913, roycaihw

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment Approvers can cancel approval by writing /approve cancel in a comment

k8s-ci-robot avatar Jun 24 '25 19:06 k8s-ci-robot