https://dev.arvados.org/https://dev.arvados.org/favicon.ico?15576888422022-03-23T18:54:29ZArvadosArvados - Feature #18881: Display container runtime_statushttps://dev.arvados.org/issues/18881?journal_id=1020642022-03-23T18:54:29ZPeter Amstutzpeter.amstutz@curii.com
<ul><li><strong>Related to</strong> <i><a class="issue tracker-6 status-3 priority-4 priority-default closed behind-schedule" href="/issues/16945">Idea #16945</a>: WB2 Workflows / containers feature parity</i> added</li></ul> Arvados - Feature #18881: Display container runtime_statushttps://dev.arvados.org/issues/18881?journal_id=1020652022-03-23T18:59:32ZPeter Amstutzpeter.amstutz@curii.com
<ul><li><strong>Description</strong> updated (<a title="View differences" href="/journals/102065/diff?detail_id=98662">diff</a>)</li></ul> Arvados - Feature #18881: Display container runtime_statushttps://dev.arvados.org/issues/18881?journal_id=1022062022-03-28T19:50:46ZLucas Di Pentimalucas.dipentima@curii.com
<ul><li><strong>Status</strong> changed from <i>New</i> to <i>In Progress</i></li></ul> Arvados - Feature #18881: Display container runtime_statushttps://dev.arvados.org/issues/18881?journal_id=1022982022-03-30T15:12:20ZPeter Amstutzpeter.amstutz@curii.com
<ul><li><strong>Target version</strong> changed from <i>2022-03-30 Sprint</i> to <i>2022-04-13 Sprint</i></li></ul> Arvados - Feature #18881: Display container runtime_statushttps://dev.arvados.org/issues/18881?journal_id=1026252022-04-08T15:47:37ZLucas Di Pentimalucas.dipentima@curii.com
<ul></ul><p>Updates at <a class="changeset" title="18881: Adds integration test for error & warning runtime status indicators. Arvados-DCO-1.1-Sign..." href="https://dev.arvados.org/projects/arvados/repository/arvados-workbench2/revisions/a5750c261be0991d8ebbe107115c9c5b01236f8d">arvados-workbench2|a5750c2</a> - branch <code>18881-process-runtime-status</code><br />Test run: <a class="external" href="https://ci.arvados.org/view/Developer/job/developer-tests-workbench2/649/"<a href="https://ci.arvados.org/view/Developer/job/developer-tests-workbench2/649/">developer-tests-workbench2: #649 <img src="https://ci.arvados.org/buildStatus/icon?job=developer-tests-workbench2&build=649" alt="" /></a></a></p>
<ul>
<li>Adds "Failing" and "Warning" as new process states that depend not only on the container's state but also the runtime status data.</li>
<li>Fixes status indications by conforming on how workbech1 renders them.</li>
<li>Adds runtime status indications to the process info panel.</li>
<li>Adds tests.</li>
</ul> Arvados - Feature #18881: Display container runtime_statushttps://dev.arvados.org/issues/18881?journal_id=1026402022-04-08T20:30:37ZLucas Di Pentimalucas.dipentima@curii.com
<ul></ul><p>Updates at <a class="changeset" title="18881: Adds timeouts to flaky tests to avoid false negatives. Arvados-DCO-1.1-Signed-off-by: Luc..." href="https://dev.arvados.org/projects/arvados/repository/arvados-workbench2/revisions/078bedef5794ee91497d4dd12157c86d6e490969">arvados-workbench2|078bedef</a> - <a class="external" href="https://ci.arvados.org/view/Developer/job/developer-tests-workbench2/652/"<a href="https://ci.arvados.org/view/Developer/job/developer-tests-workbench2/652/">developer-tests-workbench2: #652 <img src="https://ci.arvados.org/buildStatus/icon?job=developer-tests-workbench2&build=652" alt="" /></a></a></p>
<ul>
<li>Adds workarounds on unrelated flaky tests.</li>
</ul> Arvados - Feature #18881: Display container runtime_statushttps://dev.arvados.org/issues/18881?journal_id=1027192022-04-12T17:11:12ZWard Vandewegeward@curii.com
<ul></ul><p>Lucas Di Pentima wrote:</p>
<blockquote>
<p>Updates at <a class="changeset" title="18881: Adds timeouts to flaky tests to avoid false negatives. Arvados-DCO-1.1-Signed-off-by: Luc..." href="https://dev.arvados.org/projects/arvados/repository/arvados-workbench2/revisions/078bedef5794ee91497d4dd12157c86d6e490969">arvados-workbench2|078bedef</a> - <a class="external" href="https://ci.arvados.org/view/Developer/job/developer-tests-workbench2/652/"<a href="https://ci.arvados.org/view/Developer/job/developer-tests-workbench2/652/">developer-tests-workbench2: #652 <img src="https://ci.arvados.org/buildStatus/icon?job=developer-tests-workbench2&build=652" alt="" /></a></a></p>
<ul>
<li>Adds workarounds on unrelated flaky tests.</li>
</ul>
</blockquote>
<p>This looks great! My only comment is that it would be nice to use the same colorful button-style status fields in the process list view (e.g. "All Processes"), rather than the current colored text there, some of which can be hard to read (e.g. 'Queued' in light gray on a white background). I think it would also be nicer aesthetically.</p>
<p>LGTM otherwise.</p> Arvados - Feature #18881: Display container runtime_statushttps://dev.arvados.org/issues/18881?journal_id=1027232022-04-12T18:09:13ZLucas Di Pentimalucas.dipentima@curii.com
<ul></ul><p>Updates at <a class="changeset" title="18881: Make all process status widgets consistent. Updates tests. Arvados-DCO-1.1-Signed-off-by:..." href="https://dev.arvados.org/projects/arvados/repository/arvados-workbench2/revisions/18425d0dfa8a652fef8377a6fb09018f55d2beec">arvados-workbench2|18425d0d</a> - <a class="external" href="https://ci.arvados.org/view/Developer/job/developer-tests-workbench2/653/"<a href="https://ci.arvados.org/view/Developer/job/developer-tests-workbench2/653/">developer-tests-workbench2: #653 <img src="https://ci.arvados.org/buildStatus/icon?job=developer-tests-workbench2&build=653" alt="" /></a></a></p>
<ul>
<li>Makes the process status indicator to be consistent on listings with the style that's used on the process info panel.</li>
<li>Updates tests.</li>
</ul> Arvados - Feature #18881: Display container runtime_statushttps://dev.arvados.org/issues/18881?journal_id=1027252022-04-12T18:44:07ZWard Vandewegeward@curii.com
<ul><li><strong>File</strong> <a href="/attachments/2989">20220412-status-button-empty-projects.png</a> <a class="icon-only icon-download" title="Download" href="/attachments/download/2989/20220412-status-button-empty-projects.png">20220412-status-button-empty-projects.png</a> added</li></ul><p>Lucas Di Pentima wrote:</p>
<blockquote>
<p>Updates at <a class="changeset" title="18881: Make all process status widgets consistent. Updates tests. Arvados-DCO-1.1-Signed-off-by:..." href="https://dev.arvados.org/projects/arvados/repository/arvados-workbench2/revisions/18425d0dfa8a652fef8377a6fb09018f55d2beec">arvados-workbench2|18425d0d</a> - <a class="external" href="https://ci.arvados.org/view/Developer/job/developer-tests-workbench2/653/"<a href="https://ci.arvados.org/view/Developer/job/developer-tests-workbench2/653/">developer-tests-workbench2: #653 <img src="https://ci.arvados.org/buildStatus/icon?job=developer-tests-workbench2&build=653" alt="" /></a></a></p>
<ul>
<li>Makes the process status indicator to be consistent on listings with the style that's used on the process info panel.</li>
<li>Updates tests.</li>
</ul>
</blockquote>
<p>Loving the buttons in the data tables! But I guess something went wrong in the project listing, cf.</p>
<p><img src="https://dev.arvados.org/attachments/download/2989/20220412-status-button-empty-projects.png" alt="" /></p> Arvados - Feature #18881: Display container runtime_statushttps://dev.arvados.org/issues/18881?journal_id=1027272022-04-12T18:53:22ZLucas Di Pentimalucas.dipentima@curii.com
<ul></ul><p>Whoops! fixed at <a class="changeset" title="18881: Fixes the "non-status" indication on objects that aren't processes. Arvados-DCO-1.1-Signe..." href="https://dev.arvados.org/projects/arvados/repository/arvados-workbench2/revisions/03578e716a63e7c5e0ce75cb70d0aaa803c02a77">arvados-workbench2|03578e71</a> - <a class="external" href="https://ci.arvados.org/view/Developer/job/developer-tests-workbench2/654/"<a href="https://ci.arvados.org/view/Developer/job/developer-tests-workbench2/654/">developer-tests-workbench2: #654 <img src="https://ci.arvados.org/buildStatus/icon?job=developer-tests-workbench2&build=654" alt="" /></a></a></p> Arvados - Feature #18881: Display container runtime_statushttps://dev.arvados.org/issues/18881?journal_id=1027282022-04-12T18:55:05ZLucas Di Pentimalucas.dipentima@curii.com
<ul><li><strong>File</strong> <a href="/attachments/2990">Status indicator fix.png</a> <a class="icon-only icon-download" title="Download" href="/attachments/download/2990/Status%20indicator%20fix.png">Status indicator fix.png</a> added</li></ul> Arvados - Feature #18881: Display container runtime_statushttps://dev.arvados.org/issues/18881?journal_id=1027302022-04-12T19:28:29ZWard Vandewegeward@curii.com
<ul></ul><p>Lucas Di Pentima wrote:</p>
<blockquote>
<p>Whoops! fixed at <a class="changeset" title="18881: Fixes the "non-status" indication on objects that aren't processes. Arvados-DCO-1.1-Signe..." href="https://dev.arvados.org/projects/arvados/repository/arvados-workbench2/revisions/03578e716a63e7c5e0ce75cb70d0aaa803c02a77">arvados-workbench2|03578e71</a> - <a class="external" href="https://ci.arvados.org/view/Developer/job/developer-tests-workbench2/654/"<a href="https://ci.arvados.org/view/Developer/job/developer-tests-workbench2/654/">developer-tests-workbench2: #654 <img src="https://ci.arvados.org/buildStatus/icon?job=developer-tests-workbench2&build=654" alt="" /></a></a></p>
</blockquote>
<p>Perfect, thanks! LGTM.</p> Arvados - Feature #18881: Display container runtime_statushttps://dev.arvados.org/issues/18881?journal_id=1027402022-04-12T20:07:00ZLucas Di Pentimalucas.dipentima@curii.com
<ul><li><strong>% Done</strong> changed from <i>0</i> to <i>100</i></li><li><strong>Status</strong> changed from <i>In Progress</i> to <i>Resolved</i></li></ul><p>Applied in changeset <a class="changeset" title="Merge branch '18881-process-runtime-status'. Closes #18881 Arvados-DCO-1.1-Signed-off-by: Lucas ..." href="https://dev.arvados.org/projects/arvados-workbench-2/repository/arvados-workbench2/revisions/08ac60f877d6495a748747a3a0d30ca9f0e289d5">arvados-workbench-2:arvados-workbench2|08ac60f877d6495a748747a3a0d30ca9f0e289d5</a>.</p> Arvados - Feature #18881: Display container runtime_statushttps://dev.arvados.org/issues/18881?journal_id=1028672022-04-14T19:30:53ZLucas Di Pentimalucas.dipentima@curii.com
<ul></ul><p>Updates at <a class="changeset" title="18881: Fixes the Failing state's color, from red to orange. Arvados-DCO-1.1-Signed-off-by: Lucas..." href="https://dev.arvados.org/projects/arvados/repository/arvados-workbench2/revisions/fe2fcf6bb2af8938ea5d90d49d11b5b35971976d">arvados-workbench2|fe2fcf6b</a> - branch <code>18881-failing-state-color-fix</code><br />Test run: <a class="external" href="https://ci.arvados.org/view/Developer/job/developer-tests-workbench2/667/"<a href="https://ci.arvados.org/view/Developer/job/developer-tests-workbench2/667/">developer-tests-workbench2: #667 <img src="https://ci.arvados.org/buildStatus/icon?job=developer-tests-workbench2&build=667" alt="" /></a></a></p>
<ul>
<li>Fixes the "Failing" state's color, setting ip from red to orange.</li>
<li>Updates unit test.</li>
</ul> Arvados - Feature #18881: Display container runtime_statushttps://dev.arvados.org/issues/18881?journal_id=1039912022-05-27T16:01:31ZPeter Amstutzpeter.amstutz@curii.com
<ul><li><strong>Release</strong> set to <i>51</i></li></ul>