Open epuertat opened 3 months ago
@epuertat uuid is not a required parameter in any namespace command. If the user doesn't supply it one will be generated for him. We still allow to use in namespace commands as the namespace identifier. If the user find it unfriendly he can totally ignore it. There is no need to remove it from gRPC. People using API and not the CLI might find it usefull
Few suggestions on usability:
subsystem_nqn
,subsystem
andnqn
.anagrpid
,load_balancing_group
(are they the same thing?).namespace_add
(which is consistent with CLI), butlist_namespaces
. All methods should adopt the same convention, preferably<object>_<verb>
.uuid
really required? Compared to havingnsid
(indexes, like 1, 2, 3...), it looks like a unfriendly param. That's ok for metadata, but most namespace calls accept optionallyuuid
andnsid
.