fix: consistent defaults and improved values.yaml comments

- Change virtualService.enabled default from true to false for consistency
  with gateway.enabled and destinationRule.enabled (all default to false)
- Replace misleading '# http: []' comment with a meaningful example showing
  a custom HTTP route rule with match/route/destination structure
- Update README.md table to reflect the new virtualService.enabled default
This commit is contained in:
StyleTang 2026-03-13 16:38:31 +08:00
parent b7afd3dad6
commit 952f368af4
2 changed files with 12 additions and 4 deletions

View file

@ -177,7 +177,7 @@ Use these **instead of** `ingress` when your cluster runs Istio. Uses `networkin
| `istio.gateway.tls.maxProtocolVersion` | Maximum TLS protocol version (e.g. `TLSV1_3`) | `""` |
| `istio.gateway.labels` | Additional labels for the Gateway resource | `{}` |
| `istio.gateway.annotations` | Additional annotations for the Gateway resource | `{}` |
| `istio.virtualService.enabled` | Create a VirtualService resource | `true` |
| `istio.virtualService.enabled` | Create a VirtualService resource | `false` |
| `istio.virtualService.gateways` | Additional gateway references (e.g. `["istio-system/shared-gateway"]`). The chart's own Gateway is auto-included | `[]` |
| `istio.virtualService.http` | Custom HTTP route rules. When set, overrides the default catch-all route | `[]` |
| `istio.virtualService.timeout` | Request timeout for the default route (e.g. `"30s"`) | `""` |

View file

@ -168,7 +168,7 @@ istio:
# Maximum TLS protocol version
# maxProtocolVersion: TLSV1_3
virtualService:
enabled: true
enabled: false
labels: {}
annotations: {}
# Additional gateways to attach to (e.g. use an existing shared gateway)
@ -177,8 +177,16 @@ istio:
# # Example: use an existing shared gateway instead of creating one
# # gateways:
# # - istio-system/shared-gateway
# Custom HTTP route rules (overrides the default catch-all route)
# http: []
# Custom HTTP route rules (overrides the default catch-all route when non-empty)
# http:
# - match:
# - uri:
# prefix: /v1
# route:
# - destination:
# host: my-litellm-backend
# port:
# number: 4000
# Timeout for the default route (e.g. "30s")
# timeout: "30s"
# Retry policy for the default route