-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
Update minimum Docker version #6718
Update minimum Docker version #6718
Conversation
@bparees PTAL |
We might improve this PR if openshift/openshift-docs#1450 merges. Instead of mentioning a specific version, we might just link to the docs. |
you want to wait for that to merge? |
(docs pull merged, might as well update this to use the anchor, though it won't work until the docs get rebuilt) |
aa84220
to
aa2502e
Compare
According to the official docs at: https://docs.openshift.org/latest/install_config/install/prerequisites.html
aa2502e
to
986b5d2
Compare
@bparees updated to include the link to the official docs. I think it is fine to be merged even before the docs are rebuilt, because it will still point at the right page (but will lead you to the top of the page). |
sounds good, thanks! [merge] |
continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/merge_pull_requests_origin/4668/) (Image: devenv-rhel7_3187) |
Evaluated for origin merge up to 986b5d2 |
[Test]ing while waiting on the merge queue |
Evaluated for origin test up to 986b5d2 |
continuous-integration/openshift-jenkins/test SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/152/) |
Merged by openshift-bot
According to the official docs at:
https://docs.openshift.org/latest/install_config/install/prerequisites.html