*string PrincipalID needs to be nil-guarded (#4258)

Azure · Nov 16, 2018 · 0449af5 · 0449af5
1 parent 1b07ace
commit 0449af5
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/pkg/operations/deletevm.go b/pkg/operations/deletevm.go
@@ -90,7 +90,7 @@ func CleanDeleteVirtualMachine(az armhelpers.ACSEngineClient, 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.