Bug #15877

Cannot update collection with EnableBetaController14287: true

Added by Peter Amstutz 6 months ago. Updated 5 months ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Category:
-
Target version:
Start date:
11/26/2019
Due date:
% Done:

100%

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

Description

$ arv collection update --uuid x1u39-4zz18-6yajeop0guo48h4 --collection '{"properties": {"fruit":"banana"}}'
Error: json: cannot unmarshal string into Go struct field UpdateOptions.attrs of type map[string]interface {}

Setting EnableBetaController14287: false, I am able to make the update with the same command.


Subtasks

Task #15878: Review 15877-accept-json-in-jsonResolvedPeter Amstutz

Associated revisions

Revision 5338c3fe
Added by Tom Clegg 6 months ago

Merge branch '15877-accept-json-in-json'

fixes #15877

Arvados-DCO-1.1-Signed-off-by: Tom Clegg <>

History

#1 Updated by Peter Amstutz 6 months ago

  • Description updated (diff)

#2 Updated by Peter Amstutz 6 months ago

  • Description updated (diff)

#3 Updated by Tom Clegg 6 months ago

  • Subject changed from Cannot update properties with EnableBetaController14287: true to Cannot update collection with EnableBetaController14287: true
  • Status changed from New to In Progress
  • Assigned To set to Tom Clegg

Same error:

arv collection update --uuid x1u39-4zz18-6yajeop0guo48h4 --collection '{}'

#4 Updated by Tom Clegg 6 months ago

Evidently arv sends a request body with json-in-json, like this

Content-Type: application/json

{"collection":"{\"properties\":{}}"}

15877-accept-json-in-json @ b69a0b0f7b4b9a773fb060a592057f9bd146e480

#5 Updated by Peter Amstutz 6 months ago

  • Target version changed from 2020-01-02 Sprint to 2019-12-04 Sprint

#6 Updated by Peter Amstutz 6 months ago

Tom Clegg wrote:

Evidently arv sends a request body with json-in-json, like this

[...]

15877-accept-json-in-json @ b69a0b0f7b4b9a773fb060a592057f9bd146e480

LGTM.

#7 Updated by Tom Clegg 6 months ago

  • Status changed from In Progress to Resolved

#8 Updated by Peter Amstutz 5 months ago

  • Release set to 22

Also available in: Atom PDF