Create CI/CD test and build in the Github action (#523)

* Create test build in github action Signed-off-by: Jon Kartago Lamida <me@lamida.net> * Fix job invalid job name Signed-off-by: Jon Kartago Lamida <me@lamida.net> * Use listen port 8080 due to issue in github action Signed-off-by: Jon Kartago Lamida <me@lamida.net> * Run github action only on push to main Signed-off-by: Jon Kartago Lamida <me@lamida.net> * Update check-protos step Signed-off-by: Jon Kartago Lamida <me@lamida.net> * Use go-version matrix Signed-off-by: Jon Kartago Lamida <me@lamida.net> * Fix yaml Signed-off-by: Jon Kartago Lamida <me@lamida.net> * Revert accidentally unintended commits Signed-off-by: Jon Kartago Lamida <me@lamida.net> * Apply PR feedbacks Signed-off-by: Jon Kartago Lamida <me@lamida.net> * Capitalise workflow name Signed-off-by: Jon Kartago Lamida <me@lamida.net> * Can't use template in the job name Signed-off-by: Jon Kartago Lamida <me@lamida.net> * Revert to original job id Signed-off-by: Jon Kartago Lamida <me@lamida.net> * Use better job name description Signed-off-by: Jon Kartago Lamida <me@lamida.net> * Fix step name Signed-off-by: Jon Kartago Lamida <me@lamida.net> * Remove redundant step name Signed-off-by: Jon Kartago Lamida <me@lamida.net> * Simplify the step name again Signed-off-by: Jon Kartago Lamida <me@lamida.net> * Update server/server_test.go Co-authored-by: Charles Korn <charleskorn@users.noreply.github.com> * Use separate job for go test 1.20.x Signed-off-by: Jon Kartago Lamida <me@lamida.net> * Separate job for test and the rest Signed-off-by: Jon Kartago Lamida <me@lamida.net> * Update .github/workflows/test-build.yml Co-authored-by: Charles Korn <charleskorn@users.noreply.github.com> * Apply more PR feedbacks Signed-off-by: Jon Kartago Lamida <me@lamida.net> --------- Signed-off-by: Jon Kartago Lamida <me@lamida.net> Co-authored-by: Charles Korn <charleskorn@users.noreply.github.com>
grafana · May 15, 2024 · ab41af5 · ab41af5
1 parent a1bba12
commit ab41af5
Show file tree

Hide file tree

Showing 2 changed files with 56 additions and 0 deletions.
diff --git a/.github/workflows/test-build.yml b/.github/workflows/test-build.yml
@@ -0,0 +1,54 @@
+name: CI
+on:
+  push:
+    branches:
+      - main
+  pull_request:
+
+concurrency:
+  # Cancel any running workflow for the same branch when new commits are pushed.
+  # We group both by ref_name (available when CI is triggered by a push to a branch/tag)
+  # and head_ref (available when CI is triggered by a PR).
+  group: "${{ github.ref_name }}-${{ github.head_ref }}"
+  cancel-in-progress: true
+
+jobs:
+  check:
+    name: Check
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+      - name: Set up Go
+        uses: actions/setup-go@v5
+        with:
+          go-version: '1.21.x'
+      - name: Go mod check
+        run: make mod-check
+      - name: Lint
+        run: make lint
+      - name: Test benchmarks
+        run: make test-benchmarks
+      - name: Check protos
+        run: |
+          apt-get update && apt-get -y install unzip
+          go mod vendor
+          make check-protos
+
+  test:
+    name: Test on Go ${{ matrix.go-version }}
+    runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        # We want to make sure dskit can support multiple golang versions
+        # by ensuring the test would pass using all these supported versions.
+        go-version: ['1.20.x', '1.21.x']
+    steps:
+      - uses: actions/checkout@v4
+      - name: Set up Go ${{ matrix.go-version }}
+        uses: actions/setup-go@v5
+        with:
+          go-version: ${{ matrix.go-version }}
+      - name: Display Go version
+        run: go version
+      - name: Test
+        run: make test
diff --git a/server/server_test.go b/server/server_test.go
@@ -949,6 +949,8 @@ func TestGrpcOverProxyProtocol(t *testing.T) {
 	var cfg Config
 	cfg.RegisterFlags(flag.NewFlagSet("", flag.ExitOnError))
 	cfg.ProxyProtocolEnabled = true
+	// Set this to 0 to have it choose a random port
+	cfg.HTTPListenPort = 0
 
 	fakeSourceIP := "1.2.3.4"