Skip to content

Commit 21a3f21

Browse files
committed
Address PR comments
Signed-off-by: Ernest Wong <[email protected]>
1 parent 52f6c94 commit 21a3f21

File tree

2 files changed

+5
-3
lines changed

2 files changed

+5
-3
lines changed

Makefile

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -132,7 +132,7 @@ vet: ## Run go vet against code.
132132
go vet ./...
133133

134134
.PHONY: test
135-
test: manifests generate fmt vet envtest image-build ## Run tests.
135+
test: manifests generate fmt vet envtest image-build verify-crds ## Run tests.
136136
CGO_ENABLED=1 KUBEBUILDER_ASSETS="$(shell $(ENVTEST) use $(ENVTEST_K8S_VERSION) --bin-dir $(LOCALBIN) -p path)" go test $$(go list ./... | grep -v /e2e | grep -v /conformance) -race -coverprofile cover.out
137137

138138
.PHONY: test-unit
@@ -163,6 +163,10 @@ ci-lint: golangci-lint
163163
verify: vet fmt-verify manifests generate ci-lint verify-all
164164
git --no-pager diff --exit-code config api client-go
165165

166+
.PHONY: verify-crds
167+
verify-crds: kubectl-validate
168+
hack/verify-manifests.sh
169+
166170
# Run static analysis.
167171
.PHONY: verify-all
168172
verify-all:

hack/verify-manifests.sh

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -44,8 +44,6 @@ main() {
4444
fetch_crds "https://raw.githubusercontent.com/GoogleCloudPlatform/gke-gateway-api/refs/tags/${GKE_GATEWAY_API_VERSION}/config/crd/networking.gke.io_healthcheckpolicies.yaml"
4545
fetch_crds "https://raw.githubusercontent.com/istio/istio/refs/tags/${ISTIO_VERSION}/manifests/charts/base/files/crd-all.gen.yaml"
4646

47-
make kubectl-validate
48-
4947
${SCRIPT_ROOT}/bin/kubectl-validate "${TEMP_DIR}"
5048
${SCRIPT_ROOT}/bin/kubectl-validate "${SCRIPT_ROOT}/config/manifests" --local-crds "${TEMP_DIR}"
5149
}

0 commit comments

Comments
 (0)