*string needs nil guard (#28)

Azure · Nov 16, 2018 · 54664d8 · 54664d8
1 parent dd06fd5
commit 54664d8
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/pkg/operations/deletevm.go b/pkg/operations/deletevm.go
@@ -93,7 +93,7 @@ func CleanDeleteVirtualMachine(az armhelpers.AKSEngineClient, logger *log.Entry,
 		}
 	}
 
-	if vm.Identity != nil {
+	if vm.Identity != nil && vm.Identity.PrincipalID != nil {
 		// Role assignments are not deleted if the VM is destroyed, so we must cleanup ourselves!
 		// The role assignments should only be relevant if managed identities are used,
 		// but always cleaning them up is easier than adding rule based logic here and there.