Log result. (zalando#1387)

Jan-M · web-flow · commit e837751ae0c5 · 2021-02-26T14:54:26.000+01:00
diff --git a/pkg/cluster/majorversionupgrade.go b/pkg/cluster/majorversionupgrade.go
@@ -86,10 +86,12 @@ func (c *Cluster) majorVersionUpgrade() error {
 			podName := &spec.NamespacedName{Namespace: masterPod.Namespace, Name: masterPod.Name}
 			c.logger.Infof("triggering major version upgrade on pod %s of %d pods", masterPod.Name, numberOfPods)
 			upgradeCommand := fmt.Sprintf("/usr/bin/python3 /scripts/inplace_upgrade.py %d 2>&1 | tee last_upgrade.log", numberOfPods)
-			_, err := c.ExecCommand(podName, "/bin/su", "postgres", "-c", upgradeCommand)
+			result, err := c.ExecCommand(podName, "/bin/su", "postgres", "-c", upgradeCommand)
 			if err != nil {
 				return err
 			}
+
+			c.logger.Infof("upgrade action triggered and command completed: %s", result[:50])
 		}
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -86,10 +86,12 @@ func (c *Cluster) majorVersionUpgrade() error {`
`86`	`86`	`podName := &spec.NamespacedName{Namespace: masterPod.Namespace, Name: masterPod.Name}`
`87`	`87`	`c.logger.Infof("triggering major version upgrade on pod %s of %d pods", masterPod.Name, numberOfPods)`
`88`	`88`	`upgradeCommand := fmt.Sprintf("/usr/bin/python3 /scripts/inplace_upgrade.py %d 2>&1 \| tee last_upgrade.log", numberOfPods)`
`89`		`- _, err := c.ExecCommand(podName, "/bin/su", "postgres", "-c", upgradeCommand)`
	`89`	`+ result, err := c.ExecCommand(podName, "/bin/su", "postgres", "-c", upgradeCommand)`
`90`	`90`	`if err != nil {`
`91`	`91`	`return err`
`92`	`92`	`}`
	`93`	`+`
	`94`	`+ c.logger.Infof("upgrade action triggered and command completed: %s", result[:50])`
`93`	`95`	`}`
`94`	`96`	`}`
`95`	`97`