From 7e2d7c65a114373e3bce7813a0d41ad868b7b18f Mon Sep 17 00:00:00 2001
From: Xavier-Do <xdo@odoo.com>
Date: Mon, 19 Oct 2020 01:30:39 +0200
Subject: [PATCH] [FIX] runbot: fix message formating

---
 runbot/models/batch.py | 4 +---
 runbot/models/build.py | 4 ++++
 2 files changed, 5 insertions(+), 3 deletions(-)

diff --git a/runbot/models/batch.py b/runbot/models/batch.py
index 01810fd8..aa1d0a49 100644
--- a/runbot/models/batch.py
+++ b/runbot/models/batch.py
@@ -358,11 +358,9 @@ class Batch(models.Model):
 
     def _log(self, message, *args, level='INFO'):
         message = message % args if args else message
-        if len(message) > 300000:
-            message = message[:300000] + '[Truncate, message too long]'
         self.env['runbot.batch.log'].create({
             'batch_id': self.id,
-            'message': message % args if args else message,
+            'message': message,
             'level': level,
         })
 
diff --git a/runbot/models/build.py b/runbot/models/build.py
index ed67ea2d..e53f7fe9 100644
--- a/runbot/models/build.py
+++ b/runbot/models/build.py
@@ -814,6 +814,10 @@ class BuildResult(models.Model):
         self.env['runbot.database'].create({'name': dbname, 'build_id': self.id})
 
     def _log(self, func, message, level='INFO', log_type='runbot', path='runbot'):
+
+        if len(message) > 300000:
+            message = message[:300000] + '[Truncate, message too long]'
+
         self.ensure_one()
         _logger.debug("Build %s %s %s", self.id, func, message)
         self.env['ir.logging'].create({