Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Nginx attributes for connecting with AWS NLB not being applied to NLB instance #3991

Open
Timotej979 opened this issue Dec 20, 2024 · 1 comment
Labels
kind/bug Categorizes issue or PR as related to a bug. triage/needs-information Indicates an issue needs more information in order to work on it.

Comments

@Timotej979
Copy link

Hello 😄

currently I'm facing an issue regarding applying the helm chart values of a ingress-nginx controller connected to the AWS NLB using service annotations in EKS. My current values.yaml file for the nginx-ingress look like this:

controller:
  service:
    annotations:
      service.beta.kubernetes.io/aws-load-balancer-type: "nlb"
      service.beta.kubernetes.io/aws-load-balancer-attributes: |
        load_balancing.cross_zone.enabled=true,
        deletion_protection.enabled=true

  resources:
    limits:
      cpu: "500m"
      memory: "512Mi"
    requests:
      cpu: "500m"
      memory: "512Mi"
  admissionWebhooks:
    createSecretJob:
      resources:
        limits:
          cpu: "100m"
          memory: "64Mi"
        requests:
          cpu: "100m"
          memory: "64Mi"
    patchWebhookJob:
      resources:
        limits:
          cpu: "100m"
          memory: "64Mi"
        requests:
          cpu: "100m"
          memory: "64Mi"

They look correct to me from outside, however on the terraform apply command of a helm release with this values the load balancer fails to enable cross zone load balancing, I managed to overcome this using an alternative annotation which still works apparently, even thou in the latest docs it is said to be deprecated. Link to the annotation: service.beta.kubernetes.io/aws-load-balancer-cross-zone-load-balancing-enabled

The issue I am specifically having is the deletion protection argument/annotation, which I would really like to have on the infrastructure.

I'm using an EKS cluster version 1.30.
Latest ingress nginx helm chart version from: https://kubernetes.github.io/ingress-nginx.

Any help would be much appreciated, even thou it seems like a really simple issue 😅

@shraddhabang
Copy link
Collaborator

shraddhabang commented Dec 30, 2024

@Timotej979 Will you be able to share which version of aws-lb-controller have you installed also if possible will you be able to share the logs for lb controller? I tried this with the latest version on lb controller. It seems to be working for me. Logs would be helpful to see if there are any errors.

@shraddhabang shraddhabang added kind/bug Categorizes issue or PR as related to a bug. triage/needs-information Indicates an issue needs more information in order to work on it. labels Dec 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Categorizes issue or PR as related to a bug. triage/needs-information Indicates an issue needs more information in order to work on it.
Projects
None yet
Development

No branches or pull requests

2 participants