Hooks improvement to use ent eager loading (#261)

* additional hooks unit tests Signed-off-by: Matt Siwiec <rizzza@users.noreply.github.com> * audit loadbalancer manual hook db hits and ensure providerID included in additionalSubjects Signed-off-by: Matt Siwiec <rizzza@users.noreply.github.com> * audit origin manual hook db hits Signed-off-by: Matt Siwiec <rizzza@users.noreply.github.com> * audit pool manual hook db hits Signed-off-by: Matt Siwiec <rizzza@users.noreply.github.com> * audit port manual hook db hits Signed-off-by: Matt Siwiec <rizzza@users.noreply.github.com> * bump test event msg channel timeout to 5s; align Signed-off-by: Matt Siwiec <rizzza@users.noreply.github.com> * test with one retry Signed-off-by: Matt Siwiec <rizzza@users.noreply.github.com> * tweek flake Signed-off-by: Matt Siwiec <rizzza@users.noreply.github.com> --------- Signed-off-by: Matt Siwiec <rizzza@users.noreply.github.com>
infratographer · Nov 3, 2023 · b8cdab1 · b8cdab1
1 parent f43a386
commit b8cdab1
Show file tree

Hide file tree

Showing 4 changed files with 207 additions and 181 deletions.
diff --git a/.github/workflows/test-go.yml b/.github/workflows/test-go.yml
@@ -40,5 +40,8 @@ jobs:
       - name: Install atlas for db migrations on ${{ matrix.ci-database }}
         run: go install ariga.io/atlas/cmd/atlas@latest
 
+      # with one retry
       - name: Run go tests for ${{ matrix.ci-database }}
-        run: LOADBALANCERAPI_TESTDB_URI="${{ matrix.env-database-uri }}" go test -race -coverprofile=coverage.txt -covermode=atomic -tags testtools ./...
+        run: |
+          LOADBALANCERAPI_TESTDB_URI="${{ matrix.env-database-uri }}" go test -race -coverprofile=coverage.txt -covermode=atomic -tags testtools ./... || \
+          LOADBALANCERAPI_TESTDB_URI="${{ matrix.env-database-uri }}" go test -race -coverprofile=coverage.txt -covermode=atomic -tags testtools ./...