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

Enable rhel 7.5 and docker 1.13 #1061

Merged
merged 1 commit into from
Apr 12, 2018
Merged

Conversation

jupierce
Copy link
Contributor

@jupierce jupierce commented Feb 2, 2018

No description provided.

@openshift-ci-robot openshift-ci-robot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Feb 2, 2018
@@ -22,7 +22,7 @@ actions:
- type: "host_script"
title: "install docker"
script: |-
oct prepare docker --repourl "https://mirror.openshift.com/enterprise/rhel/dockertested/x86_64/os/" --repo 'oso-rhui-rhel-server-*'
oct prepare docker --version '1.13' --repo 'oso-rhui-rhel-server-*'
Copy link
Contributor

Choose a reason for hiding this comment

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

err, keep the dockertested for the 1.13 install

@jupierce
Copy link
Contributor Author

jupierce commented Feb 5, 2018

@stevekuznetsov If you insist on it working, fine.

@stevekuznetsov
Copy link
Contributor

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Feb 7, 2018
@0xmichalis
Copy link
Contributor

We failed to get the test suite passing with docker1.13 due to a websocket issue

openshift/origin#18726

@0xmichalis
Copy link
Contributor

We failed to get the test suite passing with docker1.13 due to a websocket issue

Seems to be fixed in openshift/origin#18755

@@ -4,7 +4,7 @@ set -eu
BASE_DIR=/mnt/redhat
REMOTE_DIR=/srv/enterprise/rhel
LATEST_RHEL_COMPOSE=${BASE_DIR}/nightly/latest-RHEL-7
LATEST_EXTRAS=${BASE_DIR}/nightly/EXTRAS-RHEL-7.4/latest-EXTRAS-7-RHEL-7/compose/Server/x86_64/os
LATEST_EXTRAS=${BASE_DIR}/nightly/EXTRAS-RHEL-7.5/latest-EXTRAS-7-RHEL-7/compose/Server/x86_64/os
Copy link
Contributor

Choose a reason for hiding this comment

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

seems scary if our AMI is still 7.4 yeah?

@openshift-ci-robot openshift-ci-robot removed the lgtm Indicates that a PR is ready to be merged. label Apr 10, 2018
@stevekuznetsov
Copy link
Contributor

/lgtm

@sdodson @jupierce can we yum upgrade from 7.4 to 7.5? If not we will need a new bare AMI

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Apr 10, 2018
@sdodson
Copy link
Contributor

sdodson commented Apr 11, 2018

Should be able to yum upgrade. If not then there are problems.

@stevekuznetsov
Copy link
Contributor

stevekuznetsov commented Apr 11, 2018

OK, will add a yum update to our AMI step

@jupierce jupierce merged commit cd41005 into openshift-eng:master Apr 12, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm Indicates that a PR is ready to be merged. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants