Skip to content

Commit

Permalink
Allow Backend Weight to be 0 (#1396)
Browse files Browse the repository at this point in the history
- Update parser/haproxy-templates
- Update docs

Fixes #1387
  • Loading branch information
kfoozminus authored and tamalsaha committed Jul 17, 2019
1 parent b6167d6 commit e61316e
Show file tree
Hide file tree
Showing 4 changed files with 3 additions and 49 deletions.
1 change: 0 additions & 1 deletion docs/guides/ingress/configuration/agent-check.md
Expand Up @@ -107,4 +107,3 @@ numeric value. Supported units are :
- m : minutes. 1m = 60s = 60000ms
- h : hours. 1h = 60m = 3600s = 3600000ms
- d : days. 1d = 24h = 1440m = 86400s = 86400000ms

2 changes: 1 addition & 1 deletion docs/guides/ingress/configuration/annotations.md
Expand Up @@ -43,7 +43,7 @@ Below is the full list of supported annotations:
| [ingress.appscode.com/auth-tls-verify-client](/docs/guides/ingress/security/tls-auth.md) | `required` or, `optional` | `required` |
| [ingress.appscode.com/backend-tls](/docs/guides/ingress/tls/backend-tls.md) | string | |
| [ingress.appscode.com/replicas](/docs/guides/ingress/scaling.md) | int | `1` |
| [ingress.appscode.com/backend-weight](/docs/guides/ingress/http/blue-green-deployment.md) | int | |
| [ingress.appscode.com/backend-weight](/docs/guides/ingress/http/blue-green-deployment.md) | int | 1 |
| [ingress.appscode.com/whitelist-source-range](/docs/guides/ingress/configuration/whitelist.md) | string | |
| [ingress.appscode.com/max-connections](/docs/guides/ingress/configuration/max-connections.md) | int | |
| [ingress.appscode.com/ssl-redirect](/docs/guides/ingress/configuration/ssl-redirect.md) | bool | `true` |
Expand Down
2 changes: 2 additions & 0 deletions docs/guides/ingress/http/blue-green-deployment.md
Expand Up @@ -127,3 +127,5 @@ spec:
servicePort: 80
path: /testpath
```

The default [weight](http://cbonte.github.io/haproxy-dconv/1.8/configuration.html#weight) is 1, and the maximal value is 256. A value of 0 means the server will not participate in load-balancing but will still accept persistent connections.

0 comments on commit e61316e

Please sign in to comment.