Creating an ingress object with no http rules makes the controller crash loop

[alex-bezek]

Kubernetes Version

1.28

Helm Chart Version

0.15.0

Ingress Controller Version

0.12.0

System Info

Testing in EKS

Kubernetes Logs

{"level":"info","ts":"2024-09-11T16:34:29Z","logger":"setup","msg":"starting manager","version":"0.12.2","commit":"b28f15b633ca48c2d8ca771df4f4ff82affc7875"}
{"level":"info","ts":"2024-09-11T16:34:29Z","logger":"setup","msg":"configured API client","base_url":"http://api.controlplane.svc.cluster.local"}
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x0 pc=0x15f42c3]

goroutine 1 [running]:
github.com/ngrok/kubernetes-ingress-controller/internal/store.Store.shouldHandleIngressIsValid({{{0x1cf47c0, 0xc0006301c8}, {0x1cf47c0, 0xc0006301e0}, {0x1cf47c0, 0xc000630210}, {0x1cf47c0, 0xc000630228}, {0x1cf47c0, 0xc000630240}, ...}, ...}, ...)
	github.com/ngrok/kubernetes-ingress-controller/internal/store/store.go:427 +0xc3
github.com/ngrok/kubernetes-ingress-controller/internal/store.Store.shouldHandleIngress({{{0x1cf47c0, 0xc0006301c8}, {0x1cf47c0, 0xc0006301e0}, {0x1cf47c0, 0xc000630210}, {0x1cf47c0, 0xc000630228}, {0x1cf47c0, 0xc000630240}, ...}, ...}, ...)
	github.com/ngrok/kubernetes-ingress-controller/internal/store/store.go:391 +0x58
github.com/ngrok/kubernetes-ingress-controller/internal/store.Store.ListNgrokIngressesV1({{{0x1cf47c0, 0xc0006301c8}, {0x1cf47c0, 0xc0006301e0}, {0x1cf47c0, 0xc000630210}, {0x1cf47c0, 0xc000630228}, {0x1cf47c0, 0xc000630240}, ...}, ...})
	github.com/ngrok/kubernetes-ingress-controller/internal/store/store.go:301 +0x11b
github.com/ngrok/kubernetes-ingress-controller/internal/store.(*Driver).PrintState(0xc00001a600, {{0x1cf17f8?, 0xc00047a180?}, 0x7f907095fb80?})
	github.com/ngrok/kubernetes-ingress-controller/internal/store/driver.go:258 +0x5d
main.getDriver({_, _}, {_, _}, {{0x7ffce9dd6c7f, 0x5}, {0x7ffce9dd6c93, 0x24}, {0x7ffce9dd6c62, 0x5}, ...})
	./main.go:413 +0x4d8
main.runController({_, _}, {{0x7ffce9dd6c7f, 0x5}, {0x7ffce9dd6c93, 0x24}, {0x7ffce9dd6c62, 0x5}, {0x7ffce9dd6bb2, 0x1b}, ...})
	./main.go:196 +0x8ce
main.cmd.func1(0xc00029c400?, {0x1a3b72d?, 0x4?, 0x1a3b731?})
	./main.go:111 +0x5b
github.com/spf13/cobra.(*Command).execute(0xc00028a608, {0xc0001b4010, 0xa, 0xa})
	github.com/spf13/cobra@v1.7.0/command.go:940 +0x882
github.com/spf13/cobra.(*Command).ExecuteC(0xc00028a608)
	github.com/spf13/cobra@v1.7.0/command.go:1068 +0x3a5
github.com/spf13/cobra.(*Command).Execute(...)
	github.com/spf13/cobra@v1.7.0/command.go:992
main.main()
	./main.go:76 +0x18

Helm Chart configuration

standard

What happened

Tried to create a minimal ingress object in order to attach a traffic policy to. Since its a redirect, I don't care about the path or other rules, just the domain, so I tried to create an ingress object like this

apiVersion: networking.k8s.io/v1
kind: Ingress
metadata:
  annotations:
    k8s.ngrok.com/traffic-policy: k8s-redirect
  creationTimestamp: null
  name: k8s-redirect
  namespace: ngrok-operators
spec:
  ingressClassName: ngrok
  rules:
  - host: k8s.dev-ngrok.com

This ingress object causes the leader pod to crash, so a new pod takes over as leader which crashes, which continues till they are all down and ingress stops working

NAME                                                     READY   STATUS             RESTARTS        AGE
kubernetes-ingress-controller-manager-849d479c4d-2lqn8   0/1     CrashLoopBackOff   6 (2m40s ago)   10m
kubernetes-ingress-controller-manager-849d479c4d-hhxnm   0/1     CrashLoopBackOff   6 (118s ago)    10m
kubernetes-ingress-controller-manager-849d479c4d-lb5m5   0/1     CrashLoopBackOff   6 (2m22s ago)   10m

What you think should happen instead

At the very least this should not cause the controller to crash which would result in an outage for someone.
Ideally, this configuration would be accepted and result in a end with no routes assuming that would work with a traffic policy to handle redirects

How to reproduce

create an ingress object in the cluster with no rules, sit back and watch them start crashing

[alex-bezek]

Looks like its probably

ngrok-operator/internal/store/store.go

Line 424 in 1e2367d

for _, path := range ing.Spec.Rules[0].HTTP.Paths {