diff --git a/app/grpc/module_handler.rb b/app/grpc/module_handler.rb index edcafacc..6ca2c8c0 100644 --- a/app/grpc/module_handler.rb +++ b/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) diff --git a/app/services/runtimes/grpc/modules/update_service.rb b/app/services/runtimes/grpc/modules/update_service.rb index 59344ac3..dd1e726a 100644 --- a/app/services/runtimes/grpc/modules/update_service.rb +++ b/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,