From a6ce3dceb05d4841f9df249e330b963a38e7f25b Mon Sep 17 00:00:00 2001
From: Xavier-Do <xdo@odoo.com>
Date: Sun, 4 Dec 2022 14:40:44 +0100
Subject: [PATCH] [IMP] runbot: add step description

---
 runbot/models/build_config.py | 1 +
 runbot/models/event.py        | 1 +
 runbot/models/host.py         | 1 -
 runbot/templates/build.xml    | 2 +-
 runbot/views/config_views.xml | 1 +
 5 files changed, 4 insertions(+), 2 deletions(-)

diff --git a/runbot/models/build_config.py b/runbot/models/build_config.py
index 1f639397..c71ad46b 100644
--- a/runbot/models/build_config.py
+++ b/runbot/models/build_config.py
@@ -138,6 +138,7 @@ class ConfigStep(models.Model):
     # general info
     name = fields.Char('Step name', required=True, tracking=True, help="Unique name for step please use trigram as postfix for custom step_ids")
     domain_filter = fields.Char('Domain filter', tracking=True)
+    description = fields.Char('Config step description')
 
     job_type = fields.Selection(TYPES, default='install_odoo', required=True, tracking=True, ondelete={t[0]: 'cascade' for t in [TYPES]})
     protected = fields.Boolean('Protected', default=False, tracking=True)
diff --git a/runbot/models/event.py b/runbot/models/event.py
index 2e91a3c9..13f9e86d 100644
--- a/runbot/models/event.py
+++ b/runbot/models/event.py
@@ -35,6 +35,7 @@ class runbot_event(models.Model):
         for build in builds:
             build_logs = logs_by_build_id[build.id]
             for ir_log in build_logs:
+                ir_log['active_step_id'] = build.active_step.id
                 if ir_log['level'].upper() == 'WARNING':
                     build.triggered_result = 'warn'
                 elif ir_log['level'].upper() == 'ERROR':
diff --git a/runbot/models/host.py b/runbot/models/host.py
index af91b761..3cb759da 100644
--- a/runbot/models/host.py
+++ b/runbot/models/host.py
@@ -221,7 +221,6 @@ class Host(models.Model):
             build_logs = logs_by_build_id[build.id]
             for ir_log in build_logs:
                 local_log_ids.append(ir_log['id'])
-                ir_log['active_step_id'] = build.active_step.id
                 ir_log['type'] = 'server'
                 log_counter -= 1
                 if log_counter == 0:
diff --git a/runbot/templates/build.xml b/runbot/templates/build.xml
index b24e9547..0e8e353e 100644
--- a/runbot/templates/build.xml
+++ b/runbot/templates/build.xml
@@ -230,7 +230,7 @@
               <t t-foreach="build.sudo().log_ids" t-as="l">
                 <t t-set="subbuild" t-value="(([child for child in build.children_ids if child.id == int(l.path)] if l.type == 'subbuild' else False) or [build.browse()])[0]"/>
                 <t t-set="logclass" t-value="dict(CRITICAL='danger', ERROR='danger', WARNING='warning', OK='success', SEPARATOR='separator').get(l.level)"/>
-                <tr t-att-class="'separator' if logclass == 'separator' else ''">
+                <tr t-att-class="'separator' if logclass == 'separator' else ''" t-att-title="l.active_step_id.description or ''">
                   <td style="white-space: nowrap; width:1%;">
                     <t t-esc="l.create_date.strftime('%Y-%m-%d %H:%M:%S')"/>
                   </td>
diff --git a/runbot/views/config_views.xml b/runbot/views/config_views.xml
index f470fdb4..3c3df12d 100644
--- a/runbot/views/config_views.xml
+++ b/runbot/views/config_views.xml
@@ -40,6 +40,7 @@
                     </div>
                     <group string="General settings">
                         <field name="name"/>
+                        <field name="description"/>
                         <field name="domain_filter"/>
                         <field name="job_type"/>
                         <field name="make_stats"/>