Skip to content
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

Updates for K8 client package #189

Closed
wants to merge 5 commits into from
Closed

Updates for K8 client package #189

wants to merge 5 commits into from

Conversation

jrnt30
Copy link
Collaborator

@jrnt30 jrnt30 commented Feb 5, 2019

  • Adds in error messages for pod interface failures
  • Updates K8 client-go to 1.10 version
  • Adjusts travis CI docker push to not tag with latest for RC builds

@coveralls
Copy link

coveralls commented Feb 5, 2019

Coverage Status

Coverage increased (+0.2%) to 19.946% when pulling 1f474ad on 1.10-api into d1ede0d on master.

rifelpet and others added 3 commits February 8, 2019 10:59
- This reverts the change made in #173 temporarily until we can pin down why this change is also
negatively impacting "healthy" pods from being properly indexed in newer clusters
@geekofalltrades
Copy link

geekofalltrades commented Sep 17, 2019

As a bonus, this MR pulls in a version of client-go that has switched from glog to klog, so that the process doesn't just randomly die sometimes if it can't write a file in /tmp. We hit this a lot in our environment, where we're running kube2iam from a scratch container; our kube2iam pods have all died and restarted hundreds of times.

@wamarcantel
Copy link

@jtblin Can we get this merged any time soon?

@angulito
Copy link

any news about merging this PR? it will fix the read-only issue with /tmp - thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants