Skip to content
This repository has been archived by the owner on Mar 13, 2022. It is now read-only.

Refactor auth-provider code paths a little. Add Azure support. #74

Merged
merged 1 commit into from
Jul 18, 2018

Conversation

brendandburns
Copy link
Contributor

No description provided.

@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Jul 14, 2018
@codecov-io
Copy link

Codecov Report

Merging #74 into master will decrease coverage by 0.37%.
The diff coverage is 72.72%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #74      +/-   ##
==========================================
- Coverage   93.51%   93.14%   -0.38%     
==========================================
  Files          11       11              
  Lines         972      977       +5     
==========================================
+ Hits          909      910       +1     
- Misses         63       67       +4
Impacted Files Coverage Δ
config/kube_config_test.py 93.51% <100%> (ø) ⬆️
config/kube_config.py 88.47% <64.7%> (-1.3%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 595ee0d...1be91e3. Read the comment docs.

@brendandburns
Copy link
Contributor Author

@mbohlool @roycaihw can I get a quick review on this?

Thanks!

@roycaihw
Copy link
Member

/lgtm
/approve

@k8s-ci-robot k8s-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Jul 18, 2018
@roycaihw roycaihw merged commit e8c0d98 into kubernetes-client:master Jul 18, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. lgtm Indicates that a PR is ready to be merged. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants