Skip to content

Commit 825c8c8

Browse files
committed
docs(model): state the current truth about the providers map's over-report
The Public-only filter's comment (I4 in the final review wave, dropped when that wave was cherry-picked before the filter itself) claimed the filter keeps the map from over-reporting against /network/provider-locations. Since the egress-health gate shipped it over-reports anyway (~305 vs ~137 for the US on beta): the map is deliberately not gated on egress health, because it is an aggregate supply-footprint statistic, not a pick list. Say so, and say what would justify revisiting.
1 parent ba3e652 commit 825c8c8

1 file changed

Lines changed: 14 additions & 3 deletions

File tree

model/providers_map_model.go

Lines changed: 14 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -133,9 +133,20 @@ func GetProvidersMap(ctx context.Context) (map[string]map[string]*RegionProvider
133133
-- for a cross-network pair, so a Public provide key is
134134
-- what makes a provider generally reachable; a
135135
-- ProvideModeNetwork provider is real supply only to its
136-
-- own network and is effectively private. Without this the
137-
-- map reports more providers than /network/provider-locations
138-
-- will let anyone pick from.
136+
-- own network and is effectively private.
137+
--
138+
-- The Public rule is the ONLY rule shared with
139+
-- UpdateClientLocations. This map is deliberately NOT
140+
-- gated on egress health or on an observed egress country,
141+
-- so since that gate shipped it reports MORE providers
142+
-- than /network/provider-locations will let anyone pick
143+
-- from (~305 vs ~137 for the US on beta). That is
144+
-- acceptable because the map is an aggregate
145+
-- supply-footprint statistic -- where the fleet physically
146+
-- is -- not a pick list, and nothing selects from it.
147+
-- Revisit if it ever becomes a selection surface, or is
148+
-- shown next to the per-location provider_count, where the
149+
-- two numbers disagreeing would read as a bug.
139150
EXISTS (
140151
SELECT 1 FROM provide_key
141152
WHERE

0 commit comments

Comments
 (0)