From 1433f99619e87a11c411e85906d69e6ac194803e Mon Sep 17 00:00:00 2001 From: Alon Braymok <138359965+alonkeyval@users.noreply.github.com> Date: Mon, 16 Dec 2024 17:16:04 +0200 Subject: [PATCH] chore(cli): remove logs (#2014) This pull request includes several changes to the `cli/cmd/ui.go` file, primarily focused on cleaning up unnecessary debug output and improving the code's readability. Code cleanup: * Removed debug print statements that output the values of `legacyFlag` and `clusterPort`, which were used for debugging purposes but are no longer needed. --------- Co-authored-by: alonkeyval --- cli/cmd/ui.go | 3 +-- .../reuseable-components/data-card/data-card-fields/index.tsx | 4 ++-- 2 files changed, 3 insertions(+), 4 deletions(-) diff --git a/cli/cmd/ui.go b/cli/cmd/ui.go index 33a48ea37..917da40c7 100644 --- a/cli/cmd/ui.go +++ b/cli/cmd/ui.go @@ -57,8 +57,7 @@ var uiCmd = &cobra.Command{ if legacyFlag { clusterPort = legacyDefaultPort } - fmt.Printf("Is legacy: %v\n", legacyFlag) - fmt.Printf("Cluster port: %d\n", clusterPort) + localAddress := cmd.Flag("address").Value.String() uiPod, err := findOdigosUIPod(client, ctx, ns) if err != nil { diff --git a/frontend/webapp/reuseable-components/data-card/data-card-fields/index.tsx b/frontend/webapp/reuseable-components/data-card/data-card-fields/index.tsx index 764960676..7ecd37c7b 100644 --- a/frontend/webapp/reuseable-components/data-card/data-card-fields/index.tsx +++ b/frontend/webapp/reuseable-components/data-card/data-card-fields/index.tsx @@ -104,9 +104,9 @@ const renderValue = (type: DataCardRow['type'], value: DataCardRow['value']) => type={NOTIFICATION_TYPE.INFO} message={ hasPresenceOfOtherAgent - ? `By default, we do not operate alongside the ${otherAgent} Agent. Please contact the Odigos team for guidance on enabling this configuration.` + ? `By default, we do not operate alongside the ${otherAgent}. Please contact the Odigos team for guidance on enabling this configuration.` : canRunInParallel - ? `We are operating alongside the ${otherAgent} Agent, which is not the recommended configuration. We suggest disabling the ${otherAgent} Agent for optimal performance.` + ? `We are operating alongside the ${otherAgent}, which is not the recommended configuration. We suggest disabling the ${otherAgent} for optimal performance.` : `Concurrent execution with the ${otherAgent} is not supported. Please disable one of the agents to enable proper instrumentation.` } />