-
Notifications
You must be signed in to change notification settings - Fork 38.6k
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
Remove undefined color_ variables #19749
Conversation
Labelling this PR as size/XS |
GCE e2e build/test failed for commit f15ab1ba2492ef1d0f02907f394a4e7a20868e8e. |
@@ -30,6 +30,7 @@ ZONE=${ZONE:-} | |||
REGION=${ZONE%-*} | |||
INSTANCE_PREFIX=${KUBE_GCE_INSTANCE_PREFIX:-${CLUSTER_NAME:-}} | |||
NETWORK=${KUBE_GCE_NETWORK:-${KUBE_GKE_NETWORK:-}} | |||
source "${KUBE_ROOT}/cluster/kube-env.sh" |
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.
KUBE_ROOT is unbound here
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.
I suspected that, I sould probably source util.sh or something
Yeah i just removed color_ |
SGTM |
GCE e2e test build/test passed for commit 7a97163. |
This will help with a flake. |
Remove undefined color_ variables
I'm guessing this is what happened in http://kubekins.dls.corp.google.com/job/kubernetes-e2e-gce-reboot/10502/consoleFull because everything passed except a list operation, which we should've retried.
I think KUBE_ROOT should be in the env.