From 49200a1dd4aa752d4a8223e3056b816216a463f6 Mon Sep 17 00:00:00 2001 From: Davanum Srinivas Date: Mon, 13 Apr 2020 09:28:42 -0400 Subject: [PATCH] CRI plugin needs selinux tag as well A simple `grep selinux_stub bin/containerd` shows that we are not using selinux tag when building containerd. The CRI plugin pulls in a bunch of selinux related code, so we need to enable this tag as well. Signed-off-by: Davanum Srinivas --- Makefile | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/Makefile b/Makefile index 0c1d9a154..f2592adf5 100644 --- a/Makefile +++ b/Makefile @@ -93,8 +93,8 @@ MANPAGES=ctr.8 containerd.8 containerd-config.8 containerd-config.toml.5 ifdef BUILDTAGS GO_BUILDTAGS = ${BUILDTAGS} endif -# Build tags seccomp and apparmor are needed by CRI plugin. -GO_BUILDTAGS ?= seccomp apparmor +# Build tags seccomp, apparmor and selinux are needed by CRI plugin. +GO_BUILDTAGS ?= seccomp apparmor selinux GO_BUILDTAGS += ${DEBUG_TAGS} GO_TAGS=$(if $(GO_BUILDTAGS),-tags "$(GO_BUILDTAGS)",) GO_LDFLAGS=-ldflags '-X $(PKG)/version.Version=$(VERSION) -X $(PKG)/version.Revision=$(REVISION) -X $(PKG)/version.Package=$(PACKAGE) $(EXTRA_LDFLAGS)'