Skip to content

Commit bfae47a

Browse files
authored
Mark weighted route destinations as deprecated in docs (#3779)
- The Istio Service Mesh routing layer is required for weighted routes to function, but development on it was discontinued in early 2020. See: https://github.com/cloudfoundry-attic/istio-release - As a consequence, weighted route destinations are no longer supported. - See this issue for more information about the removal of Istio support from Cloud Controller: #2942
1 parent bd76f71 commit bfae47a

File tree

3 files changed

+13
-11
lines changed

3 files changed

+13
-11
lines changed

docs/v3/source/includes/resources/routes/_destination_object.md.erb

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -18,11 +18,11 @@ Note that when using a custom port, the app process must be listening on the spe
1818

1919
If a destination does not specify a port, the default port depends on the app lifecycle type. For buildpack apps, traffic will be directed to port 8080. For Docker apps, the first port specified in the Dockerfile will be used.
2020

21-
| Name | Type | Description |
22-
| -------------- | ------------------------ | ------------------------------------------------------ |
23-
| **guid** | _uuid_ | Unique identifier for the destination |
24-
| **app.guid** | _uuid_ | Unique identifier for the app to route traffic to |
25-
| **app.process.type** | _string_ | Type of the process belonging to the app to route traffic to |
26-
| **port** | _integer_ | Port on the destination process to route traffic to
27-
| **weight** (*experimental*) | _integer_ or `null` | Percentage of traffic which will be routed to this destination; _this feature is only available via the BETA Service Mesh routing plane_ |
28-
| **protocol** | _string_ or `null` | Protocol to use for this destination. Valid protocols are `http1` or `http2` if route protocol is `http`, `tcp` if route protocol is `tcp`. A `null` value will set it either `http1` or `tcp` based on the route protocol; _this feature is only available if you turn it on in routing release_ |
21+
| Name | Type | Description |
22+
|---------------------------| ------------------------ |---------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------|
23+
| **guid** | _uuid_ | Unique identifier for the destination |
24+
| **app.guid** | _uuid_ | Unique identifier for the app to route traffic to |
25+
| **app.process.type** | _string_ | Type of the process belonging to the app to route traffic to |
26+
| **port** | _integer_ | Port on the destination process to route traffic to
27+
| **weight** (*deprecated*) | _integer_ or `null` | Percentage of traffic which will be routed to this destination. _**DEPRECATED:** this feature was only available via the experimental Service Mesh routing plane. It is no longer supported._ |
28+
| **protocol** | _string_ or `null` | Protocol to use for this destination. Valid protocols are `http1` or `http2` if route protocol is `http`, `tcp` if route protocol is `tcp`. A `null` value will set it either `http1` or `tcp` based on the route protocol; _this feature is only available if you turn it on in routing release_ |

docs/v3/source/includes/resources/routes/_insert_destinations.md.erb

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -43,8 +43,7 @@ Content-Type: application/json
4343

4444
Add one or more destinations to a route, preserving any existing destinations.
4545

46-
Note that weighted destinations cannot be added with this endpoint. To add weighted destinations,
47-
replace all destinations for a route at once using the [replace destinations endpoint](#replace-all-destinations-for-a-route).
46+
Weighted destinations (deprecated) cannot be added with this endpoint.
4847

4948
#### Definition
5049
`POST /v3/routes/:guid/destinations`

docs/v3/source/includes/resources/routes/_replace_destinations.md.erb

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -79,7 +79,10 @@ Content-Type: application/json
7979

8080
Replaces all destinations for a route, removing any destinations not included in the provided list.
8181

82-
If using weighted destinations, all destinations provided here must have a `weight` specified, and all weights
82+
Weighted destinations are **deprecated**. Development of the experimental Istio Service Mesh routing layer
83+
was discontinued in 2020 and is no longer supported by the platform. Specifying a `weight` for a destination will take no effect.
84+
85+
If weighted destinations are provided, however, all destinations provided here must have a `weight` specified, and all weights
8386
for this route must sum to 100. If not, all provided destinations must not have a `weight`. Mixing weighted
8487
and unweighted destinations for a route is not allowed.
8588

0 commit comments

Comments
 (0)