From 0343e28d3a97f57c9e297aa7980f169eb4a3cd88 Mon Sep 17 00:00:00 2001
From: teor <teor@riseup.net>
Date: Mon, 19 Oct 2020 14:03:24 +1000
Subject: [PATCH] Disable sync test on ubuntu CI runners

They don't seem to have DNS or network configured during the tests.

Also make capitalisation of step names consistent.
---
 .github/workflows/ci.yml       | 8 ++++++--
 .github/workflows/coverage.yml | 3 +++
 2 files changed, 9 insertions(+), 2 deletions(-)

diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
index 7e45a56f..3160c0a2 100644
--- a/.github/workflows/ci.yml
+++ b/.github/workflows/ci.yml
@@ -22,9 +22,13 @@ jobs:
         uses: actions-rs/cargo@v1
         with:
           command: fetch
-      - name: install LLVM on Windows
+      - name: Install LLVM on Windows
         if: matrix.os == 'windows-latest'
         run: choco install llvm -y
+      - name: Skip network tests on Ubuntu
+        # Ubuntu runners don't have network or DNS configured during test steps
+        if: matrix.os == 'ubuntu-latest'
+        run: echo "ZEBRA_SKIP_NETWORK_TESTS=1" >> $GITHUB_ENV
       - name: Run tests
         env:
           RUST_BACKTRACE: full
@@ -67,7 +71,7 @@ jobs:
         with:
           toolchain: stable
           override: true
-      - name: install LLVM on Windows
+      - name: Install LLVM on Windows
         if: matrix.os == 'windows-latest'
         run: choco install llvm -y
       - name: cargo fetch
diff --git a/.github/workflows/coverage.yml b/.github/workflows/coverage.yml
index bd10fcb8..f31066b7 100644
--- a/.github/workflows/coverage.yml
+++ b/.github/workflows/coverage.yml
@@ -20,6 +20,9 @@ jobs:
         with:
           toolchain: stable
           override: true
+      - name: Skip network tests on Ubuntu
+        # Ubuntu runners don't have network or DNS configured during test steps
+        run: echo "ZEBRA_SKIP_NETWORK_TESTS=1" >> $GITHUB_ENV
       - name: Run cargo-tarpaulin
         uses: actions-rs/tarpaulin@v0.1
         with: