BUILD/MINOR: ci: use better reports for ci jobs

Author: oktalz

.gitlab-ci.yml

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

.gitlab/bots.yml

@@ -1,5 +1,3 @@
-
-
 pipelines-check:
   stage: bots
   needs: []

.gitlab/diff.yml

@@ -15,12 +15,26 @@ diff:
     - task generate
     - "sed -i 's/internal: false/internal: true/g' taskfile/generate.yml"
   script:
+    - test -z "$(git diff 2> /dev/null)" || git diff
+    - test -z "$(git diff 2> /dev/null)" || junit-report add --status=failed --file=git-diff --message="git diff is not clean" --description="$(git diff)"
     - test -z "$(git diff 2> /dev/null)" || exit "Documentation is not generated, issue \`cd documentation/gen && go run .\` and commit the result"
+    - test -z "$(git ls-files --others --exclude-standard 2> /dev/null)" || git ls-files --others --exclude-standard
+    - test -z "$(git ls-files --others --exclude-standard 2> /dev/null)" || junit-report add --status=failed --file=git-ls-files --message="Documentation created untracked files" --description="$(git ls-files --others --exclude-standard 2> /dev/null)"
     - test -z "$(git ls-files --others --exclude-standard 2> /dev/null)" || exit "Documentation created untracked files, cannot proceed"
+    - junit-report add --status=ok --file=diff --message="Diff is clean" --description="Check for uncommitted changes in the repository"
+  artifacts:
+    when: always
+    paths:
+      - junit-report.xml
+    reports:
+      junit: junit-report.xml
 check-large-files:
   stage: diff
   rules:
     - if: $CI_PIPELINE_SOURCE == 'merge_request_event'
+  image:
+    name: $CI_REGISTRY_GO/golang:$GO_VERSION
+    entrypoint: [ "" ]
   tags:
     - go
   script:
@@ -38,7 +52,15 @@ check-large-files:
       if [ -s large_files.txt ]; then
         echo "❌ The following files exceed 1MB:"
         cat large_files.txt
+        junit-report add --status=failed --file=large-files --message="large files found" --description="$(cat large_files.txt)"
         exit 1
       else
         echo "✅ No large files found in the latest commit."
+        junit-report add --status=ok --file=large-files --message="commit clean" --description="No large files found in the latest commit."
       fi
+  artifacts:
+    when: always
+    paths:
+      - junit-report.xml
+    reports:
+      junit: junit-report.xml

.gitlab/lint.yml

@@ -12,6 +12,19 @@ tidy:
   script:
     - go mod tidy
     - test -z "$(git diff 2> /dev/null)" || exit 'Go modules not tidied, issue \`go mod tidy\` and commit the result'
+  after_script:
+    - |
+      if [ "$CI_JOB_STATUS" == "success" ]; then
+        junit-report add --status=ok --message="tidy check passed successfully"
+      else
+        junit-report add --status=failed --message="go mod tidy not executed" --description="$(git diff)"
+      fi
+  artifacts:
+    when: always
+    paths:
+      - junit-report.xml
+    reports:
+      junit: junit-report.xml
 format:
   stage: lint
   needs: []
@@ -28,6 +41,19 @@ format:
     - task format
     - "sed -i 's/internal: false/internal: true/g' taskfile.yml"
     - test -z "$(git diff 2> /dev/null)" || exit 'Go code not formatted, issue \`task ci\` and commit the result'
+  after_script:
+    - |
+      if [ "$CI_JOB_STATUS" == "success" ]; then
+        junit-report add --status=ok --message="format check passed successfully"
+      else
+        junit-report add --status=failed --message="go code not formatted" --description="$(git diff)"
+      fi
+  artifacts:
+    when: always
+    paths:
+      - junit-report.xml
+    reports:
+      junit: junit-report.xml
 lint:
   stage: lint
   needs: []
@@ -41,6 +67,19 @@ lint:
     - go
   script:
     - task lint
+  after_script:
+    - |
+      if [ "$CI_JOB_STATUS" == "success" ]; then
+        junit-report add --status=ok --message="linting passed successfully"
+      else
+        junit-report add --status=failed --message="git diff is not clean" --description="$(git diff)"
+      fi
+  artifacts:
+    when: always
+    paths:
+      - junit-report.xml
+    reports:
+      junit: junit-report.xml
 commit-policy:
   stage: lint
   needs: []

.gitlab/security.yml

@@ -13,6 +13,19 @@ license:
     - "sed -i 's/internal: true/internal: false/g' taskfile.yml"
   script:
     - task licenses
+  after_script:
+    - |
+      if [ "$CI_JOB_STATUS" == "success" ]; then
+        junit-report add --status=ok --message="license check passed successfully"
+      else
+        junit-report add --status=failed --message="licenses errors" --description="$(task licenses 2>&1)"
+      fi
+  artifacts:
+    when: always
+    paths:
+      - junit-report.xml
+    reports:
+      junit: junit-report.xml
 govulncheck:
   stage: security
   needs: []
@@ -28,3 +41,16 @@ govulncheck:
     - "sed -i 's/internal: true/internal: false/g' taskfile.yml"
   script:
     - task govulncheck
+  after_script:
+    - |
+      if [ "$CI_JOB_STATUS" == "success" ]; then
+        junit-report add --status=ok --message="vulnerability check passed successfully"
+      else
+        junit-report add --status=failed --message="vulnerabilities detected" --description="$(task govulncheck 2>&1)"
+      fi
+  artifacts:
+    when: always
+    paths:
+      - junit-report.xml
+    reports:
+      junit: junit-report.xml

.gitlab/unit-tests.yml

@@ -1,7 +1,5 @@
 unit-tests:
   needs: ["diff", "tidy"]
-  variables:
-    GOTESTSUM_JUNITFILE: "unit-tests-report.xml"
   rules:
     - if: $CI_PIPELINE_SOURCE == 'merge_request_event'
     - if: "$CI_PROJECT_NAMESPACE != 'haproxy-controller/community' && $CI_PIPELINE_SOURCE == 'push'"
@@ -18,6 +16,6 @@ unit-tests:
   artifacts:
     when: always
     paths:
-      - unit-tests-report.xml
+      - junit-report.xml
     reports:
-      junit: unit-tests-report.xml
+      junit: junit-report.xml