krel/announce: ask for confirmation before sending the email

The changes preserve the existing behavior when --nomock is not passed. Only when, --nomock is passed, a confirmation is seeked. Signed-off-by: Nabarun Pal <[email protected]>
kubernetes · Nov 15, 2021 · 7e4ed4a · 7e4ed4a
1 parent 365489a
commit 7e4ed4a
Showing 1 changed file with 14 additions and 2 deletions.
diff --git a/cmd/krel/cmd/announce_send.go b/cmd/krel/cmd/announce_send.go
@@ -169,8 +169,20 @@ func runAnnounce(opts *sendAnnounceOptions, announceRootOpts *announceOptions, r
 
 	logrus.Info("Sending mail")
 	subject := fmt.Sprintf("Kubernetes %s is live!", tag)
-	if err := m.Send(content, subject); err != nil {
-		return errors.Wrap(err, "unable to send mail")
+
+	yes := true
+
+	if rootOpts.nomock {
+		_, yes, err = util.Ask(fmt.Sprintf("%s (Y/n)", "Send email?"), "y:Y:yes|n:N:no|y", 10)
+		if err != nil {
+			return err
+		}
+	}
+
+	if yes {
+		if err := m.Send(content, subject); err != nil {
+			return errors.Wrap(err, "unable to send mail")
+		}
 	}
 
 	return nil