this is commit with separate column

fedora-copr · Feb 8, 2025 · e079ab3 · e079ab3
1 parent 4c09df1
commit e079ab3
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 2 deletions.
diff --git a/frontend/coprs_frontend/coprs/templates/_helpers.html b/frontend/coprs_frontend/coprs/templates/_helpers.html
@@ -143,7 +143,6 @@
     {% set url = "https://log-detective.com/explain?url=%s" %}
     <a href="{{ url | format(failed_log_url) }}"
       class="btn btn-default btn-xs"
-      style="margin-left: 4em; color: #333;"
       title="See what AI can tell you about the build failure">
       <span class="pficon pficon-info"></span> Explain fail in log with Log Detective AI
     </a>
@@ -169,7 +168,6 @@
     <span class="pficon pficon-ok"></span> {{ state }}
   {% elif state == "failed" %}
     <span class="pficon pficon-error-circle-o"></span> {{ state }}
-    {{ log_detective_explain_button(builder_live_log) }}
   {% elif state == "canceled" %}
     <span class="pficon pficon-close"></span> {{ state }}
   {% elif state == "waiting" %}

diff --git a/frontend/coprs_frontend/coprs/templates/coprs/detail/build.html b/frontend/coprs_frontend/coprs/templates/coprs/detail/build.html
@@ -261,6 +261,9 @@ <h3 class="panel-title">Results</h3>
               <th>Build Time</th>
               <th>Logs</th>
               <th>State</th>
+              {% if build.state == "failed" and build.source_state == "succeeded" %}
+              <th>Log Detective AI explanation</th>
+              {% endif %}
             </tr>
           </thead>
           <tbody>
@@ -302,6 +305,15 @@ <h3 class="panel-title">Results</h3>
               <td>
                 {{ build_state_text(chroot.state, chroot.status_reason, chroot.rpm_live_log_url) }}
               </td>
+              {% if build.state == "failed" and build.source_state == "succeeded" %}
+              <td>
+                {% if chroot.state == "failed" %}
+                  {{ log_detective_explain_button(chroot.rpm_live_log_url) }}
+                {% else %}
+                  -
+                {% endif %}
+              </td>
+              {% endif %}
             </tr>
           {% endfor %}
           </tbody>