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

SurgicalPool has no load balancing algorithm leading to "invalid-pool-load-balancing-algorithm" error #458

Open
Dariquest opened this issue Dec 13, 2024 · 0 comments
Labels

Comments

@Dariquest
Copy link
Contributor

Dariquest commented Dec 13, 2024

Current behavior

LoadBalancing Algorithm was introduced for the RoutePool in the #428.
The load balancing algorithm was not passed to the SurgicalPool (included in the RouteOpts for the initialization).

Missing load balancing algorithm in the surgical pool leads to the error "invalid-pool-load-balancing-algorithm".
This error is intended validation for a pool load balancing algorithm.
An additional log parameter needs to be added for this error for error research simplification.

Desired behavior

The load balancing algorithm is also needed to be passed to the SurgicalPool (included in the RouteOpts for the initialization).

Reproduction Steps

When I curl an app with an x-cf-app-instance header

curl -v "https://example-app.com/" -H "x-cf-app-instance: app_id:instance_id"

The same curl without the instance header works properly.

Impact

The custom per-route load balancing algorithm feature is not finally released.
This error has no functional impact in routing behaviour.

Affected Version

#428
The error occurs since routing-release version

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Development

No branches or pull requests

1 participant