Merge branch 'update-geo-node-service' into 'master'

Use NodeUpdateService in update API

Closes #199112

See merge request gitlab-org/gitlab!23894

changelogs/unreleased/update-geo-node-service.yml

+---
+title: Use NodeUpdateService for updating Geo node
+merge_request: 23894
+author: Rajendra Kadam
+type: changed

ee/app/services/geo/node_update_service.rb

@@ -10,7 +10,7 @@ module Geo
       @old_shards        = geo_node.selective_sync_shards
 
       @params = params.dup
-      @params[:namespace_ids] = @params[:namespace_ids].to_s.split(',')
+      @params[:namespace_ids] = @params[:namespace_ids].to_s.split(',') if @params[:namespace_ids].is_a? String
     end
 
     def execute

ee/lib/api/geo_nodes.rb

@@ -193,7 +193,9 @@ module API
 
           update_params = declared_params(include_missing: false)
 
-          if geo_node.update(update_params)
+          updated_geo_node = ::Geo::NodeUpdateService.new(geo_node, update_params).execute
+
+          if updated_geo_node
             present geo_node, with: EE::API::Entities::GeoNode
           else
             render_validation_error!(geo_node)