I don't think name should be made mandatory as we have UUID to do operations.
But a point here, in case user doesn't pass name while creating a bay, the bay-list displays the bay with name 'None' which doesn't make any sense to user.
$ magnum bay-list +--------------------------------------+------+------------+--------------------+ | uuid | name | node_count | status | +--------------------------------------+------+------------+--------------------+ | 7be4a7a9-9685-46b3-8f8b-7f19a02c644e | None | 1 | CREATE_COMPLETE | | 7d59afb0-1c24-4cae-93fc-4692f5438d34 | bay1 | 1 | CREATE_COMPLETE | | 972a7a42-2d77-4eac-a7ba-beeee8939a98 | None | 1 | CREATE_IN_PROGRESS | +--------------------------------------+------+------------+--------------------+
$ magnum bay-show None ERROR: Bay None could not be found. (HTTP 404)
Rather the 'name' field here should be empty.
I don't think name should be made mandatory as we have UUID to do operations.
But a point here, in case user doesn't pass name while creating a bay, the bay-list displays the bay with name 'None' which doesn't make any sense to user.
$ magnum bay-list ------- ------- ------- ------- ----+-- ----+-- ------- ---+--- ------- ------- ---+ ------- ------- ------- ------- ----+-- ----+-- ------- ---+--- ------- ------- ---+ 9685-46b3- 8f8b-7f19a02c64 4e | None | 1 | CREATE_COMPLETE | 1c24-4cae- 93fc-4692f5438d 34 | bay1 | 1 | CREATE_COMPLETE | 2d77-4eac- a7ba-beeee8939a 98 | None | 1 | CREATE_IN_PROGRESS | ------- ------- ------- ------- ----+-- ----+-- ------- ---+--- ------- ------- ---+
+------
| uuid | name | node_count | status |
+------
| 7be4a7a9-
| 7d59afb0-
| 972a7a42-
+------
$ magnum bay-show None
ERROR: Bay None could not be found. (HTTP 404)
Rather the 'name' field here should be empty.