Skip to content

Commit

Permalink
Merge pull request #163 from backguynn/main
Browse files Browse the repository at this point in the history
add ingressClassName in gateway API
  • Loading branch information
backguynn committed Aug 7, 2024
2 parents a9c0114 + 1cc43f7 commit 4ccd41a
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 6 deletions.
6 changes: 1 addition & 5 deletions pkg/agent/manager/gatewayapi/gateway.go
Original file line number Diff line number Diff line change
Expand Up @@ -353,11 +353,7 @@ func (gm *GatewayManager) createIngressLbService(ctx context.Context, gateway *v
if newService.Spec.Selector == nil {
newService.Spec.Selector = map[string]string{}
}
newService.Spec.Selector["app.kubernetes.io/component"] = "controller"
//newService.Spec.Selector["app.kubernetes.io/instance"] = "loxilb-ingress"
//newService.Spec.Selector["app.kubernetes.io/name"] = "loxilb-ingress"
newService.Spec.Selector["app.kubernetes.io/instance"] = "ingress-nginx"
newService.Spec.Selector["app.kubernetes.io/name"] = "ingress-nginx"
newService.Spec.Selector["app"] = "loxilb-ingress-app"

svc, err = gm.kubeClient.CoreV1().Services(newService.Namespace).Create(ctx, &newService, metav1.CreateOptions{})
if err != nil {
Expand Down
9 changes: 8 additions & 1 deletion pkg/agent/manager/gatewayapi/httproute.go
Original file line number Diff line number Diff line change
Expand Up @@ -296,7 +296,7 @@ func (h *HTTPRouteManager) createIngress(ctx context.Context, httpRoute *v1.HTTP
newIngress.SetLabels(map[string]string{
"implementation": implementation,
})
ingressClass := "nginx"
ingressClass := "loxilb"
newIngress.Spec.IngressClassName = &ingressClass

// If the listener has a hostname, it takes precedence over httpRoute.
Expand Down Expand Up @@ -327,6 +327,13 @@ func (h *HTTPRouteManager) createIngress(ctx context.Context, httpRoute *v1.HTTP
for _, match := range rule.Matches {
for _, backref := range rule.BackendRefs {

if backref.Namespace != nil {
if newIngress.Namespace != string(*backref.Namespace) {
newIngress.Annotations["external-backend-service"] = "true"
newIngress.Annotations["service-"+string(backref.Name)+"-namespace"] = string(*backref.Namespace)
}
}

newIngressPath := netv1.HTTPIngressPath{
Backend: netv1.IngressBackend{
Service: &netv1.IngressServiceBackend{
Expand Down

0 comments on commit 4ccd41a

Please sign in to comment.