-
Notifications
You must be signed in to change notification settings - Fork 4.1k
core: --locality-advertise-addr flag isn't hooked up #29591
Copy link
Copy link
Closed
Labels
A-kv-gossipA-kv-serverRelating to the KV-level RPC serverRelating to the KV-level RPC serverC-bugCode not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior.Code not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior.
Milestone
Description
The contents of the --locality-advertise-addr flag never actually get plumbed into the server/gossip code, meaning that the feature almost certainly doesn't work.
The localityAddress arg to Node.start is unused:
Line 396 in 44a5f06
| localityAddress []roachpb.LocalityAddress, |
The NodeDescriptor.LocalityAddress field never actually gets set anywhere in our code:
https://sourcegraph.com/github.com/cockroachdb/cockroach@master/-/blob/pkg/roachpb/metadata.pb.go#L166:2=&tab=references
That should be fixed and the feature tested end-to-end.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
A-kv-gossipA-kv-serverRelating to the KV-level RPC serverRelating to the KV-level RPC serverC-bugCode not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior.Code not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior.