https://dev.arvados.org/https://dev.arvados.org/favicon.ico?15576888422017-11-29T19:44:01ZArvadosArvados - Idea #12702: Migrate user accountshttps://dev.arvados.org/issues/12702?journal_id=575432017-11-29T19:44:01ZPeter Amstutzpeter.amstutz@curii.com
<ul><li><strong>Description</strong> updated (<a title="View differences" href="/journals/57543/diff?detail_id=55030">diff</a>)</li></ul> Arvados - Idea #12702: Migrate user accountshttps://dev.arvados.org/issues/12702?journal_id=575442017-11-29T19:48:27ZPeter Amstutzpeter.amstutz@curii.com
<ul><li><strong>Related to</strong> <i><a class="issue tracker-2 status-3 priority-4 priority-default closed parent" href="/issues/12626">Feature #12626</a>: [API] Merge user accounts (redirect=true case)</i> added</li></ul> Arvados - Idea #12702: Migrate user accountshttps://dev.arvados.org/issues/12702?journal_id=575882017-11-29T21:22:12ZPeter Amstutzpeter.amstutz@curii.com
<ul><li><strong>Description</strong> updated (<a title="View differences" href="/journals/57588/diff?detail_id=55080">diff</a>)</li></ul> Arvados - Idea #12702: Migrate user accountshttps://dev.arvados.org/issues/12702?journal_id=575892017-11-29T21:23:13ZPeter Amstutzpeter.amstutz@curii.com
<ul><li><strong>Description</strong> updated (<a title="View differences" href="/journals/57589/diff?detail_id=55081">diff</a>)</li></ul> Arvados - Idea #12702: Migrate user accountshttps://dev.arvados.org/issues/12702?journal_id=575902017-11-29T21:23:37ZPeter Amstutzpeter.amstutz@curii.com
<ul><li><strong>Story points</strong> set to <i>2.0</i></li></ul> Arvados - Idea #12702: Migrate user accountshttps://dev.arvados.org/issues/12702?journal_id=581652017-12-06T20:24:01ZTom Morristfmorris@veritasgenetics.com
<ul><li><strong>Target version</strong> changed from <i>To Be Groomed</i> to <i>2017-12-20 Sprint</i></li></ul> Arvados - Idea #12702: Migrate user accountshttps://dev.arvados.org/issues/12702?journal_id=581732017-12-06T20:32:14ZPeter Amstutzpeter.amstutz@curii.com
<ul><li><strong>Assigned To</strong> set to <i>Peter Amstutz</i></li></ul> Arvados - Idea #12702: Migrate user accountshttps://dev.arvados.org/issues/12702?journal_id=587392017-12-21T21:04:54ZPeter Amstutzpeter.amstutz@curii.com
<ul><li><strong>Target version</strong> changed from <i>2017-12-20 Sprint</i> to <i>2018-01-17 Sprint</i></li></ul> Arvados - Idea #12702: Migrate user accountshttps://dev.arvados.org/issues/12702?journal_id=589352018-01-02T18:41:22ZPeter Amstutzpeter.amstutz@curii.com
<ul><li><strong>Assigned To</strong> deleted (<del><i>Peter Amstutz</i></del>)</li></ul> Arvados - Idea #12702: Migrate user accountshttps://dev.arvados.org/issues/12702?journal_id=589392018-01-02T18:42:33ZTom Cleggtom@curii.com
<ul><li><strong>Assigned To</strong> set to <i>Tom Clegg</i></li></ul> Arvados - Idea #12702: Migrate user accountshttps://dev.arvados.org/issues/12702?journal_id=589502018-01-02T19:11:23ZTom Cleggtom@curii.com
<ul><li><strong>Related to</strong> <i><a class="issue tracker-6 status-3 priority-4 priority-default closed parent" href="/issues/12705">Idea #12705</a>: Documentation/helper scripts for migrating users to federated identity</i> added</li></ul> Arvados - Idea #12702: Migrate user accountshttps://dev.arvados.org/issues/12702?journal_id=590662018-01-05T16:29:39ZTom Cleggtom@curii.com
<ul></ul><p>12702-migrate-user @ <a class="changeset" title="12702: Add /arvados/v1/users/{uuid}/update_uuid API. Arvados-DCO-1.1-Signed-off-by: Tom Clegg <t..." href="https://dev.arvados.org/projects/arvados/repository/arvados/revisions/87647b5d3d72ae0c291fcdf1ee3b4a46b3af91c0">87647b5d3d72ae0c291fcdf1ee3b4a46b3af91c0</a></p> Arvados - Idea #12702: Migrate user accountshttps://dev.arvados.org/issues/12702?journal_id=590672018-01-05T16:29:56ZTom Cleggtom@curii.com
<ul><li><strong>Category</strong> set to <i>API</i></li><li><strong>Status</strong> changed from <i>New</i> to <i>In Progress</i></li></ul> Arvados - Idea #12702: Migrate user accountshttps://dev.arvados.org/issues/12702?journal_id=590952018-01-05T20:08:44ZTom Cleggtom@curii.com
<ul></ul><p>12702-migrate-user @ <a class="changeset" title="12702: Add API docs for users#update_uuid. Arvados-DCO-1.1-Signed-off-by: Tom Clegg <tclegg@veri..." href="https://dev.arvados.org/projects/arvados/repository/arvados/revisions/086b27a27c844178ee52a9c4186d970689147628">086b27a27c844178ee52a9c4186d970689147628</a></p> Arvados - Idea #12702: Migrate user accountshttps://dev.arvados.org/issues/12702?journal_id=591032018-01-05T22:33:38ZLucas Di Pentimalucas.dipentima@curii.com
<ul></ul><ul>
<li>File <code>services/api/test/functional/arvados/v1/users_controller_test.rb</code>
<ul>
<li>Both tests seem to be almost identical, maybe it would be convenient to reduce them to a list iteration of <code>[user, expected]</code> values?</li>
</ul>
</li>
<li>Is the system user being avoided on this operation? If not, should it be?</li>
<li>Would it be convenient to avoid migrating the current admin user that’s requesting the operation?</li>
<li>A couple tests failed on my local <code>service/api</code> run, they don’t seem to be related but just in case:
<ul>
<li><code>services/api/test/functional/arvados/v1/filters_test.rb:183</code></li>
</ul></li>
</ul> Arvados - Idea #12702: Migrate user accountshttps://dev.arvados.org/issues/12702?journal_id=591552018-01-08T18:39:11ZTom Cleggtom@curii.com
<ul></ul><p>Lucas Di Pentima wrote:</p>
<blockquote>
<ul>
<li>File <code>services/api/test/functional/arvados/v1/users_controller_test.rb</code>
<ul>
<li>Both tests seem to be almost identical, maybe it would be convenient to reduce them to a list iteration of <code>[user, expected]</code> values?</li>
</ul></li>
</ul>
</blockquote>
<p>Indeed. Combined into one parameterized test.</p>
<blockquote>
<ul>
<li>Is the system user being avoided on this operation? If not, should it be?</li>
</ul>
</blockquote>
<p>Good idea. Added checks + tests for system_user and anonymous_user.</p>
<blockquote>
<ul>
<li>Would it be convenient to avoid migrating the current admin user that’s requesting the operation?</li>
</ul>
</blockquote>
<p>AFAICS updating the current user should work the same as any other. Added a test case.</p>
<blockquote>
<ul>
<li>A couple tests failed on my local <code>service/api</code> run, they don’t seem to be related but just in case:
<ul>
<li><code>services/api/test/functional/arvados/v1/filters_test.rb:183</code></li>
</ul></li>
</ul>
</blockquote>
<p>.*Filter.* passing for me. We'll see how this goes: <a class="external" href="https://ci.curoverse.com/job/developer-run-tests/546/">https://ci.curoverse.com/job/developer-run-tests/546/</a></p>
<p>12702-migrate-user @ <a class="changeset" title="12702: Disallow updating sys/anon UUIDs. Test updating current user. Arvados-DCO-1.1-Signed-off-..." href="https://dev.arvados.org/projects/arvados/repository/arvados/revisions/680ffd64dac92aec8ad94454334db9ae69b95b56">680ffd64dac92aec8ad94454334db9ae69b95b56</a></p> Arvados - Idea #12702: Migrate user accountshttps://dev.arvados.org/issues/12702?journal_id=591622018-01-08T19:08:45ZLucas Di Pentimalucas.dipentima@curii.com
<ul></ul><p>This LGTM. Test run failed due to a misconfigured Jenkins, as Fernando mentioned.</p> Arvados - Idea #12702: Migrate user accountshttps://dev.arvados.org/issues/12702?journal_id=591992018-01-09T16:22:08ZAnonymous
<ul><li><strong>Status</strong> changed from <i>In Progress</i> to <i>Resolved</i></li><li><strong>% Done</strong> changed from <i>0</i> to <i>100</i></li></ul><p>Applied in changeset <a class="changeset" title="Merge branch '12702-migrate-user' closes #12702 Arvados-DCO-1.1-Signed-off-by: Tom Clegg <tcleg..." href="https://dev.arvados.org/projects/arvados/repository/arvados/revisions/b5a8d7052ff461073496627aef8e2e0c29901a19">arvados|b5a8d7052ff461073496627aef8e2e0c29901a19</a>.</p>