Skip to content

Commit

Permalink
Upgrade docker
Browse files Browse the repository at this point in the history
- As docker now uses by default BuildKit we can remove it.
- Install docker from docker.com instead via outdated apt distro repo

Signed-off-by: Marcus Brandenburger <[email protected]>
  • Loading branch information
mbrandenburger committed Sep 8, 2023
1 parent 687d4b3 commit 6260b68
Show file tree
Hide file tree
Showing 7 changed files with 15 additions and 14 deletions.
2 changes: 0 additions & 2 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,6 @@ jobs:

- name: build dev image
env:
DOCKER_BUILDKIT: 1
DOCKER_QUIET_BUILD: 1
run: |
DOCKER_BUILD_OPTS="--build-arg UBUNTU_VERSION=${{ matrix.os-version }} --build-arg UBUNTU_NAME=${{ matrix.os-name }}" \
Expand All @@ -31,7 +30,6 @@ jobs:
- name: run make inside dev container
env:
DOCKER_DEV_CI_MODE: 1
DOCKER_BUILDKIT: 1
DOCKER_QUIET_BUILD: 1
run: |
DOCKER_DEV_RUN_OPTS=`bash <(curl -s https://codecov.io/env)` \
Expand Down
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -191,7 +191,7 @@ official docker [documentation](https://docs.docker.com/install/linux/linux-post
for more details.

First make sure your host has
* Docker 18.09 (or higher).
* Docker v23.0 (or higher).
It also should use `/var/run/docker.sock` as socket to interact with the daemon (or you
will have to override in `$FPC_PATH/config.override.mk` the default definition in make of `DOCKER_DAEMON_SOCKET`)
* GNU make
Expand Down
2 changes: 1 addition & 1 deletion build.mk
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ include $(TOP)/config.mk
-include $(TOP)/config.override.mk

# define composites only here and not in config.mk so we can override parts in config.override.mk
DOCKER := DOCKER_BUILDKIT=$(DOCKER_BUILDKIT) $(DOCKER_CMD) $(DOCKERFLAGS)
DOCKER := $(DOCKER_CMD) $(DOCKERFLAGS)
ifeq (${SGX_MODE}, HW)
GOTAGS = -tags sgx_hw_mode
endif
Expand Down
4 changes: 0 additions & 4 deletions config.mk
Original file line number Diff line number Diff line change
Expand Up @@ -14,10 +14,6 @@ export GO_BUILD_OPT ?= -buildvcs=false

# Docker related settings
#--------------------------------------------------
export DOCKER_BUILDKIT ?= 1
# Building with build-kit makes multi-stage builds more efficient
# and also provides nicer output. If you experience issues with build-kit,
# you can disable it by overriding the default in your `config.override.mk`
DOCKERFLAGS :=
DOCKER_CMD := docker
# Note:
Expand Down
2 changes: 1 addition & 1 deletion samples/demos/irb/experimenter/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ protos:
$(MAKE) -C ../protos

build: protos
DOCKER_BUILDKIT=0 docker build -f Dockerfile -t irb-experimenter-worker ..
docker build -f Dockerfile -t irb-experimenter-worker ..

run: build
$(MAKE) -C worker stop-docker run-docker
Expand Down
3 changes: 1 addition & 2 deletions samples/deployment/azure/FPC_on_Azure.md
Original file line number Diff line number Diff line change
Expand Up @@ -77,15 +77,14 @@ echo 'epid-unlinkable' > ${FPC_PATH}/config/ias/spid_type.txt

There are two methods of setting up the FPC development environment.
The [docker based](../../../README.md#option-1-using-the-docker-based-fpc-development-environment) environment which is used here, and the [local development](../../../README.md#option-2-setting-up-your-system-to-do-local-development) environment.
Edit the `config.override.mk` to set HW mode and `DOCKER_BUILDKIT=1`.
Edit the `config.override.mk` to set HW mode.
```bash
vim $FPC_PATH/config.override.mk
```

paste in the following:
```bash
export SGX_MODE=HW
export DOCKER_BUILDKIT=1
```

Now we can start the container as follows:
Expand Down
14 changes: 11 additions & 3 deletions utils/docker/base-dev/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -54,13 +54,21 @@ RUN apt-get update -q \
python \
protobuf-compiler \
python-protobuf \
# docker commands (need as we use docker daemon from "outside")
docker.io \
docker-compose \
psmisc \
bc \
software-properties-common \
${APT_ADD_PKGS}

# docker commands (need as we use docker daemon from "outside")
RUN curl -fsSL https://download.docker.com/linux/ubuntu/gpg | apt-key add - \
&& add-apt-repository "deb [arch="$(dpkg --print-architecture)"] https://download.docker.com/linux/ubuntu "$(. /etc/os-release && echo "$VERSION_CODENAME")" stable" \
&& apt-get update -q \
&& apt-get install -y -q \
# docker-ce \
docker-ce-cli \
# containerd.io \
docker-buildx-plugin \
docker-compose-plugin

# Install go
ENV GOROOT=/usr/local/go
Expand Down

0 comments on commit 6260b68

Please sign in to comment.