Merge pull request #2657 from sinnykumari/logging

openshift-merge-robot · web-flow · commit 95dcdb123f1a · 2021-07-02T21:45:05.000-04:00
daemon: add log and event for uncordoning node
diff --git a/pkg/daemon/daemon.go b/pkg/daemon/daemon.go
@@ -1292,7 +1292,8 @@ func (dn *Daemon) completeUpdate(desiredConfigName string) error {
 		return err
 	}
 
-	dn.logSystem("completed update for config %s", desiredConfigName)
+	dn.logSystem("Update completed for config %s and node has been successfully uncordoned", desiredConfigName)
+	dn.recorder.Eventf(getNodeRef(dn.node), corev1.EventTypeNormal, "Uncordon", fmt.Sprintf("Update completed for config %s and node has been uncordoned", desiredConfigName))
 
 	return nil
 }

Original file line number	Diff line number	Diff line change
`@@ -1292,7 +1292,8 @@ func (dn *Daemon) completeUpdate(desiredConfigName string) error {`
`1292`	`1292`	`return err`
`1293`	`1293`	`}`
`1294`	`1294`
`1295`		`- dn.logSystem("completed update for config %s", desiredConfigName)`
	`1295`	`+ dn.logSystem("Update completed for config %s and node has been successfully uncordoned", desiredConfigName)`
	`1296`	`+ dn.recorder.Eventf(getNodeRef(dn.node), corev1.EventTypeNormal, "Uncordon", fmt.Sprintf("Update completed for config %s and node has been uncordoned", desiredConfigName))`
`1296`	`1297`
`1297`	`1298`	`return nil`
`1298`	`1299`	`}`