BUG/MINOR: Fix ingress accepted with ingressClassName set to unexisting ingressclass.

ivanmatmati · oktalz · commit 2c3369dbe446 · 2021-11-12T12:47:12.000+01:00
diff --git a/controller/ingress/ingress.go b/controller/ingress/ingress.go
@@ -68,7 +68,7 @@ func (i Ingress) supported(k8s store.K8s) (supported bool) {
 	}()
 
 	if i.controllerClass == "" {
-		if igClassAnn == "" && igClassSpec == "" {
+		if igClassAnn == "" && i.resource.Class == "" {
 			supported = true
 			return
 		}
@@ -77,7 +77,7 @@ func (i Ingress) supported(k8s store.K8s) (supported bool) {
 			return
 		}
 	} else {
-		if igClassAnn == "" && igClassSpec == "" && i.allowEmptyClass {
+		if igClassAnn == "" && i.resource.Class == "" && i.allowEmptyClass {
 			supported = true
 			return
 		}

Original file line number	Diff line number	Diff line change
`@@ -68,7 +68,7 @@ func (i Ingress) supported(k8s store.K8s) (supported bool) {`
`68`	`68`	`}()`
`69`	`69`
`70`	`70`	`if i.controllerClass == "" {`
`71`		`- if igClassAnn == "" && igClassSpec == "" {`
	`71`	`+ if igClassAnn == "" && i.resource.Class == "" {`
`72`	`72`	`supported = true`
`73`	`73`	`return`
`74`	`74`	`}`
`@@ -77,7 +77,7 @@ func (i Ingress) supported(k8s store.K8s) (supported bool) {`
`77`	`77`	`return`
`78`	`78`	`}`
`79`	`79`	`} else {`
`80`		`- if igClassAnn == "" && igClassSpec == "" && i.allowEmptyClass {`
	`80`	`+ if igClassAnn == "" && i.resource.Class == "" && i.allowEmptyClass {`
`81`	`81`	`supported = true`
`82`	`82`	`return`
`83`	`83`	`}`