Rename new::setup_env back to setup_env
This commit is contained in:
		| @@ -548,7 +548,7 @@ EOF | |||||||
|   fi |   fi | ||||||
| } | } | ||||||
|  |  | ||||||
| # kube::golang::new::setup_env will check that the `go` commands is available in | # kube::golang::setup_env will check that the `go` commands is available in | ||||||
| # ${PATH}. It will also check that the Go version is good enough for the | # ${PATH}. It will also check that the Go version is good enough for the | ||||||
| # Kubernetes build. | # Kubernetes build. | ||||||
| # | # | ||||||
| @@ -556,8 +556,7 @@ EOF | |||||||
| #   env-var GOPATH points to our local output dir | #   env-var GOPATH points to our local output dir | ||||||
| #   env-var GOBIN is unset (we want binaries in a predictable place) | #   env-var GOBIN is unset (we want binaries in a predictable place) | ||||||
| #   env-var PATH includes the local GOPATH | #   env-var PATH includes the local GOPATH | ||||||
| #FIXME: rename this when all callers are converted | kube::golang::setup_env() { | ||||||
| kube::golang::new::setup_env() { |  | ||||||
|   kube::golang::verify_go_version |   kube::golang::verify_go_version | ||||||
|  |  | ||||||
|   # Even in module mode, we need to set GOPATH for `go build` and `go install` |   # Even in module mode, we need to set GOPATH for `go build` and `go install` | ||||||
| @@ -876,7 +875,7 @@ kube::golang::build_binaries() { | |||||||
|   # Create a sub-shell so that we don't pollute the outer environment |   # Create a sub-shell so that we don't pollute the outer environment | ||||||
|   ( |   ( | ||||||
|     # Check for `go` binary and set ${GOPATH}. |     # Check for `go` binary and set ${GOPATH}. | ||||||
|     kube::golang::new::setup_env |     kube::golang::setup_env | ||||||
|     V=2 kube::log::info "Go version: $(GOFLAGS='' go version)" |     V=2 kube::log::info "Go version: $(GOFLAGS='' go version)" | ||||||
|  |  | ||||||
|     local host_platform |     local host_platform | ||||||
|   | |||||||
| @@ -31,7 +31,7 @@ PROTOC_VERSION=23.4 | |||||||
| # and formats it correctly | # and formats it correctly | ||||||
| # $1: Full path to the directory where the api.proto file is | # $1: Full path to the directory where the api.proto file is | ||||||
| function kube::protoc::generate_proto() { | function kube::protoc::generate_proto() { | ||||||
|   kube::golang::new::setup_env |   kube::golang::setup_env | ||||||
|   GOPROXY=off go install k8s.io/code-generator/cmd/go-to-protobuf/protoc-gen-gogo |   GOPROXY=off go install k8s.io/code-generator/cmd/go-to-protobuf/protoc-gen-gogo | ||||||
|  |  | ||||||
|   kube::protoc::check_protoc |   kube::protoc::check_protoc | ||||||
|   | |||||||
| @@ -35,7 +35,7 @@ kube::verify::generated() { | |||||||
|     kube::util::ensure_clean_working_dir |     kube::util::ensure_clean_working_dir | ||||||
|  |  | ||||||
|     # This sets up the environment, like GOCACHE, which keeps the worktree cleaner. |     # This sets up the environment, like GOCACHE, which keeps the worktree cleaner. | ||||||
|     kube::golang::new::setup_env |     kube::golang::setup_env | ||||||
|  |  | ||||||
|     _tmpdir="$(kube::realpath "$(mktemp -d -t "verify-generated-$(basename "$1").XXXXXX")")" |     _tmpdir="$(kube::realpath "$(mktemp -d -t "verify-generated-$(basename "$1").XXXXXX")")" | ||||||
|     git worktree add -f -q "${_tmpdir}" HEAD |     git worktree add -f -q "${_tmpdir}" HEAD | ||||||
|   | |||||||
| @@ -17,7 +17,7 @@ | |||||||
| KUBE_ROOT=$(dirname "${BASH_SOURCE[0]}")/../.. | KUBE_ROOT=$(dirname "${BASH_SOURCE[0]}")/../.. | ||||||
| source "${KUBE_ROOT}/hack/lib/init.sh" | source "${KUBE_ROOT}/hack/lib/init.sh" | ||||||
|  |  | ||||||
| kube::golang::new::setup_env | kube::golang::setup_env | ||||||
| kube::golang::setup_gomaxprocs | kube::golang::setup_gomaxprocs | ||||||
|  |  | ||||||
| # start the cache mutation detector by default so that cache mutators will be found | # start the cache mutation detector by default so that cache mutators will be found | ||||||
|   | |||||||
| @@ -21,7 +21,7 @@ set -o pipefail | |||||||
| KUBE_ROOT=$(dirname "${BASH_SOURCE[0]}")/../.. | KUBE_ROOT=$(dirname "${BASH_SOURCE[0]}")/../.. | ||||||
| source "${KUBE_ROOT}/hack/lib/init.sh" | source "${KUBE_ROOT}/hack/lib/init.sh" | ||||||
|  |  | ||||||
| kube::golang::new::setup_env | kube::golang::setup_env | ||||||
| kube::golang::setup_gomaxprocs | kube::golang::setup_gomaxprocs | ||||||
|  |  | ||||||
| # start the cache mutation detector by default so that cache mutators will be found | # start the cache mutation detector by default so that cache mutators will be found | ||||||
|   | |||||||
| @@ -27,7 +27,7 @@ source "${KUBE_ROOT}/hack/lib/init.sh" | |||||||
| source "${KUBE_ROOT}/hack/lib/protoc.sh" | source "${KUBE_ROOT}/hack/lib/protoc.sh" | ||||||
| cd "${KUBE_ROOT}" | cd "${KUBE_ROOT}" | ||||||
|  |  | ||||||
| kube::golang::new::setup_env | kube::golang::setup_env | ||||||
|  |  | ||||||
| DBG_CODEGEN="${DBG_CODEGEN:-0}" | DBG_CODEGEN="${DBG_CODEGEN:-0}" | ||||||
| GENERATED_FILE_PREFIX="${GENERATED_FILE_PREFIX:-zz_generated.}" | GENERATED_FILE_PREFIX="${GENERATED_FILE_PREFIX:-zz_generated.}" | ||||||
|   | |||||||
| @@ -21,7 +21,7 @@ set -o pipefail | |||||||
| KUBE_ROOT=$(dirname "${BASH_SOURCE[0]}")/.. | KUBE_ROOT=$(dirname "${BASH_SOURCE[0]}")/.. | ||||||
| source "${KUBE_ROOT}/hack/lib/init.sh" | source "${KUBE_ROOT}/hack/lib/init.sh" | ||||||
|  |  | ||||||
| kube::golang::new::setup_env | kube::golang::setup_env | ||||||
|  |  | ||||||
| # Nuke old files so we don't accidentally carry stuff forward. | # Nuke old files so we don't accidentally carry stuff forward. | ||||||
| rm -f staging/src/k8s.io/api/testdata/HEAD/*.{yaml,json,pb} | rm -f staging/src/k8s.io/api/testdata/HEAD/*.{yaml,json,pb} | ||||||
|   | |||||||
| @@ -25,7 +25,7 @@ set -o pipefail | |||||||
| KUBE_ROOT=$(dirname "${BASH_SOURCE[0]}")/.. | KUBE_ROOT=$(dirname "${BASH_SOURCE[0]}")/.. | ||||||
| source "${KUBE_ROOT}/hack/lib/init.sh" | source "${KUBE_ROOT}/hack/lib/init.sh" | ||||||
|  |  | ||||||
| kube::golang::new::setup_env | kube::golang::setup_env | ||||||
|  |  | ||||||
| BINS=( | BINS=( | ||||||
| 	cmd/gendocs | 	cmd/gendocs | ||||||
|   | |||||||
| @@ -28,7 +28,7 @@ source "${KUBE_ROOT}/hack/lib/init.sh" | |||||||
| source "${KUBE_ROOT}/hack/lib/protoc.sh" | source "${KUBE_ROOT}/hack/lib/protoc.sh" | ||||||
|  |  | ||||||
| kube::protoc::check_protoc | kube::protoc::check_protoc | ||||||
| kube::golang::new::setup_env | kube::golang::setup_env | ||||||
|  |  | ||||||
| GOPROXY=off go install k8s.io/code-generator/cmd/go-to-protobuf | GOPROXY=off go install k8s.io/code-generator/cmd/go-to-protobuf | ||||||
| GOPROXY=off go install k8s.io/code-generator/cmd/go-to-protobuf/protoc-gen-gogo | GOPROXY=off go install k8s.io/code-generator/cmd/go-to-protobuf/protoc-gen-gogo | ||||||
|   | |||||||
| @@ -28,8 +28,7 @@ KUBE_ROOT=$(dirname "${BASH_SOURCE[0]}")/.. | |||||||
| source "${KUBE_ROOT}/hack/lib/init.sh" | source "${KUBE_ROOT}/hack/lib/init.sh" | ||||||
|  |  | ||||||
| # This sets up the environment, like GOCACHE, which keeps the worktree cleaner. | # This sets up the environment, like GOCACHE, which keeps the worktree cleaner. | ||||||
| kube::golang::new::setup_env | kube::golang::setup_env | ||||||
| export GO111MODULE=on |  | ||||||
|  |  | ||||||
| cd "${KUBE_ROOT}" | cd "${KUBE_ROOT}" | ||||||
|  |  | ||||||
|   | |||||||
| @@ -26,7 +26,7 @@ KUBE_ROOT=$(dirname "${BASH_SOURCE[0]}")/.. | |||||||
| source "${KUBE_ROOT}/hack/lib/init.sh" | source "${KUBE_ROOT}/hack/lib/init.sh" | ||||||
|  |  | ||||||
| # This sets up the environment, like GOCACHE, which keeps the worktree cleaner. | # This sets up the environment, like GOCACHE, which keeps the worktree cleaner. | ||||||
| kube::golang::new::setup_env | kube::golang::setup_env | ||||||
|  |  | ||||||
| # Remove all files, some of them might be obsolete. | # Remove all files, some of them might be obsolete. | ||||||
| rm -f hack/golangci*.yaml | rm -f hack/golangci*.yaml | ||||||
|   | |||||||
| @@ -25,7 +25,7 @@ KUBE_ROOT=$(dirname "${BASH_SOURCE[0]}")/.. | |||||||
| source "${KUBE_ROOT}/hack/lib/init.sh" | source "${KUBE_ROOT}/hack/lib/init.sh" | ||||||
|  |  | ||||||
| kube::golang::verify_go_version | kube::golang::verify_go_version | ||||||
| kube::golang::new::setup_env | kube::golang::setup_env | ||||||
|  |  | ||||||
| echo 'installing mockgen' | echo 'installing mockgen' | ||||||
| pushd "${KUBE_ROOT}/hack/tools" >/dev/null | pushd "${KUBE_ROOT}/hack/tools" >/dev/null | ||||||
|   | |||||||
| @@ -27,7 +27,7 @@ OPENAPI_ROOT_DIR="${KUBE_ROOT}/api/openapi-spec" | |||||||
| source "${KUBE_ROOT}/hack/lib/init.sh" | source "${KUBE_ROOT}/hack/lib/init.sh" | ||||||
|  |  | ||||||
| kube::util::require-jq | kube::util::require-jq | ||||||
| kube::golang::new::setup_env | kube::golang::setup_env | ||||||
| kube::etcd::install | kube::etcd::install | ||||||
|  |  | ||||||
| make -C "${KUBE_ROOT}" WHAT=cmd/kube-apiserver | make -C "${KUBE_ROOT}" WHAT=cmd/kube-apiserver | ||||||
|   | |||||||
| @@ -25,7 +25,7 @@ KUBE_ROOT=$(dirname "${BASH_SOURCE[0]}")/.. | |||||||
| source "${KUBE_ROOT}/hack/lib/init.sh" | source "${KUBE_ROOT}/hack/lib/init.sh" | ||||||
|  |  | ||||||
| # Get all the default Go environment. | # Get all the default Go environment. | ||||||
| kube::golang::new::setup_env | kube::golang::setup_env | ||||||
| kube::golang::verify_go_version | kube::golang::verify_go_version | ||||||
|  |  | ||||||
| # Turn off workspaces until we are ready for them later | # Turn off workspaces until we are ready for them later | ||||||
|   | |||||||
| @@ -21,7 +21,7 @@ set -o pipefail | |||||||
| KUBE_ROOT=$(dirname "${BASH_SOURCE[0]}")/.. | KUBE_ROOT=$(dirname "${BASH_SOURCE[0]}")/.. | ||||||
| source "${KUBE_ROOT}/hack/lib/init.sh" | source "${KUBE_ROOT}/hack/lib/init.sh" | ||||||
|  |  | ||||||
| kube::golang::new::setup_env | kube::golang::setup_env | ||||||
|  |  | ||||||
| cd "${KUBE_ROOT}" | cd "${KUBE_ROOT}" | ||||||
|  |  | ||||||
|   | |||||||
| @@ -25,7 +25,7 @@ set -o pipefail | |||||||
| KUBE_ROOT=$(dirname "${BASH_SOURCE[0]}")/.. | KUBE_ROOT=$(dirname "${BASH_SOURCE[0]}")/.. | ||||||
| source "${KUBE_ROOT}/hack/lib/init.sh" | source "${KUBE_ROOT}/hack/lib/init.sh" | ||||||
|  |  | ||||||
| kube::golang::new::setup_env | kube::golang::setup_env | ||||||
|  |  | ||||||
| hack/make-rules/build.sh ./cmd/clicheck | hack/make-rules/build.sh ./cmd/clicheck | ||||||
|  |  | ||||||
|   | |||||||
| @@ -27,7 +27,7 @@ source "${KUBE_ROOT}/hack/lib/init.sh" | |||||||
| source "${KUBE_ROOT}/hack/lib/util.sh" | source "${KUBE_ROOT}/hack/lib/util.sh" | ||||||
|  |  | ||||||
| kube::golang::verify_go_version | kube::golang::verify_go_version | ||||||
| kube::golang::new::setup_env | kube::golang::setup_env | ||||||
|  |  | ||||||
| cd "${KUBE_ROOT}" | cd "${KUBE_ROOT}" | ||||||
|  |  | ||||||
|   | |||||||
| @@ -26,7 +26,7 @@ KUBE_ROOT=$(dirname "${BASH_SOURCE[0]}")/.. | |||||||
| source "${KUBE_ROOT}/hack/lib/init.sh" | source "${KUBE_ROOT}/hack/lib/init.sh" | ||||||
| source "${KUBE_ROOT}/hack/lib/util.sh" | source "${KUBE_ROOT}/hack/lib/util.sh" | ||||||
|  |  | ||||||
| kube::golang::new::setup_env | kube::golang::setup_env | ||||||
|  |  | ||||||
| make -C "${KUBE_ROOT}" WHAT=cmd/genswaggertypedocs | make -C "${KUBE_ROOT}" WHAT=cmd/genswaggertypedocs | ||||||
|  |  | ||||||
|   | |||||||
| @@ -26,7 +26,7 @@ KUBE_ROOT=$(dirname "${BASH_SOURCE[0]}")/.. | |||||||
| source "${KUBE_ROOT}/hack/lib/init.sh" | source "${KUBE_ROOT}/hack/lib/init.sh" | ||||||
| source "${KUBE_ROOT}/hack/lib/util.sh" | source "${KUBE_ROOT}/hack/lib/util.sh" | ||||||
|  |  | ||||||
| kube::golang::new::setup_env | kube::golang::setup_env | ||||||
|  |  | ||||||
| make -C "${KUBE_ROOT}" WHAT=cmd/fieldnamedocscheck | make -C "${KUBE_ROOT}" WHAT=cmd/fieldnamedocscheck | ||||||
|  |  | ||||||
|   | |||||||
| @@ -25,7 +25,7 @@ set -o pipefail | |||||||
| KUBE_ROOT=$(dirname "${BASH_SOURCE[0]}")/.. | KUBE_ROOT=$(dirname "${BASH_SOURCE[0]}")/.. | ||||||
| source "${KUBE_ROOT}/hack/lib/init.sh" | source "${KUBE_ROOT}/hack/lib/init.sh" | ||||||
|  |  | ||||||
| kube::golang::new::setup_env | kube::golang::setup_env | ||||||
|  |  | ||||||
| BINS=( | BINS=( | ||||||
| 	cmd/gendocs | 	cmd/gendocs | ||||||
|   | |||||||
| @@ -30,7 +30,7 @@ source "${KUBE_ROOT}/hack/lib/init.sh" | |||||||
|  |  | ||||||
| cd "${KUBE_ROOT}" | cd "${KUBE_ROOT}" | ||||||
|  |  | ||||||
| kube::golang::new::setup_env | kube::golang::setup_env | ||||||
|  |  | ||||||
| find_files() { | find_files() { | ||||||
|   find . -not \( \ |   find . -not \( \ | ||||||
|   | |||||||
| @@ -44,7 +44,7 @@ KUBE_ROOT=$(dirname "${BASH_SOURCE[0]}")/.. | |||||||
| source "${KUBE_ROOT}/hack/lib/init.sh" | source "${KUBE_ROOT}/hack/lib/init.sh" | ||||||
| source "${KUBE_ROOT}/hack/lib/util.sh" | source "${KUBE_ROOT}/hack/lib/util.sh" | ||||||
|  |  | ||||||
| kube::golang::new::setup_env | kube::golang::setup_env | ||||||
| kube::golang::verify_go_version | kube::golang::verify_go_version | ||||||
| export GOBIN="${KUBE_OUTPUT_BIN}" | export GOBIN="${KUBE_OUTPUT_BIN}" | ||||||
|  |  | ||||||
|   | |||||||
| @@ -25,7 +25,7 @@ source "${KUBE_ROOT}/hack/lib/util.sh" | |||||||
| kube::util::ensure_clean_working_dir | kube::util::ensure_clean_working_dir | ||||||
|  |  | ||||||
| # This sets up the environment, like GOCACHE, which keeps the worktree cleaner. | # This sets up the environment, like GOCACHE, which keeps the worktree cleaner. | ||||||
| kube::golang::new::setup_env | kube::golang::setup_env | ||||||
|  |  | ||||||
| go install golang.org/x/vuln/cmd/govulncheck@v1.0.1 | go install golang.org/x/vuln/cmd/govulncheck@v1.0.1 | ||||||
|  |  | ||||||
|   | |||||||
| @@ -26,7 +26,7 @@ set -o pipefail | |||||||
| KUBE_ROOT=$(dirname "${BASH_SOURCE[0]}")/.. | KUBE_ROOT=$(dirname "${BASH_SOURCE[0]}")/.. | ||||||
| source "${KUBE_ROOT}/hack/lib/init.sh" | source "${KUBE_ROOT}/hack/lib/init.sh" | ||||||
|  |  | ||||||
| kube::golang::new::setup_env | kube::golang::setup_env | ||||||
|  |  | ||||||
| GOPROXY=off go install k8s.io/code-generator/cmd/import-boss | GOPROXY=off go install k8s.io/code-generator/cmd/import-boss | ||||||
|  |  | ||||||
|   | |||||||
| @@ -26,7 +26,7 @@ set -o pipefail | |||||||
| KUBE_ROOT=$(dirname "${BASH_SOURCE[0]}")/.. | KUBE_ROOT=$(dirname "${BASH_SOURCE[0]}")/.. | ||||||
| source "${KUBE_ROOT}/hack/lib/init.sh" | source "${KUBE_ROOT}/hack/lib/init.sh" | ||||||
|  |  | ||||||
| kube::golang::new::setup_env | kube::golang::setup_env | ||||||
|  |  | ||||||
| make -C "${KUBE_ROOT}" WHAT=cmd/importverifier | make -C "${KUBE_ROOT}" WHAT=cmd/importverifier | ||||||
|  |  | ||||||
|   | |||||||
| @@ -27,7 +27,7 @@ source "${KUBE_ROOT}/hack/lib/init.sh" | |||||||
| source "${KUBE_ROOT}/hack/lib/util.sh" | source "${KUBE_ROOT}/hack/lib/util.sh" | ||||||
|  |  | ||||||
| # This sets up the environment, like GOCACHE, which keeps the worktree cleaner. | # This sets up the environment, like GOCACHE, which keeps the worktree cleaner. | ||||||
| kube::golang::new::setup_env | kube::golang::setup_env | ||||||
| kube::util::ensure-temp-dir | kube::util::ensure-temp-dir | ||||||
|  |  | ||||||
| ARTIFACTS="${ARTIFACTS:-${PWD}/_artifacts}" | ARTIFACTS="${ARTIFACTS:-${PWD}/_artifacts}" | ||||||
|   | |||||||
| @@ -25,7 +25,7 @@ set -o pipefail | |||||||
| KUBE_ROOT=$(dirname "${BASH_SOURCE[0]}")/.. | KUBE_ROOT=$(dirname "${BASH_SOURCE[0]}")/.. | ||||||
| source "${KUBE_ROOT}/hack/lib/init.sh" | source "${KUBE_ROOT}/hack/lib/init.sh" | ||||||
|  |  | ||||||
| kube::golang::new::setup_env | kube::golang::setup_env | ||||||
| kube::golang::verify_go_version | kube::golang::verify_go_version | ||||||
|  |  | ||||||
| staging_repos=() | staging_repos=() | ||||||
|   | |||||||
| @@ -26,7 +26,7 @@ set -o pipefail | |||||||
| KUBE_ROOT=$(dirname "${BASH_SOURCE[0]}")/.. | KUBE_ROOT=$(dirname "${BASH_SOURCE[0]}")/.. | ||||||
| source "${KUBE_ROOT}/hack/lib/init.sh" | source "${KUBE_ROOT}/hack/lib/init.sh" | ||||||
|  |  | ||||||
| kube::golang::new::setup_env | kube::golang::setup_env | ||||||
| kube::etcd::install | kube::etcd::install | ||||||
|  |  | ||||||
| make -C "${KUBE_ROOT}" WHAT=cmd/kube-apiserver | make -C "${KUBE_ROOT}" WHAT=cmd/kube-apiserver | ||||||
|   | |||||||
| @@ -25,7 +25,7 @@ KUBE_ROOT=$(dirname "${BASH_SOURCE[0]}")/.. | |||||||
| source "${KUBE_ROOT}/hack/lib/init.sh" | source "${KUBE_ROOT}/hack/lib/init.sh" | ||||||
| cd "${KUBE_ROOT}" | cd "${KUBE_ROOT}" | ||||||
|  |  | ||||||
| kube::golang::new::setup_env | kube::golang::setup_env | ||||||
| kube::golang::verify_go_version | kube::golang::verify_go_version | ||||||
|  |  | ||||||
| ret=0 | ret=0 | ||||||
|   | |||||||
| @@ -26,7 +26,7 @@ stability_check_setup() { | |||||||
|   kube::golang::verify_go_version |   kube::golang::verify_go_version | ||||||
|   kube::util::ensure-temp-dir |   kube::util::ensure-temp-dir | ||||||
|   cd "${KUBE_ROOT}" |   cd "${KUBE_ROOT}" | ||||||
|   kube::golang::new::setup_env |   kube::golang::setup_env | ||||||
| } | } | ||||||
|  |  | ||||||
| function find_files_to_check() { | function find_files_to_check() { | ||||||
|   | |||||||
| @@ -70,7 +70,7 @@ done | |||||||
| KUBE_ROOT=$(dirname "${BASH_SOURCE[0]}")/../../../ | KUBE_ROOT=$(dirname "${BASH_SOURCE[0]}")/../../../ | ||||||
| source "${KUBE_ROOT}/hack/lib/init.sh" | source "${KUBE_ROOT}/hack/lib/init.sh" | ||||||
|  |  | ||||||
| kube::golang::new::setup_env | kube::golang::setup_env | ||||||
|  |  | ||||||
| DIR_BASENAME=$(dirname "${BASH_SOURCE[0]}") | DIR_BASENAME=$(dirname "${BASH_SOURCE[0]}") | ||||||
| pushd "${DIR_BASENAME}" | pushd "${DIR_BASENAME}" | ||||||
|   | |||||||
		Reference in New Issue
	
	Block a user
	 Tim Hockin
					Tim Hockin