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

SPDY roundtripper + proxy fixes #7362

Merged
merged 3 commits into from Feb 19, 2016

Conversation

fabianofranz
Copy link
Member

@fabianofranz
Copy link
Member Author

FYI @liggitt

@liggitt liggitt self-assigned this Feb 16, 2016
@fabianofranz fabianofranz changed the title UPSTREAM: 21185: SPDY roundtripper must respect InsecureSkipVerify [WIP] UPSTREAM: 21185: SPDY roundtripper must respect InsecureSkipVerify Feb 17, 2016
@fabianofranz fabianofranz changed the title [WIP] UPSTREAM: 21185: SPDY roundtripper must respect InsecureSkipVerify UPSTREAM: 21185: SPDY roundtripper must respect InsecureSkipVerify Feb 17, 2016
@liggitt
Copy link
Contributor

liggitt commented Feb 17, 2016

LGTM, holding for rebase

@liggitt liggitt added the lgtm Indicates that a PR is ready to be merged. label Feb 17, 2016
@liggitt liggitt added this to the 1.2.0 milestone Feb 17, 2016
@fabianofranz fabianofranz changed the title UPSTREAM: 21185: SPDY roundtripper must respect InsecureSkipVerify SPDY roundtripper + proxy fixes Feb 17, 2016
@fabianofranz
Copy link
Member Author

@liggitt I added the other upstream commit (fixes proxy auth) to this PR.
[test]

@fabianofranz fabianofranz force-pushed the bugs_1302422 branch 4 times, most recently from e122796 to 50287ff Compare February 18, 2016 22:06
@openshift-bot
Copy link
Contributor

continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/1407/) (Image: devenv-rhel7_3484)

@liggitt
Copy link
Contributor

liggitt commented Feb 19, 2016

tests are failing consistently on the added unit test. might be related to the test server Close() call... not sure. untagging until we get a clean test run

@liggitt liggitt removed the lgtm Indicates that a PR is ready to be merged. label Feb 19, 2016
@openshift-bot
Copy link
Contributor

Evaluated for origin test up to f79894e

@liggitt
Copy link
Contributor

liggitt commented Feb 19, 2016

looking good:

ok      github.com/openshift/origin/Godeps/_workspace/src/k8s.io/kubernetes/pkg/util/httpstream/spdy    0.841s

retagging, [merge]

@liggitt liggitt added the lgtm Indicates that a PR is ready to be merged. label Feb 19, 2016
@fabianofranz
Copy link
Member Author

👍

@openshift-bot
Copy link
Contributor

continuous-integration/openshift-jenkins/test SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/1407/)

@openshift-bot
Copy link
Contributor

Evaluated for origin merge up to f79894e

openshift-bot pushed a commit that referenced this pull request Feb 19, 2016
@openshift-bot openshift-bot merged commit 8c1b94b into openshift:master Feb 19, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm Indicates that a PR is ready to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants