From 895961847159357045a9cff720ba1de75911c39a Mon Sep 17 00:00:00 2001 From: Dmitry Volodin Date: Mon, 6 Dec 2021 13:21:07 +0300 Subject: [PATCH] Make test fails due to keeping github.com autogenerated folder in root (#1713) --- .github/workflows/run_e2e.yaml | 2 +- Makefile | 3 +++ hack/update-codegen.sh | 16 ++++++++++++++-- hack/verify-codegen.sh | 5 ++--- .../acid.zalan.do/v1/zz_generated.deepcopy.go | 1 + 5 files changed, 21 insertions(+), 6 deletions(-) diff --git a/.github/workflows/run_e2e.yaml b/.github/workflows/run_e2e.yaml index a8e76f0ca..f7d984519 100644 --- a/.github/workflows/run_e2e.yaml +++ b/.github/workflows/run_e2e.yaml @@ -20,6 +20,6 @@ jobs: - name: Compile run: make linux - name: Run unit tests - run: go test ./... + run: make test - name: Run end-2-end tests run: make e2e diff --git a/Makefile b/Makefile index b83f7156b..07605c0a3 100644 --- a/Makefile +++ b/Makefile @@ -103,5 +103,8 @@ test: hack/verify-codegen.sh GO111MODULE=on go test ./... +codegen: + hack/update-codegen.sh + e2e: docker # build operator image to be tested cd e2e; make e2etest diff --git a/hack/update-codegen.sh b/hack/update-codegen.sh index 280da9385..d304004dc 100755 --- a/hack/update-codegen.sh +++ b/hack/update-codegen.sh @@ -4,10 +4,22 @@ set -o errexit set -o nounset set -o pipefail +GENERATED_PACKAGE_ROOT="github.com" +OPERATOR_PACKAGE_ROOT="${GENERATED_PACKAGE_ROOT}/zalando/postgres-operator" SCRIPT_ROOT=$(dirname ${BASH_SOURCE})/.. -CODEGEN_PKG=${CODEGEN_PKG:-$(cd ${SCRIPT_ROOT}; ls -d -1 ./vendor/k8s.io/code-generator 2>/dev/null || echo ${GOPATH}/src/k8s.io/code-generator)} +TARGET_CODE_DIR=${1-${SCRIPT_ROOT}/pkg} +CODEGEN_PKG=${CODEGEN_PKG:-$(cd "${SCRIPT_ROOT}"; ls -d -1 ./vendor/k8s.io/code-generator 2>/dev/null || echo "${GOPATH}"/src/k8s.io/code-generator)} + +cleanup() { + rm -rf "${GENERATED_PACKAGE_ROOT}" +} +trap "cleanup" EXIT SIGINT bash "${CODEGEN_PKG}/generate-groups.sh" all \ - github.com/zalando/postgres-operator/pkg/generated github.com/zalando/postgres-operator/pkg/apis \ + "${OPERATOR_PACKAGE_ROOT}/pkg/generated" "${OPERATOR_PACKAGE_ROOT}/pkg/apis" \ "acid.zalan.do:v1" \ --go-header-file "${SCRIPT_ROOT}"/hack/custom-boilerplate.go.txt + +cp -r "${OPERATOR_PACKAGE_ROOT}"/pkg/* "${TARGET_CODE_DIR}" + +cleanup diff --git a/hack/verify-codegen.sh b/hack/verify-codegen.sh index 68710015e..451ac76b7 100755 --- a/hack/verify-codegen.sh +++ b/hack/verify-codegen.sh @@ -19,15 +19,14 @@ cleanup mkdir -p "${TMP_DIFFROOT}" cp -a "${DIFFROOT}"/* "${TMP_DIFFROOT}" -"${SCRIPT_ROOT}/hack/update-codegen.sh" +"${SCRIPT_ROOT}/hack/update-codegen.sh" "${TMP_DIFFROOT}" echo "diffing ${DIFFROOT} against freshly generated codegen" ret=0 diff -Naupr "${DIFFROOT}" "${TMP_DIFFROOT}" || ret=$? -cp -a "${TMP_DIFFROOT}"/* "${DIFFROOT}" if [[ $ret -eq 0 ]] then echo "${DIFFROOT} up to date." else - echo "${DIFFROOT} is out of date. Please run hack/update-codegen.sh" + echo "${DIFFROOT} is out of date. Please run 'make codegen'" exit 1 fi diff --git a/pkg/apis/acid.zalan.do/v1/zz_generated.deepcopy.go b/pkg/apis/acid.zalan.do/v1/zz_generated.deepcopy.go index 7a8984ce4..61bbfde9c 100644 --- a/pkg/apis/acid.zalan.do/v1/zz_generated.deepcopy.go +++ b/pkg/apis/acid.zalan.do/v1/zz_generated.deepcopy.go @@ -1,3 +1,4 @@ +//go:build !ignore_autogenerated // +build !ignore_autogenerated /*