Add example env vars

[rhcarvalho]

Add examples of how to set environment variables for S2I and Docker strategy builds.

[rhcarvalho]

@mfojtik @bparees PTAL

[bparees]

lgtm. should merge after #4562

[rhcarvalho]

It won't break anything if merged before... the env is already part of
dockerStrategy, just not used.
On Sep 19, 2015 9:31 PM, "Ben Parees" [email protected] wrote:

lgtm. should merge after #4562
#4562

—
Reply to this email directly or view it on GitHub
#4735 (comment).

[bparees]

I know but I don't want to confuse people who see it and assume that means
it can be used.

Ben Parees | OpenShift
On Sep 20, 2015 03:36, "Rodolfo Carvalho" [email protected] wrote:

It won't break anything if merged before... the env is already part of
dockerStrategy, just not used.
On Sep 19, 2015 9:31 PM, "Ben Parees" [email protected] wrote:

lgtm. should merge after #4562
#4562

—
Reply to this email directly or view it on GitHub
#4735 (comment).

—
Reply to this email directly or view it on GitHub
#4735 (comment).

[bparees]

[merge]

[openshift-bot]

continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/merge_pull_requests_origin/3358/) (Image: devenv-fedora_2375)

[openshift-bot]

Evaluated for origin merge up to 37fb5a4

[openshift-bot]

[Test]ing while waiting on the merge queue

[openshift-bot]

continuous-integration/openshift-jenkins/test FAILURE (https://ci.openshift.redhat.com/jenkins/job/test_pull_requests_origin/5071/)

[openshift-bot]

Evaluated for origin test up to 37fb5a4