Skip to content
This repository has been archived by the owner on Apr 17, 2019. It is now read-only.

[ingress] Update godeps #1767

Merged
merged 1 commit into from
Sep 22, 2016

Conversation

aledbf
Copy link
Contributor

@aledbf aledbf commented Sep 21, 2016

Against tag v1.4.0-beta.9


This change is Reviewable

@aledbf
Copy link
Contributor Author

aledbf commented Sep 21, 2016

@bprashanth ping

@bprashanth
Copy link

Great! thanks a lot

@bprashanth bprashanth added the lgtm Indicates that a PR is ready to be merged. label Sep 22, 2016
@k8s-github-robot
Copy link

Automatic merge from submit-queue

@k8s-github-robot k8s-github-robot merged commit 05a063d into kubernetes-retired:master Sep 22, 2016
@aledbf aledbf deleted the update-ingress branch September 27, 2016 02:56
@ichekrygin
Copy link

Any updates on this PR? See my comments at the bottom openshift/origin#10826 - this affects ngingx-ingress controller

@aledbf
Copy link
Contributor Author

aledbf commented Sep 27, 2016

@ichekrygin we are in the process to release a new version of the Ingress controller. The fix is already present in master.
This are the missing PRs to make the release https://github.com/kubernetes/contrib/pulls/aledbf

@bprashanth
Copy link

Yeah I'm going to try and push today, if a few of the outstanding prs get in

@ichekrygin
Copy link

@aledbf just to clarify, this fix is not in v1.4.0 release, correct?

@aledbf
Copy link
Contributor Author

aledbf commented Sep 27, 2016

just to clarify, this fix is not in v1.4.0 release, correct?

In particular kubernetes/kubernetes/pull/32125 was found during the v1.4 development cycle. This issue required the update of kubernetes/kubernetes go dependencies.

Just to clarify most of the kubernetes/contrib content do not belongs/is not tied to a particular kubernetes/kubernetes release.

@robszumski
Copy link

Has this new release gone out?

@aledbf
Copy link
Contributor Author

aledbf commented Jan 31, 2017

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
lgtm Indicates that a PR is ready to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants