Merge pull request #425 from tnierman/osd-29424

openshift-merge-bot[bot] · web-flow · commit e912bb8ee263 · 2025-04-15T08:18:01.000Z
OSD-29424 - Bugfix to use `pagerdutyTitlePrefix` instead of hardcoded string when updating incident title
diff --git a/cadctl/cmd/investigate/investigate.go b/cadctl/cmd/investigate/investigate.go
@@ -243,7 +243,7 @@ func updateIncidentTitle(pdClient *pagerduty.SdkClient) error {
 	if strings.Contains(currentTitle, pagerdutyTitlePrefix) {
 		return nil
 	}
-	newTitle := fmt.Sprintf("[CAD Investigated] %s", currentTitle)
+	newTitle := fmt.Sprintf("%s %s", pagerdutyTitlePrefix, currentTitle)
 	err := pdClient.UpdateIncidentTitle(newTitle)
 	if err != nil {
 		return fmt.Errorf("failed to update PagerDuty incident title: %w", err)

Original file line number	Diff line number	Diff line change
`@@ -243,7 +243,7 @@ func updateIncidentTitle(pdClient *pagerduty.SdkClient) error {`
`243`	`243`	`if strings.Contains(currentTitle, pagerdutyTitlePrefix) {`
`244`	`244`	`return nil`
`245`	`245`	`}`
`246`		`- newTitle := fmt.Sprintf("[CAD Investigated] %s", currentTitle)`
	`246`	`+ newTitle := fmt.Sprintf("%s %s", pagerdutyTitlePrefix, currentTitle)`
`247`	`247`	`err := pdClient.UpdateIncidentTitle(newTitle)`
`248`	`248`	`if err != nil {`
`249`	`249`	`return fmt.Errorf("failed to update PagerDuty incident title: %w", err)`