Bug #16067

Copy collection doesn't work

Added by Lucas Di Pentima 8 months ago. Updated 8 months ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Category:
Workbench2
Start date:
02/05/2020
Due date:
% Done:

100%

Estimated time:
(Total: 0.00 h)
Story points:
-
Release relationship:
Auto

Description

Copying a collection on workbench 2 is producing the following error on arbvox:

request failed: http://localhost:8004/arvados/v1/collections: 422 Unprocessable Entity: #<ActiveModel::UnknownAttributeError: unknown attribute 'etag' for Collection.> (req-1ekthehdwqdgw1qo08e2)

Subtasks

Arvados - Task #16127: Review 16067-collection-copy-fixResolvedPeter Amstutz


Related issues

Related to Arvados Epics - Story #15333: Workbench2 feature parity with WorkbenchIn Progress01/01/202012/31/2020

Has duplicate Arvados - Bug #16125: Can't copy collectionDuplicate

Blocks Arvados - Story #16015: Release 2.0Resolved01/28/2020

Associated revisions

Revision 79ba1fb0
Added by Lucas Di Pentima 8 months ago

Merge branch '16067-collection-copy-fix'

Closes #16067

Arvados-DCO-1.1-Signed-off-by: Lucas Di Pentima <>

History

#1 Updated by Lucas Di Pentima 8 months ago

  • Related to Story #15333: Workbench2 feature parity with Workbench added

#2 Updated by Peter Amstutz 8 months ago

  • Has duplicate Bug #16125: Can't copy collection added

#3 Updated by Peter Amstutz 8 months ago

  • Release changed from 20 to 22
  • Target version changed from To Be Groomed to 2020-02-12 Sprint

#4 Updated by Peter Amstutz 8 months ago

  • Assigned To set to Lucas Di Pentima

#5 Updated by Peter Amstutz 8 months ago

#6 Updated by Lucas Di Pentima 8 months ago

  • Status changed from New to In Progress

#7 Updated by Lucas Di Pentima 8 months ago

Updates at commit: 6d1f423a - branch 16067-collection-copy-fix
Test run: https://ci.arvados.org/view/Developer/job/developer-tests-workbench2/11/

  • Lays the groundwork to avoid sending read-only fields to the API server on every resource type
  • Fixes the bug by:
    • Adding uuid, kind and etag as common resource read-only fields
    • Adding unsigned_manifest, storage_classes_confirmed and storage_classes_confirmed_at as collection resource read-only fields.
  • Adds tests on #create and #update

#8 Updated by Peter Amstutz 8 months ago

Lucas Di Pentima wrote:

Updates at commit: 6d1f423a - branch 16067-collection-copy-fix
Test run: https://ci.arvados.org/view/Developer/job/developer-tests-workbench2/11/

  • Lays the groundwork to avoid sending read-only fields to the API server on every resource type
  • Fixes the bug by:
    • Adding uuid, kind and etag as common resource read-only fields
    • Adding unsigned_manifest, storage_classes_confirmed and storage_classes_confirmed_at as collection resource read-only fields.
  • Adds tests on #create and #update

This LGTM.

#9 Updated by Lucas Di Pentima 8 months ago

  • Status changed from In Progress to Resolved

Also available in: Atom PDF