Skip to content

Run e2e tests inside Docker on PR Jenkins#432

Merged
ixdy merged 2 commits intokubernetes:masterfrom
ixdy:dockerized-e2e
Aug 23, 2016
Merged

Run e2e tests inside Docker on PR Jenkins#432
ixdy merged 2 commits intokubernetes:masterfrom
ixdy:dockerized-e2e

Conversation

@ixdy
Copy link
Copy Markdown
Contributor

@ixdy ixdy commented Aug 23, 2016

@ixdy
Copy link
Copy Markdown
Contributor Author

ixdy commented Aug 23, 2016

This has the side effect of fixing kubernetes/kubernetes#31148 too (at least the bc issue). cc @bgrant0607

@spxtr
Copy link
Copy Markdown
Contributor

spxtr commented Aug 23, 2016

Looks reasonable to me. Lets merge tomorrow morning.

@fejta
Copy link
Copy Markdown
Contributor

fejta commented Aug 23, 2016

LGTM

@fejta fejta added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Aug 23, 2016
@ixdy ixdy merged commit 646162d into kubernetes:master Aug 23, 2016
k8s-github-robot pushed a commit to kubernetes/kubernetes that referenced this pull request Aug 24, 2016
Automatic merge from submit-queue

Explicitly set CLOUDSDK_CONFIG override early in e2e-runner

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
foxish pushed a commit to foxish/test-infra that referenced this pull request Jan 21, 2017
[mungegithub] Only run ping-ci test on master
@ixdy ixdy deleted the dockerized-e2e branch May 15, 2018 23:50
ostromart pushed a commit to ostromart/test-infra that referenced this pull request Jul 26, 2019
grantr pushed a commit to grantr/test-infra that referenced this pull request Feb 21, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/jenkins lgtm "Looks good to me", indicates that a PR is ready to be merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Dockerize PR Jenkins

4 participants