Revert "update v4 metadata to return network stats despite missing st…

…ats for single container (aws#2719)" This reverts commit 7a82ee7.
fierlion · Nov 12, 2020 · 9b0aa80 · 9b0aa80
1 parent bacc344
commit 9b0aa80
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/agent/handlers/v4/task_metadata_handler.go b/agent/handlers/v4/task_metadata_handler.go
@@ -65,7 +65,12 @@ func TaskMetadataHandler(state dockerstate.TaskEngineState, ecsClient api.ECSCli
 			for _, containerResponse := range taskResponse.Containers {
 				networks, err := GetContainerNetworkMetadata(containerResponse.ID, state)
 				if err != nil {
-					seelog.Warnf("Error retrieving network metadata for container %s - %s", containerResponse.ID, err)
+					errResponseJSON, err := json.Marshal(err.Error())
+					if e := utils.WriteResponseIfMarshalError(w, err); e != nil {
+						return
+					}
+					utils.WriteJSONToResponse(w, http.StatusInternalServerError, errResponseJSON, utils.RequestTypeContainerMetadata)
+					return
 				}
 				containerResponse.Networks = networks
 				responses = append(responses, containerResponse)