Inconsistencies with lists of unique identifiers #377

New Issue

Closed

opened 2022-09-12 11:05:26 +00:00 by quentin · 1 comment

quentin commented

2022-09-12 11:05:26 +00:00

Owner

We have 2 ways to represent list of identifiers:

On GetClusterLayout, we use a dict of { uniq id: properties}
On ListBuckets, we use a list of [ { id: uniq_id, properties... }]

It would be great to choose either the dict or list approach.

We have 2 ways to represent list of identifiers: - On `GetClusterLayout`, we use a dict of `{ uniq id: properties}` - On `ListBuckets`, we use a list of `[ { id: uniq_id, properties... }]` It would be great to choose either the dict or list approach.

quentin added the

labels 2022-09-12 11:05:32 +00:00

quentin added this to the v0.9 milestone 2022-09-14 13:26:29 +00:00

lx referenced this issue from a commit

2023-06-14 11:45:59 +00:00

admin api: change cluster status/layout to use lists and not maps (fix #377)

lx commented

2023-06-14 12:20:09 +00:00

Owner

Fixed by commit 52376d47ca in the next branch, which will be merged for Garage v0.9

The choice has been made to use the list format everywhere

Fixed by commit 52376d47ca in the `next` branch, which will be merged for Garage v0.9 The choice has been made to use the list format everywhere

lx closed this issue

2023-06-14 12:20:09 +00:00

lx referenced this issue

2023-09-04 09:39:18 +00:00

Garage v0.9 #473

Sign in to join this conversation.

No Label

S3 Compatibility

No Milestone

No Assignees

2 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: Deuxfleurs/garage#377

No description provided.