Skip to content
This repository has been archived by the owner on May 12, 2021. It is now read-only.

rootfs: Include libseccomp support in rootfs #154

Merged
merged 1 commit into from
Sep 3, 2018
Merged

rootfs: Include libseccomp support in rootfs #154

merged 1 commit into from
Sep 3, 2018

Conversation

nitkon
Copy link
Contributor

@nitkon nitkon commented Sep 3, 2018

Ensure the image used on the guest does include libseccomp,
otherwise when a seccomp profile is passed by the runtime, it
will end up with boot failure on the guest.

Fixes: #153

Signed-off-by: Nitesh Konkar niteshkonkar@in.ibm.com

Ensure the image used on the guest does include libseccomp,
otherwise when a seccomp profile is passed by the runtime, it
will end up with boot failure on the guest.

Fixes: #153

Signed-off-by: Nitesh Konkar niteshkonkar@in.ibm.com
@nitkon
Copy link
Contributor Author

nitkon commented Sep 3, 2018

I guess #154 takes care of this as well.

@jodh-intel
Copy link
Contributor

jodh-intel commented Sep 3, 2018

lgtm

The only question is whether we want to make this package configurable to keep the size as minimal as possible - see kata-containers/agent#353 (comment).

16.04 CI restarted (network error)

Approved with PullApprove

@marcov
Copy link
Contributor

marcov commented Sep 3, 2018

@nitkon: any reasons why libseccomp-devel is required too?

@marcov
Copy link
Contributor

marcov commented Sep 3, 2018

Never mind, just saw #152. Is it actually required for all distros?

@nitkon
Copy link
Contributor Author

nitkon commented Sep 3, 2018

@marcov : Yes, I faced the issue mentioned in #152 if not installed, in all the distros.

@marcov
Copy link
Contributor

marcov commented Sep 3, 2018

@nitkon ok, thanks

@jcvenegas
Copy link
Member

jcvenegas commented Sep 3, 2018

lgtm

Approved with PullApprove

@jcvenegas jcvenegas merged commit 67dba89 into kata-containers:master Sep 3, 2018
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.

4 participants