From da06ccfd3435204646d21b5706234739b0784333 Mon Sep 17 00:00:00 2001 From: sujeet01 Date: Thu, 24 Oct 2024 12:31:39 +0530 Subject: [PATCH] Address review comments --- .github/workflows/check-codegen.yml | 5 ----- .github/workflows/test.yml | 5 ----- Makefile | 1 - hack/update-codegen.sh | 2 +- .../core/certificate/ironcore/{onmetal.go => ironcore.go} | 0 .../core/quota/ironcore/{onmetal.go => ironcore.go} | 0 .../quota/evaluator/ironcore/{onmetal.go => ironcore.go} | 0 7 files changed, 1 insertion(+), 12 deletions(-) rename internal/controllers/core/certificate/ironcore/{onmetal.go => ironcore.go} (100%) rename internal/controllers/core/quota/ironcore/{onmetal.go => ironcore.go} (100%) rename internal/quota/evaluator/ironcore/{onmetal.go => ironcore.go} (100%) diff --git a/.github/workflows/check-codegen.yml b/.github/workflows/check-codegen.yml index 61f857ea6..feeeeab19 100644 --- a/.github/workflows/check-codegen.yml +++ b/.github/workflows/check-codegen.yml @@ -14,11 +14,6 @@ jobs: - uses: actions/setup-go@v5 with: go-version-file: 'go.mod' - - name: Download k8s.io/code-generator - run: | - CODE_GEN_VERSION=$(go list -m -f '{{.Version}}' k8s.io/code-generator) - go get k8s.io/code-generator@$CODE_GEN_VERSION - echo "CODE_GEN_DIR=$(go list -m -f '{{.Dir}}' k8s.io/code-generator)" >> $GITHUB_ENV - name: Run make generate run: make generate - name: Compare the expected and actual generated/* directories diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml index 1112c1917..c5b637f2f 100644 --- a/.github/workflows/test.yml +++ b/.github/workflows/test.yml @@ -16,10 +16,5 @@ jobs: - uses: actions/setup-go@v5 with: go-version-file: 'go.mod' - - name: Download k8s.io/code-generator - run: | - CODE_GEN_VERSION=$(go list -m -f '{{.Version}}' k8s.io/code-generator) - go get k8s.io/code-generator@$CODE_GEN_VERSION - echo "CODE_GEN_DIR=$(go list -m -f '{{.Dir}}' k8s.io/code-generator)" >> $GITHUB_ENV - name: Run make test run: make test diff --git a/Makefile b/Makefile index f3ffddab9..8949544cd 100644 --- a/Makefile +++ b/Makefile @@ -359,7 +359,6 @@ GOLANGCI_LINT ?= $(LOCALBIN)/golangci-lint ## Tool Versions KUSTOMIZE_VERSION ?= v5.1.1 -CODE_GENERATOR_VERSION ?= v0.29.7 VGOPATH_VERSION ?= v0.1.3 CONTROLLER_TOOLS_VERSION ?= v0.15.0 GEN_CRD_API_REFERENCE_DOCS_VERSION ?= v0.3.0 diff --git a/hack/update-codegen.sh b/hack/update-codegen.sh index 0c29e7d0a..2e51ab62f 100755 --- a/hack/update-codegen.sh +++ b/hack/update-codegen.sh @@ -8,7 +8,7 @@ THIS_PKG="github.com/ironcore-dev/ironcore" SCRIPT_DIR="$( cd -- "$( dirname -- "${BASH_SOURCE[0]}" )" &> /dev/null && pwd )" PROJECT_ROOT="$SCRIPT_DIR/.." -CODE_GEN_DIR="${CODE_GEN_DIR:-$(go list -m -f '{{.Dir}}' k8s.io/code-generator)}" +CODE_GEN_DIR=$(go list -m -f '{{.Dir}}' k8s.io/code-generator) source "${CODE_GEN_DIR}/kube_codegen.sh" export TERM="xterm-256color" diff --git a/internal/controllers/core/certificate/ironcore/onmetal.go b/internal/controllers/core/certificate/ironcore/ironcore.go similarity index 100% rename from internal/controllers/core/certificate/ironcore/onmetal.go rename to internal/controllers/core/certificate/ironcore/ironcore.go diff --git a/internal/controllers/core/quota/ironcore/onmetal.go b/internal/controllers/core/quota/ironcore/ironcore.go similarity index 100% rename from internal/controllers/core/quota/ironcore/onmetal.go rename to internal/controllers/core/quota/ironcore/ironcore.go diff --git a/internal/quota/evaluator/ironcore/onmetal.go b/internal/quota/evaluator/ironcore/ironcore.go similarity index 100% rename from internal/quota/evaluator/ironcore/onmetal.go rename to internal/quota/evaluator/ironcore/ironcore.go