Skip to content

Commit

Permalink
Remove setuid gosu in favor of "sudo -E PATH=$PATH ..."
Browse files Browse the repository at this point in the history
Signed-off-by: Tianon Gravi <admwiggin@gmail.com>
(cherry picked from commit 17688a7)
  • Loading branch information
tianon authored and estesp committed Nov 6, 2020
1 parent ee26aa8 commit d1f19be
Showing 1 changed file with 4 additions and 14 deletions.
18 changes: 4 additions & 14 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -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: |
Expand All @@ -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
Expand Down

0 comments on commit d1f19be

Please sign in to comment.