fixup! fix: Propagate wrapped handler response

jimmidyson · jimmidyson · commit 5a7725683040 · 2023-09-12T13:32:12.000+01:00
diff --git a/common/pkg/capi/clustertopology/handlers/mutation/meta.go b/common/pkg/capi/clustertopology/handlers/mutation/meta.go
@@ -40,8 +40,8 @@ func (mgp metaGeneratePatches) GeneratePatches(
 		if wrappedResp.Message != "" {
 			resp.Message = strings.TrimPrefix(resp.Message+"\n"+wrappedResp.Message, "\n")
 		}
-		if wrappedResp.Status != runtimehooksv1.ResponseStatusSuccess {
-			wrappedResp.Status = runtimehooksv1.ResponseStatusSuccess
+		resp.Status = wrappedResp.Status
+		if resp.Status == runtimehooksv1.ResponseStatusFailure {
 			return
 		}
 	}

Original file line number	Diff line number	Diff line change
`@@ -40,8 +40,8 @@ func (mgp metaGeneratePatches) GeneratePatches(`
`40`	`40`	`if wrappedResp.Message != "" {`
`41`	`41`	`resp.Message = strings.TrimPrefix(resp.Message+"\n"+wrappedResp.Message, "\n")`
`42`	`42`	`}`
`43`		`- if wrappedResp.Status != runtimehooksv1.ResponseStatusSuccess {`
`44`		`- wrappedResp.Status = runtimehooksv1.ResponseStatusSuccess`
	`43`	`+ resp.Status = wrappedResp.Status`
	`44`	`+ if resp.Status == runtimehooksv1.ResponseStatusFailure {`
`45`	`45`	`return`
`46`	`46`	`}`
`47`	`47`	`}`