Skip to content

Comments

Fix router e2e validation for docker 1.9#7574

Merged
openshift-bot merged 1 commit intoopenshift:masterfrom
marun:fix-docker-1.9-router-e2e
Feb 24, 2016
Merged

Fix router e2e validation for docker 1.9#7574
openshift-bot merged 1 commit intoopenshift:masterfrom
marun:fix-docker-1.9-router-e2e

Conversation

@marun
Copy link
Contributor

@marun marun commented Feb 23, 2016

Router validation previously assumed that the router would be
reachable via the docker bridge's well known default address
(172.17.42.1). This default has been removed as of docker 1.9 and the
router ip can be more reliably determined via an 'oc get' call.

Router validation previously assumed that the router would be
reachable via the docker bridge's well known default address
(172.17.42.1).  This default has been removed as of docker 1.9 and the
router ip can be more reliably determined via an 'oc get' call.
@marun
Copy link
Contributor Author

marun commented Feb 23, 2016

@marun
Copy link
Contributor Author

marun commented Feb 23, 2016

[test]

@marun
Copy link
Contributor Author

marun commented Feb 23, 2016

For reference: moby/moby#17305

@openshift-bot
Copy link
Contributor

Evaluated for origin test up to 98d2747

@openshift-bot
Copy link
Contributor

continuous-integration/openshift-jenkins/test SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/1547/)

@deads2k
Copy link
Contributor

deads2k commented Feb 24, 2016

lgtm [merge]

@openshift-bot
Copy link
Contributor

continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/1547/) (Image: devenv-rhel7_3527)

@openshift-bot
Copy link
Contributor

Evaluated for origin merge up to 98d2747

openshift-bot pushed a commit that referenced this pull request Feb 24, 2016
@openshift-bot openshift-bot merged commit 1ac0b46 into openshift:master Feb 24, 2016
@marun marun deleted the fix-docker-1.9-router-e2e branch February 24, 2016 16:38
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.

3 participants