Improve logging for failed module updates

Author: Taucher2003

app/grpc/module_handler.rb

@@ -11,9 +11,15 @@ def update(request, _call)
 
     logger.debug("ModuleHandler#update response: #{response.inspect}")
     unless response.success?
+      details = if response.payload[:details].is_a?(ActiveModel::Errors)
+                  { error_code: response.payload[:error_code], details: response.payload[:details].full_messages }
+                else
+                  response.payload
+                end
+
       logger.warn(message: 'Failed to update modules',
                   error: response.message,
-                  details: response.payload)
+                  details: details)
     end
 
     response.to_grpc_response(Tucana::Sagittarius::ModuleUpdateResponse)

app/services/runtimes/grpc/modules/update_service.rb

@@ -133,6 +133,10 @@ def update_module_definitions(module_records, t)
 
               next if db_module_definitions[index].save
 
+              logger.error(message: 'Failed to update runtime module definition',
+                           module_identifier: grpc_module.identifier,
+                           errors: db_module_definitions[index].errors.full_messages)
+
               t.rollback_and_return! ServiceResponse.error(
                 message: 'Failed to update runtime module definition',
                 error_code: :invalid_runtime_module_definition,