Skip to content

cluster up: warn on error parsing Docker version#13201

Merged
openshift-bot merged 1 commit intoopenshift:masterfrom
csrwng:docker_version_warning
Mar 3, 2017
Merged

cluster up: warn on error parsing Docker version#13201
openshift-bot merged 1 commit intoopenshift:masterfrom
csrwng:docker_version_warning

Conversation

@csrwng
Copy link
Contributor

@csrwng csrwng commented Mar 2, 2017

The latest version of Docker for Mac/Windows uses the Community Edition
versioning scheme. This causes 'cluster up' to halt with an error because
the new version cannot be parsed by the 'semver' library. This commit
changes the behavior to display a warning instead of exiting with an
error.

Fixes #13202

The latest version of Docker for Mac/Windows uses the Community Edition
versioning scheme. This causes 'cluster up' to halt with an error because
the new version cannot be parsed by the 'semver' library. This commit
changes the behavior to display a warning instead of exiting with an
error.
@csrwng
Copy link
Contributor Author

csrwng commented Mar 2, 2017

@openshift/devex

@bparees
Copy link
Contributor

bparees commented Mar 2, 2017

lgtm

@csrwng
Copy link
Contributor Author

csrwng commented Mar 2, 2017

[merge]

@openshift-bot
Copy link
Contributor

openshift-bot commented Mar 2, 2017

continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pull_requests_origin_future/755/) (Image: devenv-rhel7_6027)

@openshift-bot
Copy link
Contributor

Evaluated for origin merge up to b6fbeef

@openshift-bot
Copy link
Contributor

[Test]ing while waiting on the merge queue

@openshift-bot
Copy link
Contributor

Evaluated for origin test up to b6fbeef

@openshift-bot
Copy link
Contributor

continuous-integration/openshift-jenkins/test SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pull_requests_origin_future/755/) (Base Commit: f4ef957)

@eschabell
Copy link

Fix verified...

@errordeveloper
Copy link

This worked for me, I'm using Docker for Mac. Has it made into a release yet? Are we gonna have to wait for 1.6.0 or it will make it into next 1.5.x?

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.

5 participants