Commit 682dd146 authored by Daniel Smith's avatar Daniel Smith

Merge pull request #3040 from brendandburns/pd2

Fix the update e2e test to remove a race.
parents 4943ee7b 4833b88a
...@@ -44,6 +44,10 @@ function validate() { ...@@ -44,6 +44,10 @@ function validate() {
local id local id
num_running=0 num_running=0
if [[ ${#pod_id_list[@]} -ne $num_replicas ]]; then
echo "Too few or too many replicas."
continue
fi
for id in "${pod_id_list[@]+${pod_id_list[@]}}"; do for id in "${pod_id_list[@]+${pod_id_list[@]}}"; do
local template_string current_status current_image host_ip local template_string current_status current_image host_ip
...@@ -78,7 +82,7 @@ function validate() { ...@@ -78,7 +82,7 @@ function validate() {
fi fi
template_string="{{(index .currentState.info \"${CONTROLLER_NAME}\").image}}" template_string="{{(index .currentState.info \"${CONTROLLER_NAME}\").image}}"
current_image=$($KUBECFG -template="${template_string}" get "pods/$id") current_image=$($KUBECFG -template="${template_string}" get "pods/$id") || true
if [[ "$current_image" != "${DOCKER_HUB_USER}/update-demo:${container_image_version}" ]]; then if [[ "$current_image" != "${DOCKER_HUB_USER}/update-demo:${container_image_version}" ]]; then
echo " ${id} is created but running wrong image" echo " ${id} is created but running wrong image"
continue continue
......
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