https://dev.arvados.org/https://dev.arvados.org/favicon.ico?15576888422014-09-26T14:44:44ZArvadosArvados - Bug #4004: Restore "All jobs/pipelines/collections" buttons to dashboardhttps://dev.arvados.org/issues/4004?journal_id=155032014-09-26T14:44:44ZPeter Amstutzpeter.amstutz@curii.com
<ul><li><strong>Assigned To</strong> set to <i>Peter Amstutz</i></li></ul> Arvados - Bug #4004: Restore "All jobs/pipelines/collections" buttons to dashboardhttps://dev.arvados.org/issues/4004?journal_id=155802014-09-26T20:43:54ZTom Cleggtom@curii.com
<ul><li><strong>Story points</strong> set to <i>0.5</i></li></ul> Arvados - Bug #4004: Restore "All jobs/pipelines/collections" buttons to dashboardhttps://dev.arvados.org/issues/4004?journal_id=156742014-09-30T14:08:46ZRadhika Chippadaradhika@curoverse.com
<ul></ul><p>Review comments:</p>
<ul>
<li>These new buttons are quite helpful.</li>
</ul>
<ul>
<li>If felt a little weird to see "All jobs" button in the panel with header "Compute status". It feels like it should point to "Compute nodes" page. May be we might want to improve the panel title to "Compute and job status" or something like that?</li>
</ul>
<ul>
<li>This update can absolutely use an integration test. A test that executes the following steps:
<ul>
<li>visit dashboard</li>
<li>click on "Run a pipeline..." button and verify that able to create a pipeline or at least initiate creation of pipeline</li>
<li>click on each of the 3 "All xxx..." buttons and verify that we end up in that respective page and look for one of the items listed in the fixtures</li>
</ul></li>
</ul>
<ul>
<li>I also noticed from the workbench console log the following: It seems that the dashboard is being reloaded every few (15?) seconds. In the console log, I saw multiple requests being made to the API server (see blow). I am concerned about performance impact when we have several users using the system? Do you think you can get all data (at least for each type) in one request?</li>
</ul>
<p>API client: 0.000447701 Parse response<br /> Rendered application/_pipeline_status_label.html.erb (0.2ms)<br />API client: 0.000512425 Prepare request <a class="external" href="https://localhost:3030/arvados/v1/pipeline_templates">https://localhost:3030/arvados/v1/pipeline_templates</a> {"uuid":"<a href="https://arvadosapi.com/zzzzz-p5p6p-rxj8d71854j9idn">zzzzz-p5p6p-rxj8d71854j9idn</a>"} <br />API client: 0.050315221 API transaction<br />API client: 0.041700684 request_time<br />API client: 0.000296891 Parse response<br /> Rendered application/_pipeline_status_label.html.erb (0.1ms)<br />API client: 0.000244403 Prepare request <a class="external" href="https://localhost:3030/arvados/v1/pipeline_templates">https://localhost:3030/arvados/v1/pipeline_templates</a> {"uuid":"<a href="https://arvadosapi.com/zzzzz-p5p6p-rxj8d71854j9idn">zzzzz-p5p6p-rxj8d71854j9idn</a>"} <br />API client: 0.054005062 API transaction<br />API client: 0.046160175 request_time<br />API client: 0.00074173 Parse response<br /> Rendered application/_pipeline_status_label.html.erb (0.2ms)<br />API client: 0.000304222 Prepare request <a class="external" href="https://localhost:3030/arvados/v1/pipeline_templates">https://localhost:3030/arvados/v1/pipeline_templates</a> {"uuid":"<a href="https://arvadosapi.com/zzzzz-p5p6p-rxj8d71854j9idn">zzzzz-p5p6p-rxj8d71854j9idn</a>"} <br />. . .</p> Arvados - Bug #4004: Restore "All jobs/pipelines/collections" buttons to dashboardhttps://dev.arvados.org/issues/4004?journal_id=157022014-09-30T19:42:27ZPeter Amstutzpeter.amstutz@curii.com
<ul></ul><p>Radhika Chippada wrote:</p>
<blockquote>
<p>Review comments:</p>
<ul>
<li>These new buttons are quite helpful.</li>
</ul>
<ul>
<li>If felt a little weird to see "All jobs" button in the panel with header "Compute status". It feels like it should point to "Compute nodes" page. May be we might want to improve the panel title to "Compute and job status" or something like that?</li>
</ul>
</blockquote>
<p>Fixed.</p>
<blockquote>
<ul>
<li>This update can absolutely use an integration test. A test that executes the following steps:
<ul>
<li>visit dashboard</li>
<li>click on "Run a pipeline..." button and verify that able to create a pipeline or at least initiate creation of pipeline</li>
<li>click on each of the 3 "All xxx..." buttons and verify that we end up in that respective page and look for one of the items listed in the fixtures</li>
</ul></li>
</ul>
</blockquote>
<p>Done.</p>
<blockquote>
<ul>
<li>I also noticed from the workbench console log the following: It seems that the dashboard is being reloaded every few (15?) seconds. In the console log, I saw multiple requests being made to the API server (see blow). I am concerned about performance impact when we have several users using the system? Do you think you can get all data (at least for each type) in one request?</li>
</ul>
</blockquote>
<p>Batched up requests for pipeline templates and disabled requests for collection tags. Improves dashboard rendering time by at least 25%</p> Arvados - Bug #4004: Restore "All jobs/pipelines/collections" buttons to dashboardhttps://dev.arvados.org/issues/4004?journal_id=157082014-09-30T20:09:15ZRadhika Chippadaradhika@curoverse.com
<ul></ul><p>Peter: thanks for update. It is much better now.</p>
<p>Sorry for being a strickler, but I just have one request about the test 'Run a pipeline from dashboard' in "apps/workbench/test/integration/pipeline_instances_test.rb". Please move the code that does "#create a pipeline instance" etc into a reusable method in this test class and reuse from both the tests. Thanks.</p>
<p>LGTM otherwise.</p> Arvados - Bug #4004: Restore "All jobs/pipelines/collections" buttons to dashboardhttps://dev.arvados.org/issues/4004?journal_id=157362014-10-01T14:40:09ZAnonymous
<ul><li><strong>Status</strong> changed from <i>In Progress</i> to <i>Resolved</i></li></ul><p>Applied in changeset arvados|commit:74cdb4454d4adc6b403c207169313f37332d8aac.</p>