Merge pull request #1290 from mohamed-shahid-afrid/sprnt-76-odim-dupl…

…icate Removing duplicate parameter and removing fmt statements
ODIM-Project · Aug 14, 2023 · 1c16be5 · 1c16be5
2 parents 55c76c0 + 3736c65
commit 1c16be5
Show file tree

Hide file tree

Showing 4 changed files with 2 additions and 7 deletions.
diff --git a/odim-controller/helmcharts/dellplugin/dellplugin/templates/configmaps.yaml b/odim-controller/helmcharts/dellplugin/dellplugin/templates/configmaps.yaml
@@ -8,8 +8,8 @@ data:
     {
     	"RootServiceUUID": {{ .Values.dellplugin.dellPluginRootServiceUUID | quote }},
     	"LogLevel": {{ .Values.dellplugin.logLevel | quote }},
-      "LogFormat": {{ .Values.dellplugin.logFormat | quote }},
-      "logsRedirectionToConsole" : {{ .Values.dellplugin.logsOnConsole | default false }},
+        "LogFormat": {{ .Values.dellplugin.logFormat | quote }},
+        "logsRedirectionToConsole" : {{ .Values.dellplugin.logsOnConsole | default false }},
     	"PluginConf": {
     		"ID": "DELL",
     		"Host": {{ .Values.dellplugin.host | quote }},
@@ -45,7 +45,6 @@ data:
     		"LBHost": {{ .Values.dellplugin.lbHost | quote }},
     		"LBPort": {{ .Values.dellplugin.lbPort | quote }}
     	},
-      "logsRedirectionToConsole" : {{ .Values.odimra.logsOnConsole | default false }},
     	"MessageBusConf": {
     		"MessageBusConfigFilePath": "/etc/dellplugin_config/platformconfig.toml",
     		"MessageBusType": "Kafka",

diff --git a/plugin-dell/dphandler/subscribeEvents.go b/plugin-dell/dphandler/subscribeEvents.go
@@ -177,7 +177,6 @@ func isOurSubscription(ctxt context.Context, device *dputilities.RedfishDevice)
 		l.LogWithFields(ctxt).Error(err.Error())
 		return false
 	}
-	fmt.Printf("************  %+v \n ", evtConfig.Data.LoadBalancerConf.Host+":"+evtConfig.Data.LoadBalancerConf.Port)
 	subscriptionDestinationFromDevice := subscriptionBody.(map[string]interface{})["Destination"].(string)
 	// if the subscription is ours then the destination should match with LBHOST:LBPORT.
 	//If it is not matching then retrun with MethodNotAllowed

diff --git a/svc-aggregation/rpc/aggregator.go b/svc-aggregation/rpc/aggregator.go
@@ -333,15 +333,13 @@ func (a *Aggregator) AddAggregationSource(ctx context.Context, req *aggregatorpr
 		generateResponse(authResp, resp)
 		return resp, nil
 	}
-	fmt.Println("AddAggregationSource********")
 	sessionUserName, err := a.connector.GetSessionUserName(ctx, req.SessionToken)
 	if err != nil {
 		errMsg := "Unable to get session username: " + err.Error()
 		generateResponse(common.GeneralError(http.StatusUnauthorized, response.NoValidSession, errMsg, nil, nil), resp)
 		l.LogWithFields(ctx).Error(errMsg)
 		return resp, nil
 	}
-	fmt.Println("session AddAggregationSource********", sessionUserName)
 	// parsing the AggregationSourceRequest
 	var addRequest system.AggregationSource
 	err = json.Unmarshal(req.RequestBody, &addRequest)

diff --git a/svc-systems/systems/getresource.go b/svc-systems/systems/getresource.go
@@ -362,7 +362,6 @@ func SearchAndFilter(ctx context.Context, paramStr []string, resp response.RPC)
 		for _, element := range submatchall {
 			element = strings.Trim(element, "(")
 			element = strings.Trim(element, ")")
-			fmt.Println(element)
 			if strings.Contains(element, " and ") || strings.Contains(element, " or ") {
 				var x []string
 				if strings.Contains(element, " and ") {