Skip to content
This repository has been archived by the owner on Mar 28, 2018. It is now read-only.

version: Update container image to 16020 #1002

Merged
merged 1 commit into from
Jun 22, 2017

Conversation

amshinde
Copy link
Contributor

This image contains hyperstart changes to not error out in case
there are failures in scanning scsi devices.

Signed-off-by: Archana Shinde archana.m.shinde@intel.com

@amshinde
Copy link
Contributor Author

Required for #997

@jcvenegas
Copy link
Contributor

jcvenegas commented Jun 21, 2017

lgtm

Approved with PullApprove

Copy link
Contributor

@gorozco1 gorozco1 left a comment

Choose a reason for hiding this comment

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

Please add also the linux-kernel version released with image 16020

versions.txt Outdated
@@ -4,7 +4,7 @@ json_glib_version=1.2.2
check_version=0.10.0
docker_engine_fedora_version=1.12.1
docker_engine_ubuntu_version=1.12.1-0~xenial
clear_vm_image_version=15900
clear_vm_image_version=16020
clear_vm_kernel_version=4.9.32-71
Copy link
Contributor

Choose a reason for hiding this comment

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

we need to sync out also the kernel there when we updated one or the other, so we need the linux-container kernel version that was released in clear image 16020

@chavafg
Copy link
Contributor

chavafg commented Jun 21, 2017

qa-passed

Approved with PullApprove

This image contains hyperstart changes to not error out in case
there are failures in scanning scsi devices when a block device
is passed for container rootfs.
Update to the kernel version released with 16020.

Signed-off-by: Archana Shinde <archana.m.shinde@intel.com>
@amshinde amshinde force-pushed the update-container-img-version branch from 64eef0e to 0ef2f8c Compare June 21, 2017 22:56
@jcvenegas
Copy link
Contributor

@gorozco1 can you approve this change now ?

@jcvenegas jcvenegas mentioned this pull request Jun 22, 2017
25 tasks
@jcvenegas
Copy link
Contributor

@amshinde can we remove the do-not-merge label ?

@chavafg
Copy link
Contributor

chavafg commented Jun 22, 2017

qa-passed

@gorozco1
Copy link
Contributor

lgtm

@jcvenegas jcvenegas merged commit 0117048 into intel:master Jun 22, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants