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

Clarify how we handle compression as a router env #11485

Merged
merged 1 commit into from
Nov 4, 2016

Conversation

knobunc
Copy link
Contributor

@knobunc knobunc commented Oct 21, 2016

This just cleans up how we handle compression support in the haproxy
config to make it more like the existing code. There is no functional
change.

The original code just went in as #11469

@knobunc
Copy link
Contributor Author

knobunc commented Oct 21, 2016

@JacobTanenbaum @ramr PTAL

@JacobTanenbaum
Copy link
Contributor

LGTM

@knobunc
Copy link
Contributor Author

knobunc commented Oct 25, 2016

[test]

This just cleans up how we handle compression support in the haproxy
config to make it more like the existing code.  There is no functional
change.
@knobunc knobunc force-pushed the fix/clean-up-compression branch from 318833b to b078a7a Compare October 25, 2016 17:52
@knobunc
Copy link
Contributor Author

knobunc commented Oct 25, 2016

Flake #11560 [test]

@knobunc
Copy link
Contributor Author

knobunc commented Oct 28, 2016

@ramr, @rajatchopra PTAL

@ramr
Copy link
Contributor

ramr commented Nov 3, 2016

LGTM

Copy link
Contributor

@ramr ramr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@ramr
Copy link
Contributor

ramr commented Nov 3, 2016

[test]

@openshift-bot
Copy link
Contributor

Evaluated for origin test up to b078a7a

@openshift-bot
Copy link
Contributor

continuous-integration/openshift-jenkins/test FAILURE (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/11103/) (Base Commit: 54f5f92)

@knobunc
Copy link
Contributor Author

knobunc commented Nov 4, 2016

[merge] Flaked on #10773

@knobunc
Copy link
Contributor Author

knobunc commented Nov 4, 2016

[merge] flaked on #10773

@openshift-bot
Copy link
Contributor

Evaluated for origin merge up to b078a7a

@openshift-bot
Copy link
Contributor

openshift-bot commented Nov 4, 2016

continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/11148/) (Base Commit: f7f9c83) (Image: devenv-rhel7_5319)

@openshift-bot openshift-bot merged commit da2c207 into openshift:master Nov 4, 2016
@knobunc knobunc deleted the fix/clean-up-compression branch June 7, 2018 12:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants