Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Add default kube config location #64

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
merged 1 commit into from
Dec 15, 2016
Merged

Conversation

mbohlool
Copy link
Contributor

@mbohlool mbohlool commented Dec 14, 2016

kubenetes.config.load_config needs the location of config file to load it. We can set the location to the default location of ~/.kube/config.

@mbohlool mbohlool added the kind/feature Categorizes issue or PR as related to a new feature. label Dec 14, 2016
@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Dec 14, 2016
@codecov-io
Copy link

codecov-io commented Dec 14, 2016

Current coverage is 93.40% (diff: 42.85%)

Merging #64 into master will decrease coverage by 0.58%

@@             master        #64   diff @@
==========================================
  Files             9          9          
  Lines           632        637     +5   
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
+ Hits            594        595     +1   
- Misses           38         42     +4   
  Partials          0          0          

Powered by Codecov. Last update 8d536e6...fff29bf

@mbohlool mbohlool changed the title [WIP] Add default kube config location Add default kube config location Dec 14, 2016
def list_kube_config_contexts(config_file):
def list_kube_config_contexts(config_file=None):

if not config_file:
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would use if config_file is not None here.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You mean if config_file is None right?

@@ -259,6 +264,9 @@ def load_kube_config(config_file, context=None):
from config file will be used.
"""

if not config_file:
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would use if config_file is not None here.

@mbohlool mbohlool force-pushed the c4 branch 3 times, most recently from 8d0607c to fff29bf Compare December 15, 2016 22:46
@mbohlool mbohlool merged commit 729d543 into kubernetes-client:master Dec 15, 2016
yliaog added a commit to yliaog/client-python that referenced this pull request Jan 8, 2022
Watch properly decode resourceVersion from custom object response
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. kind/feature Categorizes issue or PR as related to a new feature.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants