Commit 35837d80 authored by Kubernetes Submit Queue's avatar Kubernetes Submit Queue Committed by GitHub

Merge pull request #45605 from rmmh/fix-ip-alias

Automatic merge from submit-queue Fix ip-alias testing. IP aliases are an alpha feature, and node accelerators are a beta feature. $gcloud determines which is appropriate. Before, this would try to run "gcloud alpha beta", which is incoherent. ```release-note NOTE ```
parents aba95a16 62235c3b
...@@ -505,6 +505,15 @@ function create-node-template() { ...@@ -505,6 +505,15 @@ function create-node-template() {
fi fi
local gcloud="gcloud" local gcloud="gcloud"
local accelerator_args=""
# VMs with Accelerators cannot be live migrated.
# More details here - https://cloud.google.com/compute/docs/gpus/add-gpus#create-new-gpu-instance
if [[ ! -z "${NODE_ACCELERATORS}" ]]; then
accelerator_args="--maintenance-policy TERMINATE --restart-on-failure --accelerator ${NODE_ACCELERATORS}"
gcloud="gcloud beta"
fi
if [[ "${ENABLE_IP_ALIASES:-}" == 'true' ]]; then if [[ "${ENABLE_IP_ALIASES:-}" == 'true' ]]; then
gcloud="gcloud alpha" gcloud="gcloud alpha"
fi fi
...@@ -527,16 +536,10 @@ function create-node-template() { ...@@ -527,16 +536,10 @@ function create-node-template() {
"${IP_ALIAS_SUBNETWORK:-}" \ "${IP_ALIAS_SUBNETWORK:-}" \
"${IP_ALIAS_SIZE:-}") "${IP_ALIAS_SIZE:-}")
local accelerator_args=""
# VMs with Accelerators cannot be live migrated.
# More details here - https://cloud.google.com/compute/docs/gpus/add-gpus#create-new-gpu-instance
if [[ ! -z "${NODE_ACCELERATORS}" ]]; then
accelerator_args="--maintenance-policy TERMINATE --restart-on-failure --accelerator ${NODE_ACCELERATORS}"
fi
local attempt=1 local attempt=1
while true; do while true; do
echo "Attempt ${attempt} to create ${1}" >&2 echo "Attempt ${attempt} to create ${1}" >&2
if ! ${gcloud} beta compute instance-templates create \ if ! ${gcloud} compute instance-templates create \
"$template_name" \ "$template_name" \
--project "${PROJECT}" \ --project "${PROJECT}" \
--machine-type "${NODE_SIZE}" \ --machine-type "${NODE_SIZE}" \
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment