lint / cspell

controllers/gateway/controller.go

@@ -1,3 +1,4 @@
+// Package gateway contains controllers for Gateway API objects.
 package gateway
 
 import (
@@ -18,6 +19,7 @@ import (
 	"github.com/pomerium/ingress-controller/pomerium"
 )
 
+// DefaultClassControllerName is the default GatewayClass ControllerName.
 const DefaultClassControllerName = "pomerium.io/gateway-controller"
 
 // ControllerConfig contains configuration options for the Gateway controller.
@@ -47,8 +49,11 @@ func NewGatewayController(
 		ControllerConfig:  config,
 	}
 
-	mgr.GetFieldIndexer().IndexField(ctx, &corev1.Secret{}, "type",
+	err := mgr.GetFieldIndexer().IndexField(ctx, &corev1.Secret{}, "type",
 		func(o client.Object) []string { return []string{string(o.(*corev1.Secret).Type)} })
+	if err != nil {
+		return fmt.Errorf("couldn't create index on Secret type: %w", err)
+	}
 
 	// All updates will trigger the same reconcile request.
 	enqueueRequest := handler.EnqueueRequestsFromMapFunc(
@@ -60,7 +65,7 @@ func NewGatewayController(
 			}}
 		})
 
-	err := ctrl.NewControllerManagedBy(mgr).
+	err = ctrl.NewControllerManagedBy(mgr).
 		Named("gateway").
 		Watches(
 			&gateway_v1.Gateway{},
@@ -92,7 +97,10 @@ func (c *gatewayController) Reconcile(ctx context.Context, _ ctrl.Request) (ctrl
 
 	config := c.processGateways(ctx, o)
 
-	c.SetGatewayConfig(ctx, config)
+	_, err = c.SetGatewayConfig(ctx, config)
+	if err != nil {
+		return ctrl.Result{}, err
+	}
 
 	return ctrl.Result{}, nil
 }

controllers/gateway/fetch.go

@@ -4,12 +4,13 @@ import (
 	context "context"
 
 	"github.com/hashicorp/go-set/v3"
-	"github.com/pomerium/ingress-controller/util"
 	corev1 "k8s.io/api/core/v1"
 	"k8s.io/apimachinery/pkg/types"
 	"sigs.k8s.io/controller-runtime/pkg/client"
 	gateway_v1 "sigs.k8s.io/gateway-api/apis/v1"
 	gateway_v1beta1 "sigs.k8s.io/gateway-api/apis/v1beta1"
+
+	"github.com/pomerium/ingress-controller/util"
 )
 
 // objects holds all relevant Gateway objects and their dependencies.

controllers/gateway/gateway.go

@@ -4,12 +4,13 @@ import (
 	context "context"
 	golog "log"
 
-	"github.com/pomerium/ingress-controller/model"
 	"k8s.io/apimachinery/pkg/api/equality"
 	metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
 	"k8s.io/apimachinery/pkg/types"
 	"sigs.k8s.io/controller-runtime/pkg/log"
 	gateway_v1 "sigs.k8s.io/gateway-api/apis/v1"
+
+	"github.com/pomerium/ingress-controller/model"
 )
 
 // processGateways updates the status of all Gateways and associated routes, and returns a

controllers/gateway/httproute.go

@@ -55,7 +55,7 @@ func processHTTPRoute(
 		return result
 	}
 
-	// Otherwise check for route attachement with all listeners.
+	// Otherwise check for route attachment with all listeners.
 	hostnamesSet := set.New[gateway_v1.Hostname](0)
 	var reason gateway_v1.RouteConditionReason
 	for _, l := range listeners {

controllers/gateway/listener.go

@@ -6,11 +6,12 @@ import (
 	"strings"
 
 	"github.com/hashicorp/go-set/v3"
-	"github.com/pomerium/ingress-controller/model"
 	corev1 "k8s.io/api/core/v1"
 	metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
 	"k8s.io/apimachinery/pkg/runtime/schema"
 	gateway_v1 "sigs.k8s.io/gateway-api/apis/v1"
+
+	"github.com/pomerium/ingress-controller/model"
 )
 
 type listenerAndStatus struct {