Merge pull request #29 from packethost/creation-id

handle no ID on creation

Author: deitch

controllers/packetmachine_controller.go

@@ -24,6 +24,7 @@ import (
 
 	"github.com/go-logr/logr"
 	"github.com/google/uuid"
+	"github.com/packethost/packngo"
 	"github.com/pkg/errors"
 
 	apierrors "k8s.io/apimachinery/pkg/api/errors"
@@ -184,9 +185,17 @@ func (r *PacketMachineReconciler) reconcile(ctx context.Context, machineScope *s
 		return ctrl.Result{}, nil
 	}
 
-	dev, err := r.PacketClient.GetDevice(machineScope.GetInstanceID())
-	if err != nil {
-		return ctrl.Result{}, err
+	providerID := machineScope.GetInstanceID()
+	var (
+		dev *packngo.Device
+		err error
+	)
+	// if we have no provider ID, then we are creating
+	if providerID != "" {
+		dev, err = r.PacketClient.GetDevice(providerID)
+		if err != nil {
+			return ctrl.Result{}, err
+		}
 	}
 	if dev == nil {
 		// generate a unique UID that will survive pivot, i.e. is not tied to the cluster itself
@@ -210,7 +219,8 @@ func (r *PacketMachineReconciler) reconcile(ctx context.Context, machineScope *s
 		}
 	}
 
-	machineScope.SetProviderID(fmt.Sprintf("%s://%s", providerName, dev.ID))
+	// we do not need to set this as packet://<id> because SetProviderID() does the formatting for us
+	machineScope.SetProviderID(dev.ID)
 	machineScope.SetInstanceStatus(infrastructurev1alpha3.PacketResourceStatus(dev.State))
 
 	addrs, err := r.PacketClient.GetDeviceAddresses(dev)