Skip to content

UPSTREAM: 34434: Print valid json/yaml output in oc set image#11664

Merged
openshift-bot merged 2 commits intoopenshift:masterfrom
juanvallejo:jvallejo/fix-oc-set-image-output
Nov 1, 2016
Merged

UPSTREAM: 34434: Print valid json/yaml output in oc set image#11664
openshift-bot merged 2 commits intoopenshift:masterfrom
juanvallejo:jvallejo/fix-oc-set-image-output

Conversation

@juanvallejo
Copy link
Contributor

@juanvallejo juanvallejo commented Oct 31, 2016

Related UPSTREAM: kubernetes/kubernetes#34434
Related Bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=1390140

This patch updates oc set image to print the info running in local mode... through stderr in order to allow only json or yaml output when
using the --local flag.

This patch also prevents a potential panic here, by assigning a non-nil value to o.Err in set_image.go.

@openshift/cli-review

@juanvallejo juanvallejo changed the title UPSTREAM: 0000: Print valid json/yaml output in oc set image UPSTREAM: 34434: Print valid json/yaml output in oc set image Oct 31, 2016
@juanvallejo juanvallejo force-pushed the jvallejo/fix-oc-set-image-output branch from 28dfbc5 to 5ad9bcf Compare October 31, 2016 14:42
@juanvallejo
Copy link
Contributor Author

[test]

This patch updates `oc set image` to print the info `running in local
mode...` through stderr in order to allow only json or yaml output when
using the `--local` flag.
@juanvallejo juanvallejo force-pushed the jvallejo/fix-oc-set-image-output branch from 5ad9bcf to 429f299 Compare November 1, 2016 13:33
@openshift-bot
Copy link
Contributor

Evaluated for origin test up to 429f299

@openshift-bot
Copy link
Contributor

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

@fabianofranz
Copy link
Member

LGTM [merge]

@openshift-bot
Copy link
Contributor

openshift-bot commented Nov 1, 2016

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

@openshift-bot
Copy link
Contributor

Evaluated for origin merge up to 429f299

@openshift-bot openshift-bot merged commit 1c5401f into openshift:master Nov 1, 2016
@juanvallejo juanvallejo deleted the jvallejo/fix-oc-set-image-output branch November 1, 2016 22:48
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