Skip to content

Explicitly set CLOUDSDK_CONFIG override early in e2e-runner#31330

Merged
k8s-github-robot merged 1 commit intokubernetes:masterfrom
ixdy:kubekins-docker
Aug 24, 2016
Merged

Explicitly set CLOUDSDK_CONFIG override early in e2e-runner#31330
k8s-github-robot merged 1 commit intokubernetes:masterfrom
ixdy:kubekins-docker

Conversation

@ixdy
Copy link
Copy Markdown
Contributor

@ixdy ixdy commented Aug 24, 2016

I'm guessing this is why a number of gci-on-gke builds started failing after kubernetes/test-infra#432 was merged - we're overriding the gcloud configuration location from the default, so the activated service account no longer works. Previously we did this service activation late, but after we'd overridden the value.

I've changed it to explicitly set to a known-good value every time at the very start.

cc @vishh @maisem @spxtr @fejta


This change is Reviewable

@ixdy ixdy added area/test-infra release-note-none Denotes a PR that doesn't merit a release note. labels Aug 24, 2016
@ixdy ixdy changed the title Remove CLOUDSDK_CONFIG override from e2e-runner Explicitly set CLOUDSDK_CONFIG override early in e2e-runner Aug 24, 2016
@k8s-github-robot k8s-github-robot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Aug 24, 2016
@k8s-bot
Copy link
Copy Markdown

k8s-bot commented Aug 24, 2016

GCE e2e build/test passed for commit 12ea9cb.

@vishh
Copy link
Copy Markdown
Contributor

vishh commented Aug 24, 2016

lgtm

@vishh vishh added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Aug 24, 2016
@vishh vishh added this to the v1.4 milestone Aug 24, 2016
@k8s-github-robot
Copy link
Copy Markdown

Automatic merge from submit-queue

@k8s-github-robot k8s-github-robot merged commit b551bf1 into kubernetes:master Aug 24, 2016
: ${KUBE_GCS_RELEASE_BUCKET:="kubernetes-release"}
: ${KUBE_GCS_DEV_RELEASE_BUCKET:="kubernetes-release-dev"}

# Explicitly set config path so staging gcloud (if installed_ uses the same path
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

maybe fix the typo though @fejta (s/_/)/)

@ixdy ixdy deleted the kubekins-docker branch May 15, 2018 23:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/test-infra lgtm "Looks good to me", indicates that a PR is ready to be merged. release-note-none Denotes a PR that doesn't merit a release note. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants