-
Notifications
You must be signed in to change notification settings - Fork 43
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
cert-manager: optional webhook #1413
Conversation
@@ -26,7 +26,7 @@ pharos license assign --help || exit $? | |||
pharos license inspect --help || exit $? | |||
|
|||
# Test cluster bootstrapping | |||
timeout 600 pharos up -y -c e2e/digitalocean/cluster.yml --tf-json e2e/digitalocean/tf.json || exit $? | |||
timeout 700 pharos up -y -c e2e/digitalocean/cluster.yml --tf-json e2e/digitalocean/tf.json || exit $? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Had to raise timeout limit, it was too close.
@@ -75,8 +75,8 @@ ssh_userhost="root@${pharos_worker_host}" | |||
scp -o StrictHostKeyChecking=no -i "${ssh_key}" pharos-cluster*.gem "${ssh_userhost}:" | |||
|
|||
ssh -o StrictHostKeyChecking=no -o ServerAliveInterval=30 -i "${ssh_key}" "${ssh_userhost}" -- "sudo bash -c '\ | |||
export http_proxy=http://10.133.37.156:8888 HTTP_PROXY=http://10.133.37.156:8888 https_proxy=http://10.133.37.156:8888 HTTPS_PROXY=http://10.133.37.156:8888; | |||
apt-get update && apt-get -y install ruby build-essential ruby-dev && \ | |||
export http_proxy=http://10.133.37.156:8888 HTTP_PROXY=http://10.133.37.156:8888 https_proxy=http://10.133.37.156:8888 HTTPS_PROXY=http://10.133.37.156:8888 DEBIAN_FRONTEND=noninteractive; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Had to fix this because libssl
upgrade prompted something.
e2e/drone_setup.sh
Outdated
@@ -10,15 +10,22 @@ fi | |||
cd e2e/digitalocean | |||
terraform init | |||
|
|||
until terraform apply -auto-approve -var "cluster_name=${DRONE_BUILD_NUMBER}-do" -var "image=$1" | |||
WORKER_UP=${WORKER_UP:-false} | |||
if [ "${WORKER_UP}" = "true" ]; then |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reason for this: we should only provision extra machine if we test "worker up".
e2e/drone_setup.sh
Outdated
do | ||
echo "Apply failed... trying again in 5s" | ||
sleep 5 | ||
done | ||
|
||
terraform output -json > tf.json | ||
|
||
if [ "${WORKER_UP:-false}" = "up" ]; then | ||
if [ "${WORKER_UP}" = "true" ]; then |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We didn't really test "worker up" at all..
Co-Authored-By: Kimmo Lehto <kimmo.lehto@gmail.com>
Webhook makes initial cluster deployment very slow.. this PR allows to disable it.