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

Network Load Balancers won't reconcile in main #4790

Closed
nrb opened this issue Feb 15, 2024 · 2 comments · Fixed by #4794
Closed

Network Load Balancers won't reconcile in main #4790

nrb opened this issue Feb 15, 2024 · 2 comments · Fixed by #4794
Labels
kind/bug Categorizes issue or PR as related to a bug. priority/critical-urgent Highest priority. Must be actively worked on as someone's top priority right now. triage/accepted Indicates an issue or PR is ready to be actively worked on.
Milestone

Comments

@nrb
Copy link
Contributor

nrb commented Feb 15, 2024

/kind bug

What steps did you take and what happened:
[A clear and concise description of what the bug is.]

Upgrading from a version of CAPA prior to adding #4733 causes network load balancers to fail reconciliation.

E0212 15:21:48.220015       1 awscluster_controller.go:280] "failed to reconcile load balancer" err=<
        failed to apply security groups to load balancer "aws-us-cw-mf-test-1": InvalidConfigurationRequest: You cannot set security groups on a Network Load Balancer which was created without any security groups.
                status code: 400, request id: 05010ea1-3b8f-4c6c-9674-2f785030006e
 > controller="awscluster" controllerGroup="infrastructure.cluster.x-k8s.io" controllerKind="AWSCluster" AWSCluster="system-clusters/aws-us-cw-mf-test-1-66z8p" namespace="system-clusters" name="aws-us-cw-mf-test-1-66z8p" reconcileID="8e92c000-83ee-411e-a9d1-6aa72495b6c8" cluster="system-clusters/aws-us-cw-mf-test-1"

What did you expect to happen:

Network load balancers previously created shouldn't fail to reconcile.

Anything else you would like to add:

The network load balancer documentation states that an NLB created without security groups cannot have security groups added or removed later.

Environment:

@k8s-ci-robot k8s-ci-robot added kind/bug Categorizes issue or PR as related to a bug. needs-priority needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. labels Feb 15, 2024
@nrb
Copy link
Contributor Author

nrb commented Feb 15, 2024

/triage accepted
/priority critical-urgent

@k8s-ci-robot k8s-ci-robot added triage/accepted Indicates an issue or PR is ready to be actively worked on. priority/critical-urgent Highest priority. Must be actively worked on as someone's top priority right now. and removed needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. needs-priority labels Feb 15, 2024
@nrb nrb changed the title Network Load Balancers created prior to v2.4.0 won't reconcile Network Load Balancers won't reconcile in main Feb 15, 2024
@richardcase
Copy link
Member

/milestone v2.4.0

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. priority/critical-urgent Highest priority. Must be actively worked on as someone's top priority right now. triage/accepted Indicates an issue or PR is ready to be actively worked on.
Projects
None yet
3 participants