diff --git a/.github/workflows/go-test.yaml b/.github/workflows/go-test.yaml index e0c6975ee..da1adca2d 100644 --- a/.github/workflows/go-test.yaml +++ b/.github/workflows/go-test.yaml @@ -9,6 +9,22 @@ permissions: contents: read jobs: + uncommitted-code-check: + name: uncommitted-code-check + runs-on: ubuntu-latest + steps: + - name: Checkout the repo + uses: actions/checkout@v4 + + - name: Check generated deploy code + run: make generate-deploy + + - name: Check for uncommitted changes + run: make check-all-committed + + - name: Show the uncommitted "git diff" + if: ${{ failure() }} + run: git diff ; false e2e-build-test: name: e2e-build-test runs-on: ubuntu-latest diff --git a/.mergify.yml b/.mergify.yml index dd9fc38db..b5a623655 100644 --- a/.mergify.yml +++ b/.mergify.yml @@ -34,6 +34,7 @@ queue_rules: - "status-success=golangci-lint" - "status-success=mod-check" - "status-success=lint-extras" + - "status-success=uncommitted-code-check" - "status-success=ci/centos/k8s-e2e-external-storage/1.27" - "status-success=ci/centos/k8s-e2e-external-storage/1.28" - "status-success=ci/centos/k8s-e2e-external-storage/1.29" @@ -72,6 +73,7 @@ queue_rules: - "status-success=golangci-lint" - "status-success=mod-check" - "status-success=lint-extras" + - "status-success=uncommitted-code-check" - "status-success=ci/centos/k8s-e2e-external-storage/1.27" - "status-success=ci/centos/k8s-e2e-external-storage/1.28" - "status-success=ci/centos/k8s-e2e-external-storage/1.29" @@ -187,6 +189,7 @@ pull_request_rules: - "status-success=lint-extras" - "status-success=mod-check" - "status-success=multi-arch-build" + - "status-success=uncommitted-code-check" - "status-success=ci/centos/k8s-e2e-external-storage/1.27" - "status-success=ci/centos/k8s-e2e-external-storage/1.28" - "status-success=ci/centos/k8s-e2e-external-storage/1.29" @@ -274,6 +277,7 @@ pull_request_rules: - "status-success=mod-check" - "status-success=lint-extras" - "#changes-requested-reviews-by=0" + - "status-success=uncommitted-code-check" - "status-success=ci/centos/k8s-e2e-external-storage/1.27" - "status-success=ci/centos/k8s-e2e-external-storage/1.28" - "status-success=ci/centos/k8s-e2e-external-storage/1.29" diff --git a/Makefile b/Makefile index 7a4067fbf..3d984dcfa 100644 --- a/Makefile +++ b/Makefile @@ -184,6 +184,10 @@ generate-deploy: go mod vendor $(MAKE) -C deploy +.PHONY: check-all-committed +check-all-committed: ## Fail in case there are uncommitted changes + test -z "$(shell git status --short)" || (echo "files were modified: " ; git status --short ; false) + # # e2e testing by compiling e2e.test in case it does not exist and running the # executable. The e2e.test executable is not checked as a dependency in the