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

kubelet unable to parse docker version "17.04.0-ce" #44140

Closed
opsnull opened this issue Apr 6, 2017 · 3 comments · Fixed by #44199
Closed

kubelet unable to parse docker version "17.04.0-ce" #44140

opsnull opened this issue Apr 6, 2017 · 3 comments · Fixed by #44199

Comments

@opsnull
Copy link
Contributor

opsnull commented Apr 6, 2017

Is this a request for help? (If yes, you should use our troubleshooting guide and community support channels, see http://kubernetes.io/docs/troubleshooting/.):

no

What keywords did you search in Kubernetes issues before filing this one? (If you have found any duplicates, you should instead reply there.):

Unable to parse docker version


Is this a BUG REPORT or FEATURE REQUEST? (choose one):

Kubernetes version (use kubectl version):

Client Version: version.Info{Major:"1", Minor:"6", GitVersion:"v1.6.1", GitCommit:"b0b7a323cc5a4a2019b2e9520c21c7830b7f708e", GitTreeState:"clean", BuildDate:"2017-04-03T20:44:38Z", GoVersion:"go1.7.5", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"6", GitVersion:"v1.6.1", GitCommit:"b0b7a323cc5a4a2019b2e9520c21c7830b7f708e", GitTreeState:"clean", BuildDate:"2017-04-03T20:33:27Z", GoVersion:"go1.7.5", Compiler:"gc", Platform:"linux/amd64"}

Environment:

  • Cloud provider or hardware configuration:
  • OS (e.g. from /etc/os-release):
  • Kernel (e.g. uname -a):
  • Install tools:
  • Others:

docker version

Client:
Version: 17.04.0-ce
API version: 1.28
Go version: go1.7.5
Git commit: 4845c56
Built: Wed Apr 5 23:33:17 2017
OS/Arch: linux/amd64

Server:
Version: 17.04.0-ce
API version: 1.28 (minimum version 1.12)
Go version: go1.7.5
Git commit: 4845c56
Built: Wed Apr 5 23:33:17 2017
OS/Arch: linux/amd64
Experimental: false

What happened:

I found some error messages in kubelet log liking:
kubelet[51799]: E0406 16:24:23.966622 51799 container_manager_linux.go:90] Unable to parse docker version "17.04.0-ce": illegal zero-prefixed version component "04" in "17.04.0-ce"

What you expected to happen:

kublet can parse docker version "17.04.0-ce"

How to reproduce it (as minimally and precisely as possible):

Install kubelet with docker version 17.04.0-ce

Anything else we need to know:

@opsnull opsnull changed the title Unable to parse docker version "17.04.0-ce" kubelet unable to parse docker version "17.04.0-ce" Apr 6, 2017
@supereagle
Copy link
Contributor

supereagle commented Apr 7, 2017

Docker has change its release strategy and versioning scheme from v17.03.0-ce-rc1. We need to update the version verify condition to satisfy the new docker versions.

@traceypooh
Copy link

+1

@Random-Liu
Copy link
Member

Random-Liu commented Apr 8, 2017

Actually, Kubernetes only supports docker version 1.10.x, 1.11.x and 1.12.x.

We don't recommend user to use version higher or lower than that range because we don't have enough test coverage on that.

As for docker new version schema, I'm fine with changing the code to tolerant new version schema, but one more remind there is no guarantee with newer docker version. :)

k8s-github-robot pushed a commit that referenced this issue Apr 8, 2017
Automatic merge from submit-queue

update docker version parser for its new versioning scheme

**What this PR does / why we need it**:
Docker has change its release strategy and versioning scheme from [v17.03.0-ce-rc1](https://github.com/docker/docker/releases/tag/v17.03.0-ce-rc1). We need to update the version verify condition to satisfy the new docker versions.

**Which issue this PR fixes** : fixes #44140

**Special notes for your reviewer**:

**Release note**:

```
NONE
```
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 a pull request may close this issue.

4 participants