fixed some lint errors

hashicorp · Sep 22, 2023 · 9c4fb95 · 9c4fb95
1 parent 1276430
commit 9c4fb95
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 6 deletions.
diff --git a/control-plane/connect-inject/common/annotation_processor.go b/control-plane/connect-inject/common/annotation_processor.go
@@ -15,13 +15,13 @@ const (
 	ConsulNodeAddress = "127.0.0.1"
 )
 
-// PodAnnotationProcessor processes a pod annotation into pbmesh.Upstreams
+// PodAnnotationProcessor processes a pod annotation into pbmesh.Upstreams.
 type PodAnnotationProcessor struct {
 	enablePartitions bool
 	enableNamespaces bool
 }
 
-// NewPodAnnotationProcessor constructs a PodAnnotationProcessor for processing pod annotations into pbmesh.Upsreams
+// NewPodAnnotationProcessor constructs a PodAnnotationProcessor for processing pod annotations into pbmesh.Upsreams.
 func NewPodAnnotationProcessor(enablePartitions, enableNamespaces bool) PodAnnotationProcessor {
 	return PodAnnotationProcessor{
 		enablePartitions: enablePartitions,

diff --git a/control-plane/connect-inject/common/annotation_processor_test.go b/control-plane/connect-inject/common/annotation_processor_test.go
@@ -964,7 +964,7 @@ func TestUpstreamRefrenceType(t *testing.T) {
 		Kind:         "Service",
 	}
 	actual := UpstreamReferenceType()
-	require.Equal(t, *actual, *expect)
+	require.Equal(t, actual, expect)
 }
 
 // createPod creates a multi-port pod as a base for tests.

diff --git a/control-plane/connect-inject/constants/constants.go b/control-plane/connect-inject/constants/constants.go
@@ -56,7 +56,7 @@ const (
 )
 
 // GetDefaultConsulNamespace returns the default namespace if the passed namespace
-// is empty, otherwise returns back the passed in namespace
+// is empty, otherwise returns back the passed in namespace.
 func GetDefaultConsulNamespace(ns string) string {
 	if ns == "" {
 		ns = DefaultConsulNS
@@ -66,7 +66,7 @@ func GetDefaultConsulNamespace(ns string) string {
 }
 
 // GetDefaultConsulPartition returns the default partition if the passed partition
-// is empty, otherwise returns back the passed in partition
+// is empty, otherwise returns back the passed in partition.
 func GetDefaultConsulPartition(ap string) string {
 	if ap == "" {
 		ap = DefaultConsulPartition
@@ -76,7 +76,7 @@ func GetDefaultConsulPartition(ap string) string {
 }
 
 // GetDefaultConsulPeer returns the default peer if the passed peer
-// is empty, otherwise returns back the passed in peer
+// is empty, otherwise returns back the passed in peer.
 func GetDefaultConsulPeer(peer string) string {
 	if peer == "" {
 		peer = DefaultConsulPeer