cli: Support copies number parameter in object put #350

Merged
fyrchik merged 1 commit from acid-ant/frostfs-node:feature/cli-obj-put-copies-num into master 2023-05-16 08:22:58 +00:00
Member

Close #351

Signed-off-by: Anton Nikiforov an.nikiforov@yadro.com

Close #351 Signed-off-by: Anton Nikiforov <an.nikiforov@yadro.com>
acid-ant force-pushed feature/cli-obj-put-copies-num from e7dc990c5f to 84020bdf6a 2023-05-15 12:27:00 +00:00 Compare
acid-ant force-pushed feature/cli-obj-put-copies-num from 84020bdf6a to f3b45ca59b 2023-05-15 12:42:27 +00:00 Compare
acid-ant changed title from WIP: cli: Support copies number parameter in object put to cli: Support copies number parameter in object put 2023-05-15 12:42:55 +00:00
fyrchik reviewed 2023-05-15 12:45:49 +00:00
@ -57,2 +58,4 @@
flags.String(notificationFlag, "", "Object notification in the form of *epoch*:*topic*; '-' topic means using default")
flags.Bool(binaryFlag, false, "Deserialize object structure from given file.")
flags.UintSlice(copiesNumberFlag, nil, "Number of copies of the object to store within the RPC call")
Owner

I believe UintSlice allows following usage: --copies-number 1 --copies-number 2.
It should be a bug, it is really and atomic vector and not an arbitrary-sized slice.
What about using String and split it by ,?

I believe `UintSlice` allows following usage: `--copies-number 1 --copies-number 2`. It should be a bug, it is really and _atomic_ vector and not an arbitrary-sized slice. What about using `String` and split it by `,`?
Author
Member

Surprised, will use string instead of uint slice.

Surprised, will use string instead of uint slice.
Author
Member

Done, please review.

Done, please review.
acid-ant force-pushed feature/cli-obj-put-copies-num from f3b45ca59b to 0f10899ba4 2023-05-15 13:23:00 +00:00 Compare
acid-ant requested review from storage-core-committers 2023-05-15 13:23:22 +00:00
acid-ant requested review from storage-core-developers 2023-05-15 13:23:22 +00:00
dstepanov-yadro approved these changes 2023-05-15 14:20:10 +00:00
fyrchik approved these changes 2023-05-15 16:12:37 +00:00
fyrchik approved these changes 2023-05-16 08:22:52 +00:00
fyrchik merged commit 8da6530f41 into master 2023-05-16 08:22:58 +00:00
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
3 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: TrueCloudLab/frostfs-node#350
No description provided.