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

Configuration weight RR (1/4) #905

Merged
merged 1 commit into from
Jun 14, 2022
Merged

Configuration weight RR (1/4) #905

merged 1 commit into from
Jun 14, 2022

Conversation

kuritka
Copy link
Collaborator

@kuritka kuritka commented Jun 13, 2022

Related to #50
The weight (optional field) configuration goes into the GSLB strategy. This PR only aims to load and validate the weight configuration, which is specified in percentages. The weight distribution must be available to all clusters, so the weight field contains the weights of all regions.

  strategy:
    type: roundRobin # Use a round robin load balancing strategy, when deciding which downstream clusters to route clients too
    weight:
      us: 60%
      eu: 15%
      za: 25%

gslb.spec.strategy.weight is not allowed for failover and geoip strategies. If it is not specified in roundRobin,
then it is not a weight RoundRobin but classic one.

Signed-off-by: kuritka [email protected]

@kuritka kuritka changed the title Wrr 1a ep Make weights 1:N in GSLB configuration Jun 13, 2022
Related to #50
The weight (optional field) configuration goes into the GSLB strategy. This PR only aims
to load and validate the weight configuration, which is specified in percentages.
The weight distribution must be available to all clusters, so the weight field contains the weights of all regions.

```yaml
  strategy:
    type: roundRobin # Use a round robin load balancing strategy, when deciding which downstream clusters to route clients too
    weight:
      us: 60%
      eu: 15%
      za: 25%
```
gslb.spec.strategy.weight is not allowed for failover and geoip strategies. If it is not specified in roundRobin,
then it is not a weight RoundRobin but classic one.

Signed-off-by: kuritka <[email protected]>
@kuritka kuritka marked this pull request as ready for review June 14, 2022 08:28
@kuritka kuritka changed the title Make weights 1:N in GSLB configuration Configuration weight RR (1/4) Jun 14, 2022
Copy link
Collaborator

@k0da k0da left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@kuritka kuritka merged commit 793c486 into master Jun 14, 2022
@kuritka kuritka deleted the wrr-1a-ep branch June 14, 2022 12:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants