cluster: use typed IDs, introduce Region.ID, and pre-compute primary region #49

Merged

karasz merged 7 commits from pr-amery-regions into main 2 years ago

amery commented 2 years ago

Owner

There is no content yet.

amery added 7 commits 2 years ago

d98c879409 build-sys: use revive 1.3.7 instead of master

Signed-off-by: Alejandro Mery <[email protected]>

f4b837f3e0 rings: add String() to RegionID, ZoneID, NodeID

Signed-off-by: Alejandro Mery <[email protected]>

bb40f395d9 rings: fix NodeZeroMax/NodeMax to not accept .255

Signed-off-by: Alejandro Mery <[email protected]>

7f849415ca cluster: use rings.ZoneID and rings.NodeID types

Signed-off-by: Alejandro Mery <[email protected]>

1df02f2b2b cluster: assign valid rings.RegionID to each primary region

Signed-off-by: Alejandro Mery <[email protected]>

410a9f8e30 cluster: run initRegions() before scanZoneIDs()

Signed-off-by: Alejandro Mery <[email protected]>

c84318448c cluster: pre-compute Zone's primary region

Signed-off-by: Alejandro Mery <[email protected]>

amery added 7 commits 2 years ago

83921c1e13 build-sys: use revive 1.3.7 instead of master

Signed-off-by: Alejandro Mery <[email protected]>

ff8f2c6ea1 rings: add String() to RegionID, ZoneID, NodeID

Signed-off-by: Alejandro Mery <[email protected]>

6d89e0ea3c rings: fix NodeZeroMax/NodeMax to not accept .255

Signed-off-by: Alejandro Mery <[email protected]>

99dece1e43 cluster: use rings.ZoneID and rings.NodeID types

Signed-off-by: Alejandro Mery <[email protected]>

14e1c447c9 cluster: assign valid rings.RegionID to each primary region

Signed-off-by: Alejandro Mery <[email protected]>

cede95e119 cluster: run initRegions() before scanZoneIDs()

Signed-off-by: Alejandro Mery <[email protected]>

982e16581c cluster: pre-compute Zone's primary region

Signed-off-by: Alejandro Mery <[email protected]>

karasz approved these changes 2 years ago

karasz left a comment

Owner

LGTM

karasz merged commit 932a41a3ac into main 2 years ago

karasz referenced this issue from a commit 2 years ago

Merge pull request 'cluster: use typed IDs, introduce Region.ID, and pre-compute primary region' (#49) from pr-amery-regions into main

karasz deleted branch pr-amery-regions 2 years ago

amery commented 2 years ago

Poster

Owner

v0.8.2 tagged

Reviewers

karasz approved these changes 2 years ago

The pull request has been merged as 932a41a3ac.

Sign in to join this conversation.

No Label

No Milestone

No project

No Assignees

2 Participants

Notifications

Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.