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
Always give new-app output a common label #3888
Conversation
Review..... :) |
@@ -235,7 +235,7 @@ func setupAppConfig(f *clientcmd.Factory, c *cobra.Command, args []string, confi | |||
|
|||
func setLabels(labels map[string]string, result *newcmd.AppResult) error { | |||
if len(labels) == 0 { | |||
return nil | |||
labels = map[string]string{"app": result.Name} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should likely check when result.Name is empty... no pipelines got created because you were only instantiating templates.
Just one comment, other than that, looks good |
Only if no labels are specified - uses the first component name
2f66b7e
to
16f9790
Compare
[merge] |
continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pull_requests_origin/3779/) (Image: devenv-fedora_2050) |
Evaluated for origin up to 16f9790 |
[Test]ing while waiting on the merge queue |
continuous-integration/openshift-jenkins/test SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pull_requests_origin/3779/) |
Merged by openshift-bot
app= where name is --name or the first component's name.
@csrwng review please