fix: incorporate review comments

Author: deesharma24

health-monitors/syslog-health-monitor/pkg/common/common.go

@@ -82,13 +82,13 @@ func MapActionStringToProto(s string) pb.RecommendedAction {
 	// RECOVER_FEATURE_RESET_GPU: power-smoothing feature recovery (XID 163);
 	// trigger guidance is "reload the driver or reset the GPU" — same proto
 	// target as the RESET_GPU bucket.
-	case "WORKFLOW_XID_48", "RESET_GPU", "RESET_FABRIC", "RECOVER_FEATURE_RESET_GPU":
+	case "WORKFLOW_XID_48", "RESET_GPU", "RECOVER_FEATURE_RESET_GPU":
 		return pb.RecommendedAction_COMPONENT_RESET
 	// WORKFLOW_XID_168: WPR/ECC config issue (XID 168). Catalog guidance is
 	// "either ECC should be enabled (to enable row-remapping) or boot
 	// re-attempted with shifted WPR" — reboot the VM so init can re-attempt
 	// with a shifted Write-Protect-Region.
-	case "WORKFLOW_XID_168":
+	case "WORKFLOW_XID_168", "RESET_FABRIC":
 		return pb.RecommendedAction_RESTART_VM
 	default:
 		slog.Warn("Unknown action string, defaulting to CONTACT_SUPPORT", "action", s)

health-monitors/syslog-health-monitor/pkg/common/common_test.go

@@ -101,10 +101,6 @@ func TestMapActionStringToProto(t *testing.T) {
 			input:          "RESET_GPU",
 			expectedOutput: pb.RecommendedAction_COMPONENT_RESET,
 		},
-		{
-			input:          "RESET_FABRIC",
-			expectedOutput: pb.RecommendedAction_COMPONENT_RESET,
-		},
 		{
 			input:          "RECOVER_FEATURE_RESET_GPU",
 			expectedOutput: pb.RecommendedAction_COMPONENT_RESET,
@@ -113,6 +109,10 @@ func TestMapActionStringToProto(t *testing.T) {
 			input:          "WORKFLOW_XID_168",
 			expectedOutput: pb.RecommendedAction_RESTART_VM,
 		},
+		{
+			input:          "RESET_FABRIC",
+			expectedOutput: pb.RecommendedAction_RESTART_VM,
+		},
 		{
 			input:          "NONE",
 			expectedOutput: pb.RecommendedAction_NONE,