Skip to content

Commit e971a8d

Browse files
dprotasoyoungnick
andauthored
recommend 503 status code for a service with no healthy endpoints (#3121)
* recommend 503 status code for a service with no healthy endpoints * MAY=>SHOULD * Update apis/v1/httproute_types.go Co-authored-by: Nick Young <[email protected]> * run codegen * Update apis/v1/httproute_types.go Co-authored-by: Nick Young <[email protected]> * run codegen --------- Co-authored-by: Nick Young <[email protected]>
1 parent cb5bf15 commit e971a8d

File tree

4 files changed

+30
-1
lines changed

4 files changed

+30
-1
lines changed

apis/v1/httproute_types.go

+5
Original file line numberDiff line numberDiff line change
@@ -263,6 +263,11 @@ type HTTPRouteRule struct {
263263
// invalid, 50 percent of traffic must receive a 500. Implementations may
264264
// choose how that 50 percent is determined.
265265
//
266+
// When a HTTPBackendRef refers to a Service that has no ready endpoints,
267+
// implementations SHOULD return a 503 for requests to that backend instead.
268+
// If an implementation chooses to do this, all of the above rules for 500 responses
269+
// MUST also apply for responses that return a 503.
270+
//
266271
// Support: Core for Kubernetes Service
267272
//
268273
// Support: Extended for Kubernetes ServiceImport

config/crd/experimental/gateway.networking.k8s.io_httproutes.yaml

+12
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

config/crd/standard/gateway.networking.k8s.io_httproutes.yaml

+12
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

pkg/generated/openapi/zz_generated.openapi.go

+1-1
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)