Return an error if no machine connected to switch port. (#244)

cmd/switch.go

@@ -172,7 +172,7 @@ Operational steps to replace a switch:
 	}
 
 	switchPortUpCmd := &cobra.Command{
-		Use:   "up <machine ID>",
+		Use:   "up <switch ID>",
 		Short: "sets the given switch port state up",
 		Long:  "sets the port status to UP so the connected machine will be able to connect to the switch.",
 		RunE: func(cmd *cobra.Command, args []string) error {
@@ -182,7 +182,7 @@ Operational steps to replace a switch:
 	}
 
 	switchPortDownCmd := &cobra.Command{
-		Use:   "down <machine ID>",
+		Use:   "down <switch ID>",
 		Short: "sets the given switch port state down",
 		Long:  "sets the port status to DOWN so the connected machine will not be able to connect to the switch.",
 		RunE: func(cmd *cobra.Command, args []string) error {
@@ -482,6 +482,10 @@ func (c *switchCmd) dumpPortState(rsp *models.V1SwitchResponse, portid string) e
 		}
 	}
 
+	if state.Actual.Nic == nil {
+		return fmt.Errorf("no machine connected to port %s on switch %s", portid, *rsp.ID)
+	}
+
 	return c.describePrinter.Print(state)
 }
 

docs/metalctl_switch_port_down.md

@@ -7,7 +7,7 @@ sets the given switch port state down
 sets the port status to DOWN so the connected machine will not be able to connect to the switch.
 
 ```
-metalctl switch port down <machine ID> [flags]
+metalctl switch port down <switch ID> [flags]
 ```
 
 ### Options

docs/metalctl_switch_port_up.md

@@ -7,7 +7,7 @@ sets the given switch port state up
 sets the port status to UP so the connected machine will be able to connect to the switch.
 
 ```
-metalctl switch port up <machine ID> [flags]
+metalctl switch port up <switch ID> [flags]
 ```
 
 ### Options