From 1d40c3ff76a2274c7209edf3a99637e4c1df5377 Mon Sep 17 00:00:00 2001 From: deads2k Date: Tue, 14 Feb 2017 10:04:28 -0500 Subject: [PATCH] update scripts for new kube-aggregator location --- build/BUILD | 2 +- build/release-tars/BUILD | 2 +- hack/godep-save.sh | 3 ++ hack/lib/golang.sh | 2 +- hack/local-up-cluster.sh | 2 +- hack/local-up-kube-aggregator.sh | 8 ++--- hack/make-rules/test.sh | 3 ++ hack/update-codegen.sh | 2 +- hack/verify-codegen.sh | 2 +- hack/verify-staging-imports.sh | 7 +++-- .../kube-aggregator/hack/build-image.sh | 6 ++-- .../kube-aggregator/hack/update-codegen.sh | 31 ++++++++++--------- .../kube-aggregator/hack/verify-codegen.sh | 2 +- vendor/k8s.io/kube-aggregator | 1 + 14 files changed, 41 insertions(+), 32 deletions(-) create mode 120000 vendor/k8s.io/kube-aggregator diff --git a/build/BUILD b/build/BUILD index dbf1f0e3ee0..c89454ecfda 100644 --- a/build/BUILD +++ b/build/BUILD @@ -65,7 +65,7 @@ DOCKERIZED_BINARIES = { }, "kube-aggregator": { "base": ":busybox-libc", - "target": "//cmd/kube-aggregator:kube-aggregator", + "target": "//vendor/k8s.io/kube-aggregator:kube-aggregator", }, "kube-proxy": { "base": ":busybox-net", diff --git a/build/release-tars/BUILD b/build/release-tars/BUILD index a8df00cb712..5b0a1debc5f 100644 --- a/build/release-tars/BUILD +++ b/build/release-tars/BUILD @@ -69,7 +69,7 @@ NODE_TARGETS = [ # since we include them in the actual build rule. SERVER_TARGETS = [ "//cmd/hyperkube", - "//cmd/kube-aggregator", + "//vendor/k8s.io/kube-aggregator", "//cmd/kube-apiserver", "//cmd/kube-controller-manager", "//cmd/kube-discovery", diff --git a/hack/godep-save.sh b/hack/godep-save.sh index a66cc679bf6..87d017e499b 100755 --- a/hack/godep-save.sh +++ b/hack/godep-save.sh @@ -47,6 +47,9 @@ pushd "${KUBE_ROOT}" > /dev/null if [ ! -e "vendor/k8s.io/apimachinery" ]; then ln -s ../../staging/src/k8s.io/apimachinery vendor/k8s.io/apimachinery fi + if [ ! -e "vendor/k8s.io/kube-aggregator" ]; then + ln -s ../../staging/src/k8s.io/kube-aggregator vendor/k8s.io/kube-aggregator + fi if [ ! -e "vendor/k8s.io/sample-apiserver" ]; then ln -s ../../staging/src/k8s.io/sample-apiserver vendor/k8s.io/sample-apiserver fi diff --git a/hack/lib/golang.sh b/hack/lib/golang.sh index 8e57e6915fd..a8022826c96 100755 --- a/hack/lib/golang.sh +++ b/hack/lib/golang.sh @@ -32,7 +32,7 @@ kube::golang::server_targets() { cmd/kubeadm cmd/hyperkube cmd/kube-discovery - cmd/kube-aggregator + vendor/k8s.io/kube-aggregator plugin/cmd/kube-scheduler ) echo "${targets[@]}" diff --git a/hack/local-up-cluster.sh b/hack/local-up-cluster.sh index 261db2a6892..e963e98ff42 100755 --- a/hack/local-up-cluster.sh +++ b/hack/local-up-cluster.sh @@ -144,7 +144,7 @@ do done if [ "x$GO_OUT" == "x" ]; then - make -C "${KUBE_ROOT}" WHAT="cmd/kubectl cmd/hyperkube cmd/kube-aggregator" + make -C "${KUBE_ROOT}" WHAT="cmd/kubectl cmd/hyperkube vendor/k8s.io/kube-aggregator" else echo "skipped the build." fi diff --git a/hack/local-up-kube-aggregator.sh b/hack/local-up-kube-aggregator.sh index 6a3442dca8e..5440dc86941 100755 --- a/hack/local-up-kube-aggregator.sh +++ b/hack/local-up-kube-aggregator.sh @@ -66,7 +66,7 @@ function start_kube-aggregator { # make sure the resources we're about to create don't exist kubectl_core -n kube-public delete secret auth-proxy-client serving-etcd serving-kube-aggregator kube-aggregator-etcd > /dev/null 2>&1 || true kubectl_core -n kube-public delete configmap etcd-ca kube-aggregator-ca client-ca request-header-ca > /dev/null 2>&1 || true - kubectl_core -n kube-public delete -f "${KUBE_ROOT}/cmd/kube-aggregator/artifacts/local-cluster-up" > /dev/null 2>&1 || true + kubectl_core -n kube-public delete -f "${KUBE_ROOT}/vendor/k8s.io/kube-aggregator/artifacts/local-cluster-up" > /dev/null 2>&1 || true sudo_kubectl_core -n kube-public create secret tls auth-proxy-client --cert="${CERT_DIR}/client-auth-proxy.crt" --key="${CERT_DIR}/client-auth-proxy.key" sudo_kubectl_core -n kube-public create secret tls serving-etcd --cert="${CERT_DIR}/serving-etcd.crt" --key="${CERT_DIR}/serving-etcd.key" @@ -77,9 +77,9 @@ function start_kube-aggregator { kubectl_core -n kube-public create configmap client-ca --from-file="ca.crt=${CERT_DIR}/client-ca.crt" || true kubectl_core -n kube-public create configmap request-header-ca --from-file="ca.crt=${CERT_DIR}/request-header-ca.crt" || true - ${KUBE_ROOT}/cmd/kube-aggregator/hack/build-image.sh + ${KUBE_ROOT}/vendor/k8s.io/kube-aggregator/hack/build-image.sh - kubectl_core -n kube-public create -f "${KUBE_ROOT}/cmd/kube-aggregator/artifacts/local-cluster-up" + kubectl_core -n kube-public create -f "${KUBE_ROOT}/vendor/k8s.io/kube-aggregator/artifacts/local-cluster-up" ${sudo} cp "${CERT_DIR}/admin.kubeconfig" "${CERT_DIR}/admin-kube-aggregator.kubeconfig" ${sudo} chown ${USER} "${CERT_DIR}/admin-kube-aggregator.kubeconfig" @@ -94,7 +94,7 @@ function start_kube-aggregator { sleep 1 # create the "normal" api services for the core API server - ${kubectl} --kubeconfig="${CERT_DIR}/admin-kube-aggregator.kubeconfig" create -f "${KUBE_ROOT}/cmd/kube-aggregator/artifacts/core-apiservices" + ${kubectl} --kubeconfig="${CERT_DIR}/admin-kube-aggregator.kubeconfig" create -f "${KUBE_ROOT}/vendor/k8s.io/kube-aggregator/artifacts/core-apiservices" } kube::util::test_openssl_installed diff --git a/hack/make-rules/test.sh b/hack/make-rules/test.sh index c3cbfd3d02f..cbb23652b8f 100755 --- a/hack/make-rules/test.sh +++ b/hack/make-rules/test.sh @@ -67,6 +67,9 @@ kube::test::find_dirs() { # run tests for apimachinery find ./staging/src/k8s.io/apimachinery -name '*_test.go' \ -name '*_test.go' -print0 | xargs -0n1 dirname | sed 's|^\./staging/src/|./vendor/|' | LC_ALL=C sort -u + + find ./staging/src/k8s.io/kube-aggregator -name '*_test.go' \ + -name '*_test.go' -print0 | xargs -0n1 dirname | sed 's|^\./staging/src/|./vendor/|' | LC_ALL=C sort -u ) } diff --git a/hack/update-codegen.sh b/hack/update-codegen.sh index 7b798d23606..0f407cce5a1 100755 --- a/hack/update-codegen.sh +++ b/hack/update-codegen.sh @@ -101,4 +101,4 @@ ${informergen} \ # You may add additional calls of code generators like set-gen above. # call generation on sub-project for now -cmd/kube-aggregator/hack/update-codegen.sh +vendor/k8s.io/kube-aggregator/hack/update-codegen.sh diff --git a/hack/verify-codegen.sh b/hack/verify-codegen.sh index 6b70557629e..80910243dc2 100755 --- a/hack/verify-codegen.sh +++ b/hack/verify-codegen.sh @@ -24,7 +24,7 @@ source "${KUBE_ROOT}/hack/lib/init.sh" kube::golang::setup_env # call verify on sub-project for now -cmd/kube-aggregator/hack/verify-codegen.sh +vendor/k8s.io/kube-aggregator/hack/verify-codegen.sh "${KUBE_ROOT}/hack/update-codegen.sh" --verify-only diff --git a/hack/verify-staging-imports.sh b/hack/verify-staging-imports.sh index b48441079af..9a299eca312 100755 --- a/hack/verify-staging-imports.sh +++ b/hack/verify-staging-imports.sh @@ -51,9 +51,10 @@ function print_forbidden_imports () { RC=0 print_forbidden_imports apimachinery k8s.io/ || RC=1 -print_forbidden_imports apiserver k8s.io/kubernetes || RC=1 -print_forbidden_imports client-go k8s.io/kubernetes k8s.io/apiserver || RC=1 -print_forbidden_imports sample-apiserver k8s.io/kubernetes || RC=1 +print_forbidden_imports apiserver k8s.io/kubernetes k8s.io/sample-apiserver k8s.io/kube-aggregator || RC=1 +print_forbidden_imports client-go k8s.io/kubernetes k8s.io/apiserver k8s.io/sample-apiserver k8s.io/kube-aggregator || RC=1 +print_forbidden_imports kube-aggregator k8s.io/kubernetes k8s.io/sample-apiserver || RC=1 +print_forbidden_imports sample-apiserver k8s.io/kubernetes k8s.io/kube-aggregator || RC=1 if [ ${RC} != 0 ]; then exit ${RC} fi diff --git a/staging/src/k8s.io/kube-aggregator/hack/build-image.sh b/staging/src/k8s.io/kube-aggregator/hack/build-image.sh index c753eee9bba..014252f3046 100755 --- a/staging/src/k8s.io/kube-aggregator/hack/build-image.sh +++ b/staging/src/k8s.io/kube-aggregator/hack/build-image.sh @@ -20,9 +20,9 @@ source "${KUBE_ROOT}/hack/lib/util.sh" # Register function to be called on EXIT to remove generated binary. function cleanup { - rm "${KUBE_ROOT}/cmd/kube-aggregator/artifacts/simple-image/kube-aggregator" + rm "${KUBE_ROOT}/vendor/k8s.io/kube-aggregator/artifacts/simple-image/kube-aggregator" } trap cleanup EXIT -cp -v ${KUBE_ROOT}/_output/local/bin/linux/amd64/kube-aggregator "${KUBE_ROOT}/cmd/kube-aggregator/artifacts/simple-image/kube-aggregator" -docker build -t kube-aggregator:latest ${KUBE_ROOT}/cmd/kube-aggregator/artifacts/simple-image +cp -v ${KUBE_ROOT}/_output/local/bin/linux/amd64/kube-aggregator "${KUBE_ROOT}/vendor/k8s.io/kube-aggregator/artifacts/simple-image/kube-aggregator" +docker build -t kube-aggregator:latest ${KUBE_ROOT}/vendor/k8s.io/kube-aggregator/artifacts/simple-image diff --git a/staging/src/k8s.io/kube-aggregator/hack/update-codegen.sh b/staging/src/k8s.io/kube-aggregator/hack/update-codegen.sh index 539f888efc1..6e4aab5ec59 100755 --- a/staging/src/k8s.io/kube-aggregator/hack/update-codegen.sh +++ b/staging/src/k8s.io/kube-aggregator/hack/update-codegen.sh @@ -18,7 +18,7 @@ set -o errexit set -o nounset set -o pipefail -KUBE_ROOT=$(dirname "${BASH_SOURCE}")/../../.. +KUBE_ROOT=$(dirname "${BASH_SOURCE}")/../../../../.. APIFEDERATOR_ROOT=$(dirname "${BASH_SOURCE}")/.. source "${KUBE_ROOT}/hack/lib/init.sh" @@ -43,26 +43,26 @@ echo "Building client-gen" CLIENTGEN="${PWD}/client-gen-binary" go build -o "${CLIENTGEN}" ./cmd/libs/go2idl/client-gen -PREFIX=k8s.io/kubernetes/cmd/kube-aggregator/pkg/apis +PREFIX=k8s.io/kube-aggregator/pkg/apis INPUT_BASE="--input-base ${PREFIX}" INPUT_APIS=( apiregistration/ apiregistration/v1alpha1 ) INPUT="--input ${INPUT_APIS[@]}" -CLIENTSET_PATH="--clientset-path k8s.io/kubernetes/cmd/kube-aggregator/pkg/client/clientset_generated" +CLIENTSET_PATH="--clientset-path k8s.io/kube-aggregator/pkg/client/clientset_generated" -${CLIENTGEN} ${INPUT_BASE} ${INPUT} ${CLIENTSET_PATH} -${CLIENTGEN} --clientset-name="clientset" ${INPUT_BASE} --input apiregistration/v1alpha1 ${CLIENTSET_PATH} +${CLIENTGEN} ${INPUT_BASE} ${INPUT} ${CLIENTSET_PATH} --output-base ${KUBE_ROOT}/vendor +${CLIENTGEN} --clientset-name="clientset" ${INPUT_BASE} --input apiregistration/v1alpha1 ${CLIENTSET_PATH} --output-base ${KUBE_ROOT}/vendor echo "Building lister-gen" listergen="${PWD}/lister-gen" go build -o "${listergen}" ./cmd/libs/go2idl/lister-gen -LISTER_INPUT="--input-dirs k8s.io/kubernetes/cmd/kube-aggregator/pkg/apis/apiregistration --input-dirs k8s.io/kubernetes/cmd/kube-aggregator/pkg/apis/apiregistration/v1alpha1" -LISTER_PATH="--output-package k8s.io/kubernetes/cmd/kube-aggregator/pkg/client/listers" -${listergen} ${LISTER_INPUT} ${LISTER_PATH} +LISTER_INPUT="--input-dirs k8s.io/kube-aggregator/pkg/apis/apiregistration --input-dirs k8s.io/kube-aggregator/pkg/apis/apiregistration/v1alpha1" +LISTER_PATH="--output-package k8s.io/kube-aggregator/pkg/client/listers" +${listergen} ${LISTER_INPUT} ${LISTER_PATH} --output-base ${KUBE_ROOT}/vendor echo "Building informer-gen" @@ -70,19 +70,20 @@ informergen="${PWD}/informer-gen" go build -o "${informergen}" ./cmd/libs/go2idl/informer-gen ${informergen} \ - --input-dirs k8s.io/kubernetes/cmd/kube-aggregator/pkg/apis/apiregistration --input-dirs k8s.io/kubernetes/cmd/kube-aggregator/pkg/apis/apiregistration/v1alpha1 \ - --versioned-clientset-package k8s.io/kubernetes/cmd/kube-aggregator/pkg/client/clientset_generated/clientset \ - --internal-clientset-package k8s.io/kubernetes/cmd/kube-aggregator/pkg/client/clientset_generated/internalclientset \ - --listers-package k8s.io/kubernetes/cmd/kube-aggregator/pkg/client/listers \ - --output-package k8s.io/kubernetes/cmd/kube-aggregator/pkg/client/informers + --output-base ${KUBE_ROOT}/vendor \ + --input-dirs k8s.io/kube-aggregator/pkg/apis/apiregistration --input-dirs k8s.io/kube-aggregator/pkg/apis/apiregistration/v1alpha1 \ + --versioned-clientset-package k8s.io/kube-aggregator/pkg/client/clientset_generated/clientset \ + --internal-clientset-package k8s.io/kube-aggregator/pkg/client/clientset_generated/internalclientset \ + --listers-package k8s.io/kube-aggregator/pkg/client/listers \ + --output-package k8s.io/kube-aggregator/pkg/client/informers "$@" # this is a temporary hack until we manage to update codegen to accept a scheme instead of hardcoding it echo "rewriting imports" -grep -R -H "\"k8s.io/kubernetes/pkg" "${KUBE_ROOT}/cmd/kube-aggregator/pkg/client" | cut -d: -f1 | sort | uniq | \ +grep -R -H "\"k8s.io/kubernetes/pkg" "${KUBE_ROOT}/vendor/k8s.io/kube-aggregator/pkg/client" | cut -d: -f1 | sort | uniq | \ grep "\.go" | \ xargs ${SED} -i "s|\"k8s.io/kubernetes/pkg|\"k8s.io/client-go/pkg|g" echo "running gofmt" -find "${KUBE_ROOT}/cmd/kube-aggregator/pkg/client" -type f -name "*.go" -print0 | xargs -0 gofmt -w +find "${KUBE_ROOT}/vendor/k8s.io/kube-aggregator/pkg/client" -type f -name "*.go" -print0 | xargs -0 gofmt -w diff --git a/staging/src/k8s.io/kube-aggregator/hack/verify-codegen.sh b/staging/src/k8s.io/kube-aggregator/hack/verify-codegen.sh index 79a834b4cab..4c6f51af8bf 100755 --- a/staging/src/k8s.io/kube-aggregator/hack/verify-codegen.sh +++ b/staging/src/k8s.io/kube-aggregator/hack/verify-codegen.sh @@ -18,7 +18,7 @@ set -o errexit set -o nounset set -o pipefail -KUBE_ROOT=$(dirname "${BASH_SOURCE}")/../../.. +KUBE_ROOT=$(dirname "${BASH_SOURCE}")/../../../../.. APIFEDERATOR_ROOT=$(dirname "${BASH_SOURCE}")/.. source "${KUBE_ROOT}/hack/lib/init.sh" diff --git a/vendor/k8s.io/kube-aggregator b/vendor/k8s.io/kube-aggregator new file mode 120000 index 00000000000..0047935ae76 --- /dev/null +++ b/vendor/k8s.io/kube-aggregator @@ -0,0 +1 @@ +../../staging/src/k8s.io/kube-aggregator \ No newline at end of file