Change serf-tag references to field references.

agent/consul/leader.go

@@ -1073,13 +1073,11 @@ func (s *Server) handleAliveMember(member serf.Member, nodeEntMeta *acl.Enterpri
 			},
 		}
 
-		grpcPortStr := member.Tags["grpc_port"]
-		if v, err := strconv.Atoi(grpcPortStr); err == nil && v > 0 {
-			service.Meta["grpc_port"] = grpcPortStr
+		if parts.ExternalGRPCPort > 0 {
+			service.Meta["grpc_port"] = strconv.Itoa(parts.ExternalGRPCPort)
 		}
-		grpcTLSPortStr := member.Tags["grpc_tls_port"]
-		if v, err := strconv.Atoi(grpcTLSPortStr); err == nil && v > 0 {
-			service.Meta["grpc_tls_port"] = grpcTLSPortStr
+		if parts.ExternalGRPCTLSPort > 0 {
+			service.Meta["grpc_tls_port"] = strconv.Itoa(parts.ExternalGRPCTLSPort)
 		}
 
 		// Attempt to join the consul server

agent/metadata/server.go

@@ -23,30 +23,27 @@ func (k *Key) Equal(x *Key) bool {
 
 // Server is used to return details of a consul server
 type Server struct {
-	Name         string // <node>.<dc>
-	ShortName    string // <node>
-	ID           string
-	Datacenter   string
-	Segment      string
-	Port         int
-	SegmentAddrs map[string]string
-	SegmentPorts map[string]int
-	WanJoinPort  int
-	LanJoinPort  int
-
-	// TODO why are these ports needed? It looks like nothing is referencing them.
+	Name                string // <node>.<dc>
+	ShortName           string // <node>
+	ID                  string
+	Datacenter          string
+	Segment             string
+	Port                int
+	SegmentAddrs        map[string]string
+	SegmentPorts        map[string]int
+	WanJoinPort         int
+	LanJoinPort         int
 	ExternalGRPCPort    int
 	ExternalGRPCTLSPort int
-
-	Bootstrap    bool
-	Expect       int
-	Build        version.Version
-	Version      int
-	RaftVersion  int
-	Addr         net.Addr
-	Status       serf.MemberStatus
-	ReadReplica  bool
-	FeatureFlags map[string]int
+	Bootstrap           bool
+	Expect              int
+	Build               version.Version
+	Version             int
+	RaftVersion         int
+	Addr                net.Addr
+	Status              serf.MemberStatus
+	ReadReplica         bool
+	FeatureFlags        map[string]int
 
 	// If true, use TLS when connecting to this server
 	UseTLS bool