connect: fixup some leftover debugging comments

nomad/job_endpoint_hook_connect.go

@@ -128,6 +128,7 @@ func groupConnectHook(job *structs.Job, g *structs.TaskGroup) error {
 			task.Canonicalize(job, g)
 
 			makePort := func(label string) {
+				// check that port hasn't already been defined before adding it to tg
 				for _, p := range g.Networks[0].DynamicPorts {
 					if p.Label == label {
 						return

nomad/job_endpoint_test.go

@@ -263,7 +263,7 @@ func TestJobEndpoint_Register_ConnectProxyExposeChecks(t *testing.T) {
 		ListenerPort:  "v2Port",
 	}, grpcPath)
 
-	// todo: round tripping is probably gonna duplicate since we only append
+	// make sure round tripping does not create duplicate expose paths
 	out.Meta["test"] = "abc"
 	req.Job = out
 	r.NoError(msgpackrpc.CallWithCodec(codec, "Job.Register", req, &resp))
@@ -275,7 +275,7 @@ func TestJobEndpoint_Register_ConnectProxyExposeChecks(t *testing.T) {
 	out, err = state.JobByID(ws, job.Namespace, job.ID)
 	r.NoError(err)
 	r.NotNil(out)
-	r.Equal(resp.JobModifyIndex, out.CreateIndex) // todo uhh
+	r.Equal(resp.JobModifyIndex, out.CreateIndex)
 
 	// make sure we are not re-adding what has already been added
 	r.Len(out.TaskGroups[0].Services[0].Connect.SidecarService.Proxy.Expose.Paths, 2)

nomad/structs/services.go

@@ -47,7 +47,7 @@ type ServiceCheck struct {
 	Path          string              // path of the health check url for http type check
 	Protocol      string              // Protocol to use if check is http, defaults to http
 	PortLabel     string              // The port to use for tcp/http checks
-	AddressMode   string              // 'host' to use host ip:port or 'driver' to use driver'
+	AddressMode   string              // 'host' to use host ip:port or 'driver' to use driver's
 	Interval      time.Duration       // Interval of the check
 	Timeout       time.Duration       // Timeout of the response from the check before consul fails the check
 	InitialStatus string              // Initial status of the check