fix: do not indicate timeout when context is canceled

When a user quits nctl while it is waiting for create/delete, we should
only show "timeout waiting for ..." if the context deadline is reached.
If the contact was simply canceled we should just exit without a special
message.

create/create.go

@@ -218,11 +218,17 @@ func (w *waitStage) watch(ctx context.Context, client *api.Client) error {
 				return nil
 			}
 		case <-ctx.Done():
-			msg := "timeout waiting for %s"
-			w.spinner.StopFailMessage(format.ProgressMessagef("", msg, w.kind))
-			_ = w.spinner.StopFail()
+			switch ctx.Err() {
+			case context.DeadlineExceeded:
+				msg := "timeout waiting for %s"
+				w.spinner.StopFailMessage(format.ProgressMessagef("", msg, w.kind))
+				_ = w.spinner.StopFail()
 
-			return fmt.Errorf(msg, w.kind)
+				return fmt.Errorf(msg, w.kind)
+			case context.Canceled:
+				_ = w.spinner.StopFail()
+				return nil
+			}
 		}
 	}
 }

delete/delete.go

@@ -147,11 +147,17 @@ func (d *deleter) waitForDeletion(ctx context.Context, client *api.Client) error
 				return fmt.Errorf("unable to get %s %q: %w", d.kind, d.mg.GetName(), err)
 			}
 		case <-ctx.Done():
-			msg := "timeout waiting for %s"
-			spinner.StopFailMessage(format.ProgressMessagef("", msg, d.kind))
-			_ = spinner.StopFail()
+			switch ctx.Err() {
+			case context.DeadlineExceeded:
+				msg := "timeout waiting for %s"
+				spinner.StopFailMessage(format.ProgressMessagef("", msg, d.kind))
+				_ = spinner.StopFail()
 
-			return fmt.Errorf("timeout waiting for %s", d.kind)
+				return fmt.Errorf(msg, d.kind)
+			case context.Canceled:
+				_ = spinner.StopFail()
+				return nil
+			}
 		}
 	}
 }