Skip to content

Fix broken build/push-federation-images.sh#34229

Merged
k8s-github-robot merged 1 commit intokubernetes:masterfrom
jbeda:fix-federation-push
Oct 6, 2016
Merged

Fix broken build/push-federation-images.sh#34229
k8s-github-robot merged 1 commit intokubernetes:masterfrom
jbeda:fix-federation-push

Conversation

@jbeda
Copy link
Contributor

@jbeda jbeda commented Oct 6, 2016

The federation CI build is broken by #30787. A stray bash source caused an undefined variable reference.

Apparently the federation images have a parallel nad different "release" path that isn't tested by the pre-checkin tests.


This change is Reviewable

@jbeda jbeda added priority/critical-urgent Highest priority. Must be actively worked on as someone's top priority right now. area/build-release labels Oct 6, 2016
@jbeda
Copy link
Contributor Author

jbeda commented Oct 6, 2016

I'm going to self-LGTM here as there is a CI build break that is ongoing.

@jbeda jbeda added release-note-none Denotes a PR that doesn't merit a release note. lgtm "Looks good to me", indicates that a PR is ready to be merged. retest-not-required and removed area/build-release labels Oct 6, 2016
@k8s-github-robot k8s-github-robot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Oct 6, 2016
@jbeda jbeda force-pushed the fix-federation-push branch from 3ed99c4 to 7ba1d6d Compare October 6, 2016 13:23
This was broken by kubernetes#30787. A stray bash `source` caused an undefined variable reference.

Apparently the federation images have a parallel nad different "release" path
that isn't tested by the pre-checkin tests.
@jbeda jbeda force-pushed the fix-federation-push branch from 7ba1d6d to ce00838 Compare October 6, 2016 13:23
@k8s-github-robot k8s-github-robot removed the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Oct 6, 2016
@k8s-ci-robot
Copy link
Contributor

Jenkins GCI GKE smoke e2e failed for commit ce00838. Full PR test history.

The magic incantation to run this job again is @k8s-bot gci gke e2e test this. Please help us cut down flakes by linking to an open flake issue when you hit one in your PR.

@k8s-ci-robot
Copy link
Contributor

Jenkins GKE smoke e2e failed for commit ce00838. Full PR test history.

The magic incantation to run this job again is @k8s-bot gke e2e test this. Please help us cut down flakes by linking to an open flake issue when you hit one in your PR.

@jbeda jbeda added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Oct 6, 2016
@k8s-github-robot
Copy link

Automatic merge from submit-queue

@k8s-github-robot k8s-github-robot merged commit 4077c86 into kubernetes:master Oct 6, 2016
@ixdy
Copy link
Contributor

ixdy commented Oct 6, 2016

LGTM

@jbeda jbeda deleted the fix-federation-push branch October 13, 2016 18:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm "Looks good to me", indicates that a PR is ready to be merged. priority/critical-urgent Highest priority. Must be actively worked on as someone's top priority right now. 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.

5 participants