cluster template update error is too generic
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Magnum |
New
|
Undecided
|
M V P Nitesh |
Bug Description
Trying to update the floating_ip_enabled field:
# openstack coe cluster template update test-template replace floating_
'add' and 'replace' operations needs value (HTTP 400) (Request-ID: req-4629053f-
It's not obvious that the value is wrong. If i do instead (note uppercase False):
# openstack coe cluster template update test-template replace floating_
+------
| Field | Value |
+------
| floating_ip_enabled | False
The same original error is given if i pass a field that does not exist:
# openstack coe cluster template update test-template replace random_field=false
'add' and 'replace' operations needs value (HTTP 400) (Request-ID: req-c60395bf-
Interestingly passing an int gives a different message:
# openstack coe cluster template update test-template replace random_field=1
Couldn't apply patch '[{'path': '/random_field', 'value': 1, 'op': 'replace'}]' (HTTP 400) (Request-ID: req-67ff0b8b-
Would be nice to get Magnum to reply with:
* 'field X is not known' when an unknown field is given for replace
* 'invalid value X for field Y' when an invalid value is given for an existing field
description: | updated |
Changed in magnum: | |
assignee: | nobody → M V P Nitesh (m-nitesh) |