Giter Site home page Giter Site logo

Comments (6)

gxthrj avatar gxthrj commented on May 21, 2024

For now, apisix-ingress-controller do not support the config admin_key from Apache APISIX.
When using apisix-ingress-controller with APISIX, it is recommended to remove the admin_key setting. You can use allow_admin to restrict the access scope of the APISIX control plane to ensure security.

Now we can discuss how to support the admin_key in apisix-ingress-controller.
In different APISIX clusters, there are different admin_keys. Usually we manage admin_key through secret resource. But it is not easy to sync with CRDs in apisix-ingress-controller, for secret may in the different namespace.

We can use ingress.class to specify APISIX cluster, and also can define an annotation for admin_key, such as ingress.admin_key, But it will need to be updated manually in the future, if admin_key need to be changed.

So anyone has a good idea, can discuss it here. Thanks.

from apisix-ingress-controller.

gxthrj avatar gxthrj commented on May 21, 2024

@tokers what do you think about this ?

from apisix-ingress-controller.

tokers avatar tokers commented on May 21, 2024

Secret is the most appropriate way, we can watch all namespaces except some unexpected like kube-system, istio-system and etc.

from apisix-ingress-controller.

tokers avatar tokers commented on May 21, 2024

The multiple APISIX cluster support should be implemented by Manager API. We as the invoker of Manager API, only need to pass cluster flag to it.

@gxthrj @membphis

from apisix-ingress-controller.

membphis avatar membphis commented on May 21, 2024

for the CP API, need to support mTLS way. The Admin API has supportted this feature. The Manage API, need @nic-chen 's confirming.

if the ingress-controller needs to connect multiple APISIX clusters, I think using mTLS is a better way than the token or JWT method.

from apisix-ingress-controller.

gxthrj avatar gxthrj commented on May 21, 2024

The multiple APISIX cluster support should be implemented by Manager API. We as the invoker of Manager API, only need to pass cluster flag to it.

@gxthrj @membphis

This sounds good.
But there is another problem. If we need to specify / add an APISIX Cluster in CRD, the manager api need to config too.
The configuration link has become too long and uncontrollable

from apisix-ingress-controller.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.