Skip to content

core: --locality-advertise-addr flag isn't hooked up #29591

@a-robinson

Description

@a-robinson

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:

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.

Metadata

Metadata

Assignees

Labels

A-kv-gossipA-kv-serverRelating to the KV-level RPC serverC-bugCode not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior.

Type

No type

Projects

No projects

Milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions