Skip to content

Conversation

@rhatdan
Copy link
Member

@rhatdan rhatdan commented Sep 29, 2017

We want to get support into skopeo for handling
override_kernel_checks so that we can use overlay
backend on RHEL.

Signed-off-by: Daniel J Walsh [email protected]

@rhatdan
Copy link
Member Author

rhatdan commented Sep 29, 2017

@runcom @mtrmac @mrunalp @nalind PTAL

@nalind
Copy link
Member

nalind commented Sep 29, 2017

Looks like a typical bump-a-dependency change, so LGTM.

@mtrmac
Copy link
Contributor

mtrmac commented Sep 30, 2017

(Do we want to switch from vendoring master (i.e. random updates over time, but keeping fairly fresh) to a specific git commit (i.e. a stable version when a human decides that it’s a good time to update, but no updates if no-one pays attention)? I don’t really have a strong opinion on this, just noting that this is not only a rebase.)

Not merging due to the test failure — updating containers/storage to support 1.7 is of course nice; looking at containers/image#352 , we might update the Go version to 1.8 instead.

@rhatdan
Copy link
Member Author

rhatdan commented Sep 30, 2017

I don't care about switching from master to a specific version. I know that @mrunalp prefers the git commit in cri-o, since he believes if he gets a bug on some vendored piece of code, it would be easier to investigate what was happening in that version.

I believe we should move to go 1.8. We need this version of Skopeo ASAP since OpenShift wants to use skopeo to move images between docker storage and container storage using overlay on RHEL.

Currently skopeo & containers storage does not support the override_kernel_check flag, to stop which prevents skopeo using overlay on a RHEL system, because container/storage tells skopeo that the version of the kernel on RHEL7.4 is too old.

@rhatdan
Copy link
Member Author

rhatdan commented Sep 30, 2017

How do I get the OSX Stuff to use a newer version of go?

We want to get support into skopeo for handling
override_kernel_checks so that we can use overlay
backend on RHEL.

Signed-off-by: Daniel J Walsh <[email protected]>
@rhatdan
Copy link
Member Author

rhatdan commented Sep 30, 2017

Switched back to master, we can talk about which is better later, and update to @nalind patch to allow containers/storage to build on golang 1.7

@rhatdan
Copy link
Member Author

rhatdan commented Oct 2, 2017

@mtrmac @runcom PTAL, Can we get this merged?

@runcom
Copy link
Member

runcom commented Oct 2, 2017

Looks ok, @mtrmac ?

@mtrmac mtrmac merged commit e317080 into containers:master Oct 2, 2017
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 5, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants