CONTROLLER_INGRESS_CLASS Problem in use

when i add CONTROLLER_INGRESS_CLASS = kong-internal as env of ingress-control
and then i add “kubernetes.io/ingress.class: kong-internal” as annotaion of Kongingress , I found it do affect on the route (if i do not add the annotation i will never affect on route ), but i also found that the kind of kongPlugin exception ,mean that ,it will also affect on plugin if i donot add the annotation to the kongplugin e.g:
apiVersion: configuration.konghq.com/v1
kind: KongPlugin
metadata:
name: echo-rate-limiting
config:
second: 10
minute: 11
plugin: rate-limiting

so ,whether this annotaion just affect on kongingress and ingress?
For KongPlugin will be detected by kong-ingress-control no matter whether “kubernetes.io/ingress.class: kong-internal” added

tl;dr what does and does not require the ingress.class annotation isn’t properly defined or implemented correctly yet. We’re working to fix that, but for the time being, adding it to any resource (including KongPlugins and other non-Ingress resources) may help to work around some of the known bugs.

We recognize that the way we use the ingress.class annotation is a point of confusion and that the existing behavior is less than ideal; we’re working on revamping that to try and make it more coherent: https://github.com/Kong/kubernetes-ingress-controller/pull/767

Currently, the ingress.class can be used on virtually all resources that the controller interacts with, and if you use a custom class (your kong-internal), it expects that resources will have that annotation even when they shouldn’t strictly need it. That’s part of what we’re working to address in that PR, to better delineate when ingress.class is required and to handle its absence when it isn’t.

If you’re having trouble, adding appropriate ingress.class annotations to resources should never hurt them, even if they aren’t actually Ingresses: KongPlugin in particular has known issues around updates not being honored if a KongPlugin resource has no ingress.class annotation, though it shouldn’t require them, and actually won’t in upcoming releases.


© 2019 Kong Inc.    Terms  •  Privacy  •  FAQ