Skip to content

Commit

Permalink
docs: note that powerOfRandomNChoices does not support fadeIn (#2474)
Browse files Browse the repository at this point in the history
Signed-off-by: Alexander Yastrebov <alexander.yastrebov@zalando.de>
  • Loading branch information
AlexanderYastrebov authored Jul 18, 2023
1 parent 6c1d068 commit a8e6dd2
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 1 deletion.
2 changes: 2 additions & 0 deletions docs/reference/backends.md
Original file line number Diff line number Diff line change
Expand Up @@ -261,6 +261,8 @@ Current implemented algorithms:
- `powerOfRandomNChoices`: backend is chosen by powerOfRandomNChoices algorithm with selecting N random endpoints and picking the one with least outstanding requests from them. (http://www.eecs.harvard.edu/~michaelm/postscripts/handbook2001.pdf)
- __TODO__: https://github.com/zalando/skipper/issues/557
All algorithms except `powerOfRandomNChoices` support [fadeIn](filters.md#fadein) filter.
Route example with 2 backends and the `roundRobin` algorithm:
```
r0: * -> <roundRobin, "http://127.0.0.1:9998", "http://127.0.0.1:9997">;
Expand Down
3 changes: 2 additions & 1 deletion docs/reference/filters.md
Original file line number Diff line number Diff line change
Expand Up @@ -2903,7 +2903,8 @@ Some filters influence how load balancing will be done
### fadeIn
When this filter is set, and the route has a load balanced backend, then the newly added endpoints will receive
When this filter is set, and the route has a load balanced backend using [supported algorithm](backends.md#load-balancer-backend),
then the newly added endpoints will receive
the traffic in a gradually increasing way, starting from their detection for the specified duration, after which
they receive equal amount traffic as the previously existing routes. The detection time of an load balanced
backend endpoint is preserved over multiple generations of the route configuration (over route changes). This
Expand Down

0 comments on commit a8e6dd2

Please sign in to comment.