Merge pull request containerd#4697 from estesp/cp-4692-1.3

fuweid · web-flow · commit 81678f0e5a5e · 2020-11-07T16:17:53.000+08:00
[release/1.3] backport: Remove setuid gosu in favor of "sudo -E PATH=$PATH ..."
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -266,16 +266,6 @@ jobs:
         with:
           go-version: '1.13.15'
 
-      - name: Setup gosu
-        shell: bash
-        run: |
-          GOSU=/usr/local/bin/gosu
-          arch="$(dpkg --print-architecture | awk -F- '{ print $NF }')"
-          sudo wget -O ${GOSU} "https://github.com/tianon/gosu/releases/download/1.12/gosu-$arch"
-          sudo chmod +x ${GOSU}
-          sudo chown root ${GOSU}
-          sudo chmod +s ${GOSU}
-
       - name: Set env
         shell: bash
         run: |
@@ -289,10 +279,10 @@ jobs:
 
       - name: Install containerd dependencies
         run: |
-          sudo PATH=$PATH script/setup/install-seccomp
-          gosu root script/setup/install-runc
-          script/setup/install-cni
-          script/setup/install-critools
+          sudo -E PATH=$PATH script/setup/install-seccomp
+          sudo -E PATH=$PATH script/setup/install-runc
+          sudo -E PATH=$PATH script/setup/install-cni
+          sudo -E PATH=$PATH script/setup/install-critools
         working-directory: src/github.com/containerd/containerd
 
       - name: Install criu
