From 6b88730cc405a0a24bc23f0243a1c7e80d958f9d Mon Sep 17 00:00:00 2001 From: Simone Basso Date: Thu, 6 May 2021 20:44:56 +0200 Subject: [PATCH] refactor(.github/workflows): linux: use ./make, run less frequently (#334) While there, flush `print`s in `./make` to have more understandable logging. Also part of https://github.com/ooni/probe/issues/1466 --- .github/workflows/linux.yml | 38 ++++++++++++++++++++++--------------- make | 5 ++--- 2 files changed, 25 insertions(+), 18 deletions(-) diff --git a/.github/workflows/linux.yml b/.github/workflows/linux.yml index 8d09ae5..3148d04 100644 --- a/.github/workflows/linux.yml +++ b/.github/workflows/linux.yml @@ -1,24 +1,32 @@ -# Build and test Linux binary +# linux builds ooniprobe for Linux name: linux on: push: - schedule: - - cron: "14 17 * * 3" + branches: + - "release/**" jobs: - build: - runs-on: "ubuntu-latest" + + build_amd64: + runs-on: "ubuntu-20.04" steps: - - name: Docker - run: | + - run: | echo $'{\n "experimental": true\n}' | sudo tee /etc/docker/daemon.json sudo service docker restart - - uses: actions/setup-go@v1 - with: - go-version: "1.16" - uses: actions/checkout@v2 - - run: DOCKER_CLI_EXPERIMENTAL=enabled ./build.sh linux + - run: ./make --disable-embedding-psiphon-config -t ./CLI/linux/amd64/ooniprobe + env: + DOCKER_CLI_EXPERIMENTAL: enabled - run: ./smoketest.sh ./CLI/linux/amd64/ooniprobe - - uses: actions/upload-artifact@v1 - with: - name: ooniprobe-linux-amd64 - path: ./CLI/linux/amd64/ooniprobe + + build_arm64: + runs-on: "ubuntu-20.04" + steps: + - run: | + echo $'{\n "experimental": true\n}' | sudo tee /etc/docker/daemon.json + sudo service docker restart + - uses: actions/checkout@v2 + - run: sudo apt-get install qemu-user-static + - run: ./make --disable-embedding-psiphon-config -t ./CLI/linux/arm64/ooniprobe + env: + DOCKER_CLI_EXPERIMENTAL: enabled + - run: ./smoketest.sh ./CLI/linux/arm64/ooniprobe diff --git a/make b/make index 0f417bb..1937ca1 100755 --- a/make +++ b/make @@ -121,8 +121,8 @@ def sdkmanager_install_cmd(binpath: str) -> List[str]: def log(msg: str) -> None: - """log prints a message on the standard error.""" - print(msg, file=sys.stderr) + """log prints a message on the standard output.""" + print(msg, flush=True) class Options(Protocol): @@ -1151,7 +1151,6 @@ class OONIProbeLinux: "linux/{}".format(self._arch), "-e", "GOARCH={}".format(self._arch), - "-it", "-v", "{}:/ooni".format(os.getcwd()), "-w",