Skip to content

Fixes get --show-labels for OpenShift resources#9152

Merged
openshift-bot merged 2 commits intoopenshift:masterfrom
fabianofranz:issues_9114
Jun 6, 2016
Merged

Fixes get --show-labels for OpenShift resources#9152
openshift-bot merged 2 commits intoopenshift:masterfrom
fabianofranz:issues_9114

Conversation

@fabianofranz
Copy link
Member

@fabianofranz fabianofranz commented Jun 2, 2016

Fixes #9114

WIP fixes deployment configs, still need to check and fix other OpenShift resources if needed. Also pending tests.

@smarterclayton are you ok with exposing append*Labels funcs in the printer upstream?

@smarterclayton
Copy link
Contributor

Yes, it should be exposed upstream.

@fabianofranz fabianofranz force-pushed the issues_9114 branch 3 times, most recently from ba45b01 to 2ab6494 Compare June 3, 2016 18:27
@fabianofranz fabianofranz changed the title [WIP] Fixes get --show-labels for OpenShift resources Fixes get --show-labels for OpenShift resources Jun 3, 2016
@fabianofranz
Copy link
Member Author

[test]

@fabianofranz
Copy link
Member Author

@smarterclayton please review.

@fabianofranz fabianofranz force-pushed the issues_9114 branch 2 times, most recently from e9409a2 to 58828b1 Compare June 3, 2016 20:51
@openshift-bot
Copy link
Contributor

Evaluated for origin test up to b691eeb

@openshift-bot
Copy link
Contributor

continuous-integration/openshift-jenkins/test SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/4475/)

@smarterclayton
Copy link
Contributor

LGTM [merge]

@openshift-bot
Copy link
Contributor

openshift-bot commented Jun 6, 2016

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

@openshift-bot
Copy link
Contributor

Evaluated for origin merge up to b691eeb

@openshift-bot openshift-bot merged commit 9789e45 into openshift:master Jun 6, 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.

oc: show-labels doesn't for DC

3 participants