microcloud/cmd/microcloud: Don't fully overwrite the default profile

Rather than overwriting the default profile when adding devices, this
appends the devices and config to the default profile. When adding a new
service to microcloud that was skipped during initial setup, we may have
to add a new device to the default profile but we shouldn't remove
anything we added during first init.

Signed-off-by: Max Asnaashari <[email protected]>

microcloud/cmd/microcloud/main_init.go

@@ -772,12 +772,34 @@ func setupCluster(s *service.Handler, systems map[string]InitSystem) error {
 
 		if !shared.ValueInSlice(profile.Name, profiles) {
 			err = lxdClient.CreateProfile(profile)
+			if err != nil {
+				return err
+			}
 		} else {
-			err = lxdClient.UpdateProfile(profile.Name, profile.ProfilePut, "")
-		}
+			// Ensure any pre-existing devices and config are carried over to the new profile, unless we are managing them.
+			existingProfile, _, err := lxdClient.GetProfile("default")
+			if err != nil {
+				return err
+			}
 
-		if err != nil {
-			return err
+			for k, v := range existingProfile.Config {
+				_, ok := profile.Config[k]
+				if !ok {
+					profile.Config[k] = v
+				}
+			}
+
+			for k, v := range existingProfile.Devices {
+				_, ok := profile.Devices[k]
+				if !ok {
+					profile.Devices[k] = v
+				}
+			}
+
+			err = lxdClient.UpdateProfile(profile.Name, profile.ProfilePut, "")
+			if err != nil {
+				return err
+			}
 		}
 	}