From fc7794469013b44f15d26ceade5130b04c858e35 Mon Sep 17 00:00:00 2001 From: Alena Varkockova Date: Fri, 11 Oct 2019 10:43:28 +0200 Subject: [PATCH 1/2] More consistent logging for install --- pkg/kudoctl/cmd/install/install.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/pkg/kudoctl/cmd/install/install.go b/pkg/kudoctl/cmd/install/install.go index 6b95e2ac2..898c93327 100644 --- a/pkg/kudoctl/cmd/install/install.go +++ b/pkg/kudoctl/cmd/install/install.go @@ -225,7 +225,7 @@ func installSingleOperatorToCluster(name, namespace string, o *v1alpha1.Operator if _, err := kc.InstallOperatorObjToCluster(o, namespace); err != nil { return errors.Wrapf(err, "installing %s-operator.yaml", name) } - clog.V(2).Printf("operator.%s/%s created\n", o.APIVersion, o.Name) + clog.Printf("operator.%s/%s created\n", o.APIVersion, o.Name) return nil } @@ -235,7 +235,7 @@ func installSingleOperatorVersionToCluster(name, namespace string, kc *kudo.Clie if _, err := kc.InstallOperatorVersionObjToCluster(ov, namespace); err != nil { return errors.Wrapf(err, "installing %s-operatorversion.yaml", name) } - clog.V(2).Printf("operatorversion.%s/%s created\n", ov.APIVersion, ov.Name) + clog.Printf("operatorversion.%s/%s created\n", ov.APIVersion, ov.Name) return nil } From 362d6f21e0e31694e76a340c700fe750846369f5 Mon Sep 17 00:00:00 2001 From: Alena Varkockova Date: Fri, 11 Oct 2019 12:35:00 +0200 Subject: [PATCH 2/2] Remove newlines --- pkg/kudoctl/cmd/install/install.go | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/pkg/kudoctl/cmd/install/install.go b/pkg/kudoctl/cmd/install/install.go index 898c93327..518745621 100644 --- a/pkg/kudoctl/cmd/install/install.go +++ b/pkg/kudoctl/cmd/install/install.go @@ -225,7 +225,7 @@ func installSingleOperatorToCluster(name, namespace string, o *v1alpha1.Operator if _, err := kc.InstallOperatorObjToCluster(o, namespace); err != nil { return errors.Wrapf(err, "installing %s-operator.yaml", name) } - clog.Printf("operator.%s/%s created\n", o.APIVersion, o.Name) + clog.Printf("operator.%s/%s created", o.APIVersion, o.Name) return nil } @@ -235,7 +235,7 @@ func installSingleOperatorVersionToCluster(name, namespace string, kc *kudo.Clie if _, err := kc.InstallOperatorVersionObjToCluster(ov, namespace); err != nil { return errors.Wrapf(err, "installing %s-operatorversion.yaml", name) } - clog.Printf("operatorversion.%s/%s created\n", ov.APIVersion, ov.Name) + clog.Printf("operatorversion.%s/%s created", ov.APIVersion, ov.Name) return nil } @@ -245,7 +245,7 @@ func installSingleInstanceToCluster(name string, instance *v1alpha1.Instance, kc if _, err := kc.InstallInstanceObjToCluster(instance, settings.Namespace); err != nil { return errors.Wrapf(err, "installing instance %s", name) } - clog.Printf("instance.%s/%s created\n", instance.APIVersion, instance.Name) + clog.Printf("instance.%s/%s created", instance.APIVersion, instance.Name) return nil }