Skip to content
This repository has been archived by the owner on Sep 24, 2021. It is now read-only.

Fix CNI problems #188

Merged
merged 1 commit into from
Jul 10, 2017
Merged

Fix CNI problems #188

merged 1 commit into from
Jul 10, 2017

Conversation

feiskyer
Copy link
Contributor

  • Setup pod name correctly
  • Fail out if CNI plugin fails
  • Warn if getting checkpoint error

@feiskyer
Fix CNI problems
a2394f1 
- Setup pod name correctly
- Fail out if CNI plugin fails
- Warn if getting checkpoint error
@feiskyer feiskyer requested a review from resouer July 10, 2017 08:27
@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. labels Jul 10, 2017
@k8s-reviewable
Copy link

This change is Reviewable

resouer
resouer approved these changes Jul 10, 2017
View reviewed changes
Copy link
Contributor

@resouer resouer left a comment

Choose a reason for hiding this comment

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

lgtm

@resouer resouer merged commit 2dac43b into kubernetes-retired:master Jul 10, 2017
@feiskyer feiskyer deleted the cni-fix branch July 10, 2017 09:22
@feiskyer feiskyer mentioned this pull request Sep 27, 2017
Closed
3 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@resouer resouer resouer approved these changes

Assignees
No one assigned
Labels
cncf-cla: yes Indicates the PR's author has signed the CNCF CLA.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@feiskyer @k8s-reviewable @resouer @k8s-ci-robot