Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

[docs] Helm install for Apisix is not setting the ingressClass name properly #409

Copy link
Copy link
@kastl-ars

Description

@kastl-ars
Issue body actions

Describe the bug
Trying out open-appsec with Apisix using the documentation.

The documentation states the following command:

helm install open-appsec-apisix openappsec/open-appsec-apisix \
--set rbac.create=true \
--set service.type=LoadBalancer \
--set appsec.persistence.enabled=false \
--set ingress-controller.enabled=true \
--set ingress-controller.config.ingressClass=appsec-apisix \
--set appsec.userEmail="<your-email-address>" \
--set appsec.agentToken= \
--namespace appsec-apisix --create-namespace

The open-appsec-apisix chart has the Apisix apisix-ingress-controller chart as a dependency (called ingress-controller). So ingress-controller.config.ingressClass=appsec-apisix is meant to be handed over to the dependency chart.

Unfortunately, the chart does not have such a parameter. In the chart it is called config.kubernetes.ingressClass:

config:
 [...]
  kubernetes:
    ingressClass: apisix
    defaultIngressClass: false

https://github.com/apache/apisix-helm-chart/blob/master/charts/apisix-ingress-controller/values.yaml#L99

So the helm install command should use --set ingress-controller.config.kubernetes.ingressClass=appsec-apisix to properly set the ingressClass name. Otherwise the ingressClass name will be the apisix-ingress-controller chart's default value apisix.

Kind Regards,
Johannes

Reactions are currently unavailable

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't workingSomething isn't working

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions

      Morty Proxy This is a proxified and sanitized view of the page, visit original site.