opendev/system-config
Code Issues Proposed changes

Zuul status: only display time on live changes

Browse Source 
Time has no meaning for non-live changes.  Really only the time
for the live change at the end of the list has any relevance.
The enqueue time is just repeated, and the ETA of 0 is just weird.
Don't display them for non-live changes.

Change-Id: Ibfcd4c997c1f2447d56ad12f7f7c0203c62d5bce
This commit is contained in:
James E. Blair 2015-02-11 18:21:04 -08:00
parent ab07b3fc0b
commit 7204ca363b
1 changed files with 10 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
modules/openstack_project/files/zuul/status.js
View File

@ -272,9 +272,12 @@ function format_change(change, change_queue) {
// Row #1 of the header (project and remaining time)
html += '<span class="project">' + change['project'] + '</span>';
html += '<span class="time" title="Remaining Time">';
html += format_time(change['remaining_time'], true);
html += '</span><br/>';
if (change.live === true) {
html += '<span class="time" title="Remaining Time">';
html += format_time(change['remaining_time'], true);
html += '</span>';
}
html += '<br/>';
// Row #2 of the header (change id and enqueue time)
html += '<span class="changeid"> ';
@ -297,9 +300,10 @@ function format_change(change, change_queue) {
html += '&nbsp;';
}
html += '</span>';
html += '<span class="time" title="Queued Time">';
html += format_enqueue_time(change['enqueue_time']) + '</span>';
if (change.live === true) {
html += '<span class="time" title="Queued Time">';
html += format_enqueue_time(change['enqueue_time']) + '</span>';
}
html += '</div>';
// Job listing from here down