Skip to content
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

Fix KeyError for 3-part versions. #2801

Merged
merged 1 commit into from
May 18, 2017
Merged

Conversation

enisoc
Copy link
Member

@enisoc enisoc commented May 18, 2017

Fixes #2778.

cc @fejta @david-mcmahon

This is blocking kubernetes/kubernetes#46046 and kubernetes/kubernetes#46049, which are needed for the emergency v1.6.4 release.

What is needed to push changes in these files to Jenkins?

@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label May 18, 2017
@enisoc enisoc force-pushed the three-part-version branch from 0a06971 to a8a7374 Compare May 18, 2017 20:00
@rmmh
Copy link
Contributor

rmmh commented May 18, 2017

/lgtm

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label May 18, 2017
@rmmh
Copy link
Contributor

rmmh commented May 18, 2017

I tested the regexes manually, merging now.

@rmmh rmmh merged commit 2815393 into kubernetes:master May 18, 2017
@enisoc enisoc deleted the three-part-version branch May 18, 2017 20:44
@enisoc
Copy link
Member Author

enisoc commented May 18, 2017

@rmmh Thanks! How do changes to these files take effect? It seems like the two PRs above still fail with the same error.

@enisoc
Copy link
Member Author

enisoc commented May 18, 2017

Oh actually they are passing. I must have been looking at the wrong gubernator links. Sorry!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. 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.

3 participants