From 39cfbf95f57fa263b9a36a69c64c48e568ab6ae1 Mon Sep 17 00:00:00 2001 From: Jonathan Basseri Date: Wed, 9 May 2018 13:51:40 -0700 Subject: [PATCH] test: allow etcd to run on other ports. running `make test-integration` with the `ETCD_PORT` option set should work. Numerous integration test expect the correct URL to be populated in the `KUBE_INTEGRATION_ETCD_URL` environment variable. --- hack/lib/etcd.sh | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/hack/lib/etcd.sh b/hack/lib/etcd.sh index 3719dbf67b..7adf0f053b 100755 --- a/hack/lib/etcd.sh +++ b/hack/lib/etcd.sh @@ -19,6 +19,7 @@ ETCD_VERSION=${ETCD_VERSION:-3.2.18} ETCD_HOST=${ETCD_HOST:-127.0.0.1} ETCD_PORT=${ETCD_PORT:-2379} +export KUBE_INTEGRATION_ETCD_URL="http://${ETCD_HOST}:${ETCD_PORT}" kube::etcd::validate() { # validate if in path @@ -73,13 +74,13 @@ kube::etcd::start() { else ETCD_LOGFILE=/dev/null fi - kube::log::info "etcd --advertise-client-urls http://${ETCD_HOST}:${ETCD_PORT} --data-dir ${ETCD_DIR} --listen-client-urls http://${ETCD_HOST}:${ETCD_PORT} --debug > \"${ETCD_LOGFILE}\" 2>/dev/null" - etcd --advertise-client-urls http://${ETCD_HOST}:${ETCD_PORT} --data-dir ${ETCD_DIR} --listen-client-urls http://${ETCD_HOST}:${ETCD_PORT} --debug 2> "${ETCD_LOGFILE}" >/dev/null & + kube::log::info "etcd --advertise-client-urls ${KUBE_INTEGRATION_ETCD_URL} --data-dir ${ETCD_DIR} --listen-client-urls http://${ETCD_HOST}:${ETCD_PORT} --debug > \"${ETCD_LOGFILE}\" 2>/dev/null" + etcd --advertise-client-urls ${KUBE_INTEGRATION_ETCD_URL} --data-dir ${ETCD_DIR} --listen-client-urls ${KUBE_INTEGRATION_ETCD_URL} --debug 2> "${ETCD_LOGFILE}" >/dev/null & ETCD_PID=$! echo "Waiting for etcd to come up." - kube::util::wait_for_url "http://${ETCD_HOST}:${ETCD_PORT}/v2/machines" "etcd: " 0.25 80 - curl -fs -X PUT "http://${ETCD_HOST}:${ETCD_PORT}/v2/keys/_test" + kube::util::wait_for_url "${KUBE_INTEGRATION_ETCD_URL}/v2/machines" "etcd: " 0.25 80 + curl -fs -X PUT "${KUBE_INTEGRATION_ETCD_URL}/v2/keys/_test" } kube::etcd::stop() {