Improve CI and have it use alpine instead

Personal preference and nice documentation on how to setup deps.

Author: nhooyr

.circleci/config.yml

@@ -2,7 +2,7 @@ version: 2
 jobs:
   fmt:
     docker:
-      - image: golang:1
+      - image: nhooyr/websocket-ci
     steps:
       - checkout
       - restore_cache:
@@ -19,7 +19,7 @@ jobs:
 
   lint:
     docker:
-      - image: golang:1
+      - image: nhooyr/websocket-ci
     steps:
       - checkout
       - restore_cache:
@@ -36,7 +36,7 @@ jobs:
 
   test:
     docker:
-      - image: golang:1
+      - image: nhooyr/websocket-ci
     steps:
       - checkout
       - restore_cache:
@@ -46,7 +46,26 @@ jobs:
             - go-
       - run: ./ci/test.sh
       - store_artifacts:
-          path: out
+          path: ci/out
+      - save_cache:
+          paths:
+            - /go
+            - /root/.cache/go-build
+          key: go-{{ checksum "go.sum" }}
+
+  bench:
+    docker:
+      - image: nhooyr/websocket-ci
+    steps:
+      - checkout
+      - restore_cache:
+          keys:
+            - go-{{ checksum "go.sum" }}
+            # Fallback to using the latest cache if no exact match is found.
+            - go-
+      - run: ./ci/bench.sh
+      - store_artifacts:
+          path: ci/out
       - save_cache:
           paths:
             - /go

ci/Dockerfile

@@ -0,0 +1,16 @@
+FROM golang:1-alpine
+
+RUN apk add --update bash git
+
+# Autobahn Test suite.
+RUN apk add py2-pip python2-dev gcc libffi-dev libc-dev openssl-dev && \
+  pip install autobahntestsuite
+
+# prettier.
+RUN apk add npm && npm install -g prettier
+
+# shellcheck
+RUN apk add xz curl && \
+  curl -o /tmp/shellcheck.tar.xz -L https://storage.googleapis.com/shellcheck/shellcheck-stable.linux.x86_64.tar.xz && \
+  tar -xf /tmp/shellcheck.tar.xz && \
+  ln -s "${PWD}/shellcheck-stable/shellcheck" /usr/local/bin/shellcheck

ci/bench.sh

@@ -4,7 +4,7 @@ set -euo pipefail
 cd "$(dirname "${0}")"
 source ./lib.sh
 
-go test --vet=off --run=^$ -bench=. -o=ci/out/websocket.test \
+go test -vet=off -run=^$ -bench=. -o=ci/out/websocket.test \
   -cpuprofile=ci/out/cpu.prof \
   -memprofile=ci/out/mem.prof \
   -blockprofile=ci/out/block.prof \

ci/fmt.sh

@@ -4,35 +4,45 @@ set -euo pipefail
 cd "$(dirname "${0}")"
 source ./lib.sh
 
-# Unfortunately, this is the only way to ensure go.mod and go.sum are correct.
-# See https://github.com/golang/go/issues/27005
-go list ./... > /dev/null
-go mod tidy
+gen() {
+  # Unfortunately, this is the only way to ensure go.mod and go.sum are correct.
+  # See https://github.com/golang/go/issues/27005
+  go list ./... > /dev/null
+  go mod tidy
 
-go generate ./...
+  go generate ./...
+}
 
-gofmt -w -s .
-go run go.coder.com/go-tools/cmd/goimports -w "-local=$(go list -m)" .
-go run mvdan.cc/sh/cmd/shfmt -i 2 -w -s -sr .
-# shellcheck disable=SC2046
-npx prettier \
-  --write \
-  --print-width 120 \
-  --no-semi \
-  --trailing-comma all \
-  --loglevel silent \
-  $(git ls-files "*.yaml" "*.yml" "*.md")
+fmt() {
+  gofmt -w -s .
+  go run go.coder.com/go-tools/cmd/goimports -w "-local=$(go list -m)" .
+  go run mvdan.cc/sh/cmd/shfmt -i 2 -w -s -sr .
+  # shellcheck disable=SC2046
+  npx prettier \
+    --write \
+    --print-width 120 \
+    --no-semi \
+    --trailing-comma all \
+    --loglevel silent \
+    $(git ls-files "*.yaml" "*.yml" "*.md")
+}
 
 unstaged_files() {
   git ls-files --other --modified --exclude-standard
 }
 
-if [[ ${CI:-} && $(unstaged_files) != "" ]]; then
-  echo
-  echo "Files either need generation or are formatted incorrectly."
-  echo "Please run:"
-  echo "./ci/fmt.sh"
-  echo
-  git status
-  exit 1
-fi
+check() {
+  if [[ ${CI:-} && $(unstaged_files) != "" ]]; then
+    echo
+    echo "Files either need generation or are formatted incorrectly."
+    echo "Please run:"
+    echo "./ci/fmt.sh"
+    echo
+    git status
+    exit 1
+  fi
+}
+
+gen
+fmt
+check

ci/lib.sh

@@ -2,9 +2,4 @@
 
 set -euo pipefail
 
-if [[ ${CI:-} ]]; then
-  export GOFLAGS=-mod=readonly
-  export DEBIAN_FRONTEND=noninteractive
-fi
-
 cd "$(git rev-parse --show-toplevel)"

ci/lint.sh

@@ -4,11 +4,6 @@ set -euo pipefail
 cd "$(dirname "${0}")"
 source ./lib.sh
 
-if [[ ${CI:-} ]]; then
-  apt-get update -qq
-  apt-get install -qq shellcheck > /dev/null
-fi
-
 # shellcheck disable=SC2046
 shellcheck -e SC1091 -x $(git ls-files "*.sh")
 go vet ./...

ci/test.sh

@@ -4,14 +4,6 @@ set -euo pipefail
 cd "$(dirname "${0}")"
 source ./lib.sh
 
-if [[ ${CI:-} ]]; then
-  apt-get update -qq
-  apt-get install -qq python-pip > /dev/null
-  # Need to add pip install directory to $PATH.
-  export PATH="/home/circleci/.local/bin:$PATH"
-  pip install -qqq autobahntestsuite
-fi
-
 # If you'd like to modify the args to go test, just run go test directly, this script is meant
 # for running tests at the end to get coverage and test under the race detector.
 go test -race -vet=off -coverprofile=ci/out/coverage.prof -coverpkg=./... ./...