Skip to content

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