-
Notifications
You must be signed in to change notification settings - Fork 48
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Introducing new go module for API, remove g/g dependency (#988)
* removed dependency on vgopath and k8s.io/apiserver * removed ginkgo unit test and added golang native unit test for secret reconciler * removed g/g dependency * added generated client, moved code-gen scripts to api module, updated k8s dependencies * fixed unit tests and created a new Makefile target for unit tests in api go module * fixed issues reported by gosec * fixed ci check script * added clean-generated-code script * Fixed IT tests and modified .ci/check, removed GOTESTFMT tool * adapted api package change * removed check-apidiff and check-generate scripts - moved to api/hack and fixed skaffold, no need to have gcloud auth login * removed config, moved samples, generating pki resources, copying crds for helm, corrected helm charts, created client as go module * fixed crd paths to refer to api/core/crds in IT tests * added go.work * adopted go version check for multi-module go project * removed -mod=mod as it does not work with go workspaces * changed skaffold, now using latest version * e2e test script now call prepare-local-deploy * added helm chart usage doc, added new make target, corrected paths of pki resources and fixed skaffold debug profile config * added git_tags for client and api go modules ensuring proper tagging upon release * added allow dependency-type: all in dependabot, updated docs with links * using directories to specify multiple manifests for dependabot * upgraded helm to v3.16.4 * renamed chart template files and added NOTES.txt * added icon url to chart.yaml * now using make --directory to run API make targets from CI check script * added pipeline definition to push helm charts as OCI artifact * configured publish helmcharts for jobs * renamed samples to examples, refactored, added client example * removed go mod files from examples/client * added etcd-druid client dependency to parent go.mod * regenerated code with license header without YEAR for generated code * added new lines to helm template yaml files ----------- Co-authored-by: Saketh Kalaga <[email protected]> Co-authored-by: Sonu Kumar Singh <[email protected]>
- Loading branch information
1 parent
0c24c05
commit a33217c
Showing
275 changed files
with
7,553 additions
and
4,697 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,47 @@ | ||
| # SPDX-FileCopyrightText: 2025 SAP SE or an SAP affiliate company and Gardener contributors | ||
| # | ||
| # SPDX-License-Identifier: Apache-2.0 | ||
|
|
||
| API_MODULE_ROOT := $(shell dirname "$(realpath $(lastword $(MAKEFILE_LIST)))") | ||
| REPO_ROOT := $(shell dirname "$(API_MODULE_ROOT)") | ||
| HACK_DIR := $(API_MODULE_ROOT)/hack | ||
| API_MODULE_TOOLS_DIR := $(HACK_DIR)/tools | ||
| REPO_TOOLS_DIR := $(REPO_ROOT)/hack/tools | ||
|
|
||
| include $(HACK_DIR)/tools.mk | ||
|
|
||
| .PHONY: check-generate | ||
| check-generate: | ||
| @$(HACK_DIR)/check-generate.sh "$(API_MODULE_ROOT)" | ||
|
|
||
| .PHONY: generate-api-docs | ||
| generate-api-docs: $(CRD_REF_DOCS) | ||
| @crd-ref-docs --source-path "$(REPO_ROOT)/api" --config "$(HACK_DIR)/api-reference/config.yaml" --output-path "$(REPO_ROOT)/docs/api-reference/etcd-druid-api.md" --renderer markdown | ||
|
|
||
| .PHONY: check-apidiff | ||
| check-apidiff: $(GO_APIDIFF) | ||
| @$(HACK_DIR)/check-apidiff.sh | ||
|
|
||
| .PHONY: generate | ||
| generate: $(CONTROLLER_GEN) tidy generate-api-docs | ||
| @$(HACK_DIR)/generate.sh | ||
|
|
||
| .PHONY: clean-generated-code | ||
| clean-generated-code: | ||
| @$(HACK_DIR)/clean-generated-code.sh | ||
|
|
||
| .PHONY: tidy | ||
| tidy: | ||
| @env GO111MODULE=on go mod tidy | ||
|
|
||
| .PHONY: test-unit | ||
| test-unit: | ||
| @go test ./... | ||
|
|
||
| .PHONY: format | ||
| format: $(GOIMPORTS_REVISER) | ||
| @$(REPO_ROOT)/hack/format.sh ./core ./validation | ||
|
|
||
| .PHONY: check | ||
| check: $(GOLANGCI_LINT) $(GOIMPORTS) format | ||
| @$(REPO_ROOT)/hack/check.sh --golangci-lint-config=../.golangci.yaml ./... |
Oops, something went wrong.