Merge pull request #29141 from zmerlynn/fix-node-prefix
Automatic merge from submit-queue GCE bring-up: Differentiate NODE_TAGS from NODE_INSTANCE_PREFIX
This commit is contained in:
		@@ -534,6 +534,7 @@ function build-kube-env {
 | 
				
			|||||||
ENV_TIMESTAMP: $(yaml-quote $(date -u +%Y-%m-%dT%T%z))
 | 
					ENV_TIMESTAMP: $(yaml-quote $(date -u +%Y-%m-%dT%T%z))
 | 
				
			||||||
INSTANCE_PREFIX: $(yaml-quote ${INSTANCE_PREFIX})
 | 
					INSTANCE_PREFIX: $(yaml-quote ${INSTANCE_PREFIX})
 | 
				
			||||||
NODE_INSTANCE_PREFIX: $(yaml-quote ${NODE_INSTANCE_PREFIX})
 | 
					NODE_INSTANCE_PREFIX: $(yaml-quote ${NODE_INSTANCE_PREFIX})
 | 
				
			||||||
 | 
					NODE_TAGS: $(yaml-quote ${NODE_TAGS:-})
 | 
				
			||||||
CLUSTER_IP_RANGE: $(yaml-quote ${CLUSTER_IP_RANGE:-10.244.0.0/16})
 | 
					CLUSTER_IP_RANGE: $(yaml-quote ${CLUSTER_IP_RANGE:-10.244.0.0/16})
 | 
				
			||||||
SERVER_BINARY_TAR_URL: $(yaml-quote ${server_binary_tar_url})
 | 
					SERVER_BINARY_TAR_URL: $(yaml-quote ${server_binary_tar_url})
 | 
				
			||||||
SERVER_BINARY_TAR_HASH: $(yaml-quote ${SERVER_BINARY_TAR_HASH})
 | 
					SERVER_BINARY_TAR_HASH: $(yaml-quote ${SERVER_BINARY_TAR_HASH})
 | 
				
			||||||
 
 | 
				
			|||||||
@@ -422,6 +422,7 @@ function create-salt-pillar() {
 | 
				
			|||||||
  mkdir -p /srv/salt-overlay/pillar
 | 
					  mkdir -p /srv/salt-overlay/pillar
 | 
				
			||||||
  cat <<EOF >/srv/salt-overlay/pillar/cluster-params.sls
 | 
					  cat <<EOF >/srv/salt-overlay/pillar/cluster-params.sls
 | 
				
			||||||
instance_prefix: '$(echo "$INSTANCE_PREFIX" | sed -e "s/'/''/g")'
 | 
					instance_prefix: '$(echo "$INSTANCE_PREFIX" | sed -e "s/'/''/g")'
 | 
				
			||||||
 | 
					node_tags: '$(echo "$NODE_TAGS" | sed -e "s/'/''/g")'
 | 
				
			||||||
node_instance_prefix: '$(echo "$NODE_INSTANCE_PREFIX" | sed -e "s/'/''/g")'
 | 
					node_instance_prefix: '$(echo "$NODE_INSTANCE_PREFIX" | sed -e "s/'/''/g")'
 | 
				
			||||||
cluster_cidr: '$(echo "$CLUSTER_IP_RANGE" | sed -e "s/'/''/g")'
 | 
					cluster_cidr: '$(echo "$CLUSTER_IP_RANGE" | sed -e "s/'/''/g")'
 | 
				
			||||||
allocate_node_cidrs: '$(echo "$ALLOCATE_NODE_CIDRS" | sed -e "s/'/''/g")'
 | 
					allocate_node_cidrs: '$(echo "$ALLOCATE_NODE_CIDRS" | sed -e "s/'/''/g")'
 | 
				
			||||||
@@ -814,8 +815,13 @@ EOF
 | 
				
			|||||||
  fi
 | 
					  fi
 | 
				
			||||||
 | 
					
 | 
				
			||||||
  if [[ -n "${NODE_INSTANCE_PREFIX:-}" ]]; then
 | 
					  if [[ -n "${NODE_INSTANCE_PREFIX:-}" ]]; then
 | 
				
			||||||
 | 
					    if [[ -n "${NODE_TAGS:-}" ]]; then
 | 
				
			||||||
 | 
					      local -r node_tags="${NODE_TAGS}"
 | 
				
			||||||
 | 
					    else
 | 
				
			||||||
 | 
					      local -r node_tags="${NODE_INSTANCE_PREFIX}"
 | 
				
			||||||
 | 
					    fi
 | 
				
			||||||
    cat <<EOF >>/etc/gce.conf
 | 
					    cat <<EOF >>/etc/gce.conf
 | 
				
			||||||
node-tags = ${NODE_INSTANCE_PREFIX}
 | 
					node-tags = ${NODE_TAGS}
 | 
				
			||||||
node-instance-prefix = ${NODE_INSTANCE_PREFIX}
 | 
					node-instance-prefix = ${NODE_INSTANCE_PREFIX}
 | 
				
			||||||
EOF
 | 
					EOF
 | 
				
			||||||
    CLOUD_CONFIG=/etc/gce.conf
 | 
					    CLOUD_CONFIG=/etc/gce.conf
 | 
				
			||||||
 
 | 
				
			|||||||
@@ -189,8 +189,13 @@ EOF
 | 
				
			|||||||
  fi
 | 
					  fi
 | 
				
			||||||
  if [[ -n "${NODE_INSTANCE_PREFIX:-}" ]]; then
 | 
					  if [[ -n "${NODE_INSTANCE_PREFIX:-}" ]]; then
 | 
				
			||||||
    use_cloud_config="true"
 | 
					    use_cloud_config="true"
 | 
				
			||||||
 | 
					    if [[ -n "${NODE_TAGS:-}" ]]; then
 | 
				
			||||||
 | 
					      local -r node_tags="${NODE_TAGS}"
 | 
				
			||||||
 | 
					    else
 | 
				
			||||||
 | 
					      local -r node_tags="${NODE_INSTANCE_PREFIX}"
 | 
				
			||||||
 | 
					    fi
 | 
				
			||||||
    cat <<EOF >>/etc/gce.conf
 | 
					    cat <<EOF >>/etc/gce.conf
 | 
				
			||||||
node-tags = ${NODE_INSTANCE_PREFIX}
 | 
					node-tags = ${node_tags}
 | 
				
			||||||
node-instance-prefix = ${NODE_INSTANCE_PREFIX}
 | 
					node-instance-prefix = ${NODE_INSTANCE_PREFIX}
 | 
				
			||||||
EOF
 | 
					EOF
 | 
				
			||||||
  fi
 | 
					  fi
 | 
				
			||||||
 
 | 
				
			|||||||
@@ -332,8 +332,13 @@ EOF
 | 
				
			|||||||
  fi
 | 
					  fi
 | 
				
			||||||
  if [ -n "${NODE_INSTANCE_PREFIX:-}" ]; then
 | 
					  if [ -n "${NODE_INSTANCE_PREFIX:-}" ]; then
 | 
				
			||||||
    use_cloud_config="true"
 | 
					    use_cloud_config="true"
 | 
				
			||||||
 | 
					    if [[ -n "${NODE_TAGS:-}" ]]; then
 | 
				
			||||||
 | 
					      local -r node_tags="${NODE_TAGS}"
 | 
				
			||||||
 | 
					    else
 | 
				
			||||||
 | 
					      local -r node_tags="${NODE_INSTANCE_PREFIX}"
 | 
				
			||||||
 | 
					    fi
 | 
				
			||||||
    cat <<EOF >>/etc/gce.conf
 | 
					    cat <<EOF >>/etc/gce.conf
 | 
				
			||||||
node-tags = ${NODE_INSTANCE_PREFIX}
 | 
					node-tags = ${node_tags}
 | 
				
			||||||
node-instance-prefix = ${NODE_INSTANCE_PREFIX}
 | 
					node-instance-prefix = ${NODE_INSTANCE_PREFIX}
 | 
				
			||||||
EOF
 | 
					EOF
 | 
				
			||||||
  fi
 | 
					  fi
 | 
				
			||||||
 
 | 
				
			|||||||
@@ -83,6 +83,7 @@ if [[ "${ENABLE_CLUSTER_AUTOSCALER}" == "true" ]]; then
 | 
				
			|||||||
fi
 | 
					fi
 | 
				
			||||||
 | 
					
 | 
				
			||||||
NODE_INSTANCE_PREFIX="${INSTANCE_PREFIX}-minion"
 | 
					NODE_INSTANCE_PREFIX="${INSTANCE_PREFIX}-minion"
 | 
				
			||||||
 | 
					NODE_TAGS="${NODE_TAG}"
 | 
				
			||||||
 | 
					
 | 
				
			||||||
ALLOCATE_NODE_CIDRS=true
 | 
					ALLOCATE_NODE_CIDRS=true
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 
 | 
				
			|||||||
		Reference in New Issue
	
	Block a user