BUILD/MINOR: ci: add junit reports for some jobs

Author: oktalz

.aspell.yml

@@ -62,3 +62,4 @@ allowed:
   - async
   - params
   - configmap
+  - junit

.gitlab-ci.yml

@@ -15,6 +15,8 @@ variables:
   DOCKER_DRIVER: overlay2
   GO_VERSION: "1.24"
   DOCKER_VERSION: "28.1"
+  JUNIT_FILE: "junit-report.xml"
+  GOTESTSUM_JUNITFILE: "junit-report.xml"
 include:
   - local: .gitlab/bots.yml
   - local: .gitlab/diff.yml

.gitlab/lint.yml

@@ -46,9 +46,15 @@ commit-policy:
   rules:
     - if: $CI_PIPELINE_SOURCE == 'merge_request_event'
   image:
-    name: $CI_REGISTRY_GO/commit-check:5.2.0
+    name: $CI_REGISTRY_GO/commit-check:5.4.0
     entrypoint: [""]
   tags:
     - go
   script:
       - /check
+  artifacts:
+    when: on_failure
+    paths:
+      - junit-report.xml
+    reports:
+      junit: junit-report.xml

.gitlab/unit-tests.yml

@@ -10,5 +10,4 @@ unit-tests:
   tags:
     - go
   script:
-    - go build -v .
-    - go test -v ./...
+    - make test

Makefile

@@ -3,11 +3,11 @@ TARGETPLATFORM?=linux/amd64
 GOOS?=linux
 GOARCH?=amd64
 GOLANGCI_LINT_VERSION=1.64.5
-CHECK_COMMIT=5.2.0
+CHECK_COMMIT=5.4.0
 
 .PHONY: test
 test:
-	go test ./...
+	gotestsum --format-icons=hivis --format=testdox --format-hide-empty-pkg -- $(go list ./... | grep -v /deploy)
 
 .PHONY: e2e
 e2e:

deploy/tests/e2e/crd-tcp/cr_tcp_test.go

@@ -54,14 +54,29 @@ func (suite *TCPSuite) Test_CRD_TCP_OK() {
 			return true
 		}, e2e.WaitDuration, e2e.TickDuration)
 
-		// Get updated config and check it
-		cfg, err := suite.test.GetIngressControllerFile("/etc/haproxy/haproxy.cfg")
-		suite.Require().NoError(err, "Could not get Haproxy config")
-		reader := strings.NewReader(cfg)
-		p, err := parser.New(options.Reader(reader))
-		suite.Require().NoError(err, "Could not get Haproxy config parser")
+		var p parser.Parser
+		suite.Eventually(func() bool {
+			// Get updated config and check it
+			cfg, err := suite.test.GetIngressControllerFile("/etc/haproxy/haproxy.cfg")
+			if err != nil {
+				suite.T().Log(err)
+				return false
+			}
+			reader := strings.NewReader(cfg)
+			p, err = parser.New(options.Reader(reader))
+			if err != nil {
+				suite.T().Log(err)
+				return false
+			}
 
-		suite.checkFrontends(p)
+			err = suite.checkFrontends(p)
+			if err != nil {
+				suite.T().Log(err)
+				return false
+			}
+
+			return true
+		}, e2e.WaitDuration, e2e.TickDuration)
 
 		// Checks for backend
 		// Checks that we have correctly applied the backend CR