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

Add compression to haproxy v2 #11469

Merged
merged 1 commit into from Oct 20, 2016
Merged

Add compression to haproxy v2 #11469

merged 1 commit into from Oct 20, 2016

Conversation

ghost
Copy link

@ghost ghost commented Oct 20, 2016

The original discussion was in pr #11272 .
This pr exists because github interface have no git stash possibility.

The original discussion was in pr  #11272 .
This pr exists because github interface have no **git stash** possibility.
@ghost ghost mentioned this pull request Oct 20, 2016
Copy link
Contributor

@knobunc knobunc left a comment

Choose a reason for hiding this comment

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

LGTM

@knobunc
Copy link
Contributor

knobunc commented Oct 20, 2016

[merge] based on @ramr's review on #11272

@openshift-bot
Copy link
Contributor

openshift-bot commented Oct 20, 2016

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

@openshift-bot
Copy link
Contributor

Evaluated for origin merge up to b484959

@openshift-bot
Copy link
Contributor

[Test]ing while waiting on the merge queue

@openshift-bot
Copy link
Contributor

Evaluated for origin test up to b484959

@openshift-bot
Copy link
Contributor

continuous-integration/openshift-jenkins/test SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/10338/) (Base Commit: 1ec25ef)

@openshift-bot openshift-bot merged commit e1d9196 into openshift:master Oct 20, 2016
@smarterclayton
Copy link
Contributor

@knobunc can we make sure to document and maintain a document of all env vars when we add them to the router template? Needs to be obvious which are supported where and what they do.

@knobunc
Copy link
Contributor

knobunc commented Oct 21, 2016

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.

3 participants