Skip to content

Update docs wrt multiple --env/--param arguments #12227

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

Merged
merged 1 commit into from
Dec 13, 2016

Conversation

mmilata
Copy link
Contributor

@mmilata mmilata commented Dec 12, 2016

The following syntax is not supported anymore:

--env K1=V1,K2=V2,...
--param K1=V1,K2=V2,...

Fixes #12076

@php-coder @bparees PTAL

The following syntax is not supported anymore:

  --env K1=V1,K2=V2,...
  --param K1=V1,K2=V2,...
@bparees
Copy link
Contributor

bparees commented Dec 12, 2016

lgtm [merge]

but i agree with @Kargakis, long term it seems like this content should be moved into the official docs and this should be updated w/ just a link to the docs.

@bparees bparees self-assigned this Dec 12, 2016
@php-coder
Copy link
Contributor

@mmilata Thanks!

@openshift-bot
Copy link
Contributor

[Test]ing while waiting on the merge queue

@openshift-bot
Copy link
Contributor

Evaluated for origin test up to a2b177f

@openshift-bot
Copy link
Contributor

continuous-integration/openshift-jenkins/test FAILURE (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/12272/) (Base Commit: 481d820)

@bparees
Copy link
Contributor

bparees commented Dec 13, 2016

flake #10773
[merge]

@bparees
Copy link
Contributor

bparees commented Dec 13, 2016

flake #10228
flake #11662
[merge]

@openshift-bot
Copy link
Contributor

Evaluated for origin merge up to a2b177f

@openshift-bot openshift-bot merged commit 595b6bd into openshift:master Dec 13, 2016
@openshift-bot
Copy link
Contributor

continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/12353/) (Base Commit: 3ec0a8d)

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