You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@apisix.apache.org by GitBox <gi...@apache.org> on 2022/05/17 08:23:30 UTC
[GitHub] [apisix] navendu-pottekkat opened a new pull request, #7064: docs: update "Traffic" Plugin docs 3
navendu-pottekkat opened a new pull request, #7064:
URL: https://github.com/apache/apisix/pull/7064
Signed-off-by: Navendu Pottekkat <na...@gmail.com>
### Description
Updates the documentation of the "traffic-split", "request-id", "proxy-control", and "client-control" Plugins.
Child PR of #6734
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix] navendu-pottekkat commented on a diff in pull request #7064: docs: update "Traffic" Plugin docs 3
Posted by GitBox <gi...@apache.org>.
navendu-pottekkat commented on code in PR #7064:
URL: https://github.com/apache/apisix/pull/7064#discussion_r877706739
##########
docs/en/latest/plugins/traffic-split.md:
##########
@@ -141,13 +152,27 @@ curl http://127.0.0.1:9080/apisix/admin/routes/1 -H 'X-API-KEY: edd1c9f034335f13
}'
```
->Note: **1.** Use the `upstream_id` to bind the defined upstream, it can reuse upstream health detection, retry and other functions. **2.** Support the two configuration methods of `upstream` and `upstream_id` to be used together.
+:::tip
+
+Configure via `upstream_id` to reuse Upstream's health detection, retires, and other functions.
+
+:::
+
+:::note
-## Example
+You can use both `upstream` configuration and `upstream_id` configuration together.
-### Grayscale Release
+:::
-The `match` rule part is missing, and the traffic is split according to the `weight` value configured by the `weighted_upstreams` in the plugin. Divide `plugin's upstream` and `route's upstream` according to the traffic ratio of 3:2, of which 60% of the traffic reaches the upstream of the `1981` port in the plugin, and 40% of the traffic reaches the default `1980` port on the route Upstream.
+## Example usage
+
+The examples below shows different use cases for using the `traffic-split` Plugin.
+
+### Grayscale/Canary release
Review Comment:
Okay, done!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix] navendu-pottekkat commented on pull request #7064: docs: update "Traffic" Plugin docs 3
Posted by GitBox <gi...@apache.org>.
navendu-pottekkat commented on PR #7064:
URL: https://github.com/apache/apisix/pull/7064#issuecomment-1130207055
@yzeng25 Made the changes as suggested.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix] navendu-pottekkat commented on pull request #7064: docs: update "Traffic" Plugin docs 3
Posted by GitBox <gi...@apache.org>.
navendu-pottekkat commented on PR #7064:
URL: https://github.com/apache/apisix/pull/7064#issuecomment-1128573518
// @hf400159 @kwanhur @avinal @yzeng25 @juzhiyuan Please review when you are available.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix] yzeng25 commented on a diff in pull request #7064: docs: update "Traffic" Plugin docs 3
Posted by GitBox <gi...@apache.org>.
yzeng25 commented on code in PR #7064:
URL: https://github.com/apache/apisix/pull/7064#discussion_r874652425
##########
docs/en/latest/plugins/proxy-control.md:
##########
@@ -23,19 +29,23 @@ title: proxy-control
## Description
-The `proxy-control` plugin dynamically controls the behavior of Nginx to proxy.
+The proxy-control Plugin dynamically controls the behavior of the Nginx proxy.
-**This plugin requires APISIX to run on [APISIX-Base](../FAQ.md#how-do-i-build-the-apisix-base-environment?).**
+:::info IMPORTANT
+
+This Plugin requires APISIX to run on APISIX-Base. See [apisix-build-tools](https://github.com/api7/apisix-build-tools) for more info.
+
+:::
## Attributes
-| Name | Type | Requirement | Default | Valid | Description |
-| --------- | ------------- | ----------- | ---------- | ------------------------------------------------------------------------ | --------------------------------------------------------------------------------------------------------------------------------------------------- |
-| request_buffering | boolean | optional | true | | dynamically set the `proxy_request_buffering` directive |
+| Name | Type | Required | Default | Description |
+| ----------------- | ------- | -------- | ------- | ------------------------------------------------------------------------------------------------------------------------------------------------------------- |
+| request_buffering | boolean | False | true | When set to true, dynamically sets the [proxy_request_buffering](https://nginx.org/en/docs/http/ngx_http_proxy_module.html#proxy_request_buffering) directive. |
Review Comment:
```suggestion
| request_buffering | boolean | False | true | When set to `true`, it dynamically sets the [proxy_request_buffering](https://nginx.org/en/docs/http/ngx_http_proxy_module.html#proxy_request_buffering) directive. |
```
##########
docs/en/latest/plugins/client-control.md:
##########
@@ -23,20 +29,23 @@ title: client-control
## Description
-The `client-control` plugin dynamically controls the behavior of Nginx to
-handle the client request.
+The `client-control` Plugin can be used to dynamically control the behavior of Nginx to handle a client request.
-**This plugin requires APISIX to run on [APISIX-Base](../FAQ.md#how-do-i-build-the-apisix-base-environment?).**
+:::info IMPORTANT
+
+This Plugin requires APISIX to run on APISIX-Base. See [apisix-build-tools](https://github.com/api7/apisix-build-tools) for more info.
+
+:::
## Attributes
-| Name | Type | Requirement | Default | Valid | Description |
-| --------- | ------------- | ----------- | ---------- | ------------------------------------------------------------------------ | --------------------------------------------------------------------------------------------------------------------------------------------------- |
-| max_body_size | integer | optional | | >= 0 | dynamically set the `client_max_body_size` directive |
+| Name | Type | Required | Valid values | Description |
+| ------------- | ------- | -------- | ------------ | ------------------------------------------------------------------------------------------------------------------------------------ |
+| max_body_size | integer | False | >= 0 | Dynamically set the [client_max_body_size](https://nginx.org/en/docs/http/ngx_http_core_module.html#client_max_body_size) directive. |
Review Comment:
```suggestion
| max_body_size | integer | False | [0,...] | Dynamically set the [client_max_body_size](https://nginx.org/en/docs/http/ngx_http_core_module.html#client_max_body_size) directive. |
```
##########
docs/en/latest/plugins/client-control.md:
##########
@@ -76,9 +87,7 @@ HTTP/1.1 413 Request Entity Too Large
## Disable Plugin
-When you want to disable this plugin, it is very simple,
-you can delete the corresponding json configuration in the plugin configuration,
-no need to restart the service, it will take effect immediately:
+To disable the `client-control` Plugin, you can delete the corresponding JSON configuration from the Plugin configuration. APISIX will automatically reload and you do not have to restart for this to take effect.
Review Comment:
```suggestion
To disable the `client-control` Plugin, you can delete the corresponding JSON configuration from the Plugin configuration. APISIX will automatically reload, and you do not have to restart for this to take effect.
```
##########
docs/en/latest/plugins/request-id.md:
##########
@@ -23,20 +29,60 @@ title: request-id
## Description
-`request-id` plugin adds a unique ID (UUID) to each request proxied through APISIX. This plugin can be used to track an
-API request. The plugin will not add a request id if the `header_name` is already present in the request.
+The `request-id` Plugin adds a unique ID to each request proxied through APISIX.
+
+This Plugin can be used to track API requests.
+
+:::note
+
+The Plugin will not add a unique ID if the request already has a header with the configured `header_name`.
+
+:::
## Attributes
-| Name | Type | Requirement | Default | Valid | Description |
-| ------------------- | ------- | ----------- | -------------- | ----- | -------------------------------------------------------------- |
-| header_name | string | optional | "X-Request-Id" | | Request ID header name |
-| include_in_response | boolean | optional | true | | Option to include the unique request ID in the response header |
-| algorithm | string | optional | "uuid" | ["uuid", "snowflake", "nanoid"] | ID generation algorithm |
+| Name | Type | Required | Default | Valid values | Description |
+| ------------------- | ------- | -------- | -------------- | ------------------------------- | ---------------------------------------------------------------------- |
+| header_name | string | False | "X-Request-Id" | | Header name for the unique request ID. |
+| include_in_response | boolean | False | true | | When set to `true`, adds the unique request ID in the response header. |
+| algorithm | string | False | "uuid" | ["uuid", "snowflake", "nanoid"] | Algorithm to use for generating the unique request ID. |
-## How To Enable
+### Using snowflake algorithm to generate unique ID
-Create a route and enable the request-id plugin on the route:
+To use the snowflake algorithm, you have to enable it first on your configuration file (`conf/config.yaml`):
+
+```yaml title="conf/config.yaml"
+plugin_attr:
+ request-id:
+ snowflake:
+ enable: true
+ snowflake_epoc: 1609459200000
+ data_machine_bits: 12
+ sequence_bits: 10
+ data_machine_ttl: 30
+ data_machine_interval: 10
+```
+
+:::warning
Review Comment:
I don't see "warning" listed on https://docusaurus.io/docs/markdown-features/admonitions, please specify which one you'd like to use. Personally I think `:::caution` should do the work.
##########
docs/en/latest/plugins/request-id.md:
##########
@@ -23,20 +29,60 @@ title: request-id
## Description
-`request-id` plugin adds a unique ID (UUID) to each request proxied through APISIX. This plugin can be used to track an
-API request. The plugin will not add a request id if the `header_name` is already present in the request.
+The `request-id` Plugin adds a unique ID to each request proxied through APISIX.
+
+This Plugin can be used to track API requests.
+
+:::note
+
+The Plugin will not add a unique ID if the request already has a header with the configured `header_name`.
+
+:::
## Attributes
-| Name | Type | Requirement | Default | Valid | Description |
-| ------------------- | ------- | ----------- | -------------- | ----- | -------------------------------------------------------------- |
-| header_name | string | optional | "X-Request-Id" | | Request ID header name |
-| include_in_response | boolean | optional | true | | Option to include the unique request ID in the response header |
-| algorithm | string | optional | "uuid" | ["uuid", "snowflake", "nanoid"] | ID generation algorithm |
+| Name | Type | Required | Default | Valid values | Description |
+| ------------------- | ------- | -------- | -------------- | ------------------------------- | ---------------------------------------------------------------------- |
+| header_name | string | False | "X-Request-Id" | | Header name for the unique request ID. |
+| include_in_response | boolean | False | true | | When set to `true`, adds the unique request ID in the response header. |
+| algorithm | string | False | "uuid" | ["uuid", "snowflake", "nanoid"] | Algorithm to use for generating the unique request ID. |
-## How To Enable
+### Using snowflake algorithm to generate unique ID
-Create a route and enable the request-id plugin on the route:
+To use the snowflake algorithm, you have to enable it first on your configuration file (`conf/config.yaml`):
+
+```yaml title="conf/config.yaml"
+plugin_attr:
+ request-id:
+ snowflake:
+ enable: true
+ snowflake_epoc: 1609459200000
+ data_machine_bits: 12
+ sequence_bits: 10
+ data_machine_ttl: 30
+ data_machine_interval: 10
+```
+
+:::warning
+
+Please read this documentation before deciding to use the snowflake algorithm. Once it is configured, you cannot arbitrarily change the configuration. Failure to do so may result in duplicate ID being generated.
Review Comment:
```suggestion
Please read this documentation before deciding to use the snowflake algorithm. Once it is configured, you cannot arbitrarily change the configuration. Failure to do so may result in duplicate IDs.
```
##########
docs/en/latest/plugins/request-id.md:
##########
@@ -23,20 +29,60 @@ title: request-id
## Description
-`request-id` plugin adds a unique ID (UUID) to each request proxied through APISIX. This plugin can be used to track an
-API request. The plugin will not add a request id if the `header_name` is already present in the request.
+The `request-id` Plugin adds a unique ID to each request proxied through APISIX.
+
+This Plugin can be used to track API requests.
+
+:::note
+
+The Plugin will not add a unique ID if the request already has a header with the configured `header_name`.
+
+:::
## Attributes
-| Name | Type | Requirement | Default | Valid | Description |
-| ------------------- | ------- | ----------- | -------------- | ----- | -------------------------------------------------------------- |
-| header_name | string | optional | "X-Request-Id" | | Request ID header name |
-| include_in_response | boolean | optional | true | | Option to include the unique request ID in the response header |
-| algorithm | string | optional | "uuid" | ["uuid", "snowflake", "nanoid"] | ID generation algorithm |
+| Name | Type | Required | Default | Valid values | Description |
+| ------------------- | ------- | -------- | -------------- | ------------------------------- | ---------------------------------------------------------------------- |
+| header_name | string | False | "X-Request-Id" | | Header name for the unique request ID. |
+| include_in_response | boolean | False | true | | When set to `true`, adds the unique request ID in the response header. |
+| algorithm | string | False | "uuid" | ["uuid", "snowflake", "nanoid"] | Algorithm to use for generating the unique request ID. |
-## How To Enable
+### Using snowflake algorithm to generate unique ID
-Create a route and enable the request-id plugin on the route:
+To use the snowflake algorithm, you have to enable it first on your configuration file (`conf/config.yaml`):
+
+```yaml title="conf/config.yaml"
+plugin_attr:
+ request-id:
+ snowflake:
+ enable: true
+ snowflake_epoc: 1609459200000
+ data_machine_bits: 12
+ sequence_bits: 10
+ data_machine_ttl: 30
+ data_machine_interval: 10
+```
+
+:::warning
Review Comment:
If you decide to stick with "waring", you can also do a trick like `:::caution Warning`.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix] hf400159 commented on a diff in pull request #7064: docs: update "Traffic" Plugin docs 3
Posted by GitBox <gi...@apache.org>.
hf400159 commented on code in PR #7064:
URL: https://github.com/apache/apisix/pull/7064#discussion_r874804570
##########
docs/en/latest/plugins/proxy-control.md:
##########
@@ -23,19 +29,23 @@ title: proxy-control
## Description
-The `proxy-control` plugin dynamically controls the behavior of Nginx to proxy.
+The proxy-control Plugin dynamically controls the behavior of the Nginx proxy.
Review Comment:
```suggestion
The proxy-control Plugin dynamically controls the behavior of the NGINX proxy.
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix] juzhiyuan commented on a diff in pull request #7064: docs: update "Traffic" Plugin docs 3
Posted by GitBox <gi...@apache.org>.
juzhiyuan commented on code in PR #7064:
URL: https://github.com/apache/apisix/pull/7064#discussion_r876878380
##########
docs/en/latest/plugins/traffic-split.md:
##########
@@ -141,13 +152,27 @@ curl http://127.0.0.1:9080/apisix/admin/routes/1 -H 'X-API-KEY: edd1c9f034335f13
}'
```
->Note: **1.** Use the `upstream_id` to bind the defined upstream, it can reuse upstream health detection, retry and other functions. **2.** Support the two configuration methods of `upstream` and `upstream_id` to be used together.
+:::tip
+
+Configure via `upstream_id` to reuse Upstream's health detection, retires, and other functions.
+
+:::
+
+:::note
-## Example
+You can use both `upstream` configuration and `upstream_id` configuration together.
-### Grayscale Release
+:::
-The `match` rule part is missing, and the traffic is split according to the `weight` value configured by the `weighted_upstreams` in the plugin. Divide `plugin's upstream` and `route's upstream` according to the traffic ratio of 3:2, of which 60% of the traffic reaches the upstream of the `1981` port in the plugin, and 40% of the traffic reaches the default `1980` port on the route Upstream.
+## Example usage
+
+The examples below shows different use cases for using the `traffic-split` Plugin.
+
+### Grayscale/Canary release
Review Comment:
Please use Canary Release instead
##########
docs/en/latest/plugins/traffic-split.md:
##########
@@ -141,13 +152,27 @@ curl http://127.0.0.1:9080/apisix/admin/routes/1 -H 'X-API-KEY: edd1c9f034335f13
}'
```
->Note: **1.** Use the `upstream_id` to bind the defined upstream, it can reuse upstream health detection, retry and other functions. **2.** Support the two configuration methods of `upstream` and `upstream_id` to be used together.
+:::tip
+
+Configure via `upstream_id` to reuse Upstream's health detection, retires, and other functions.
+
+:::
+
+:::note
-## Example
+You can use both `upstream` configuration and `upstream_id` configuration together.
-### Grayscale Release
+:::
-The `match` rule part is missing, and the traffic is split according to the `weight` value configured by the `weighted_upstreams` in the plugin. Divide `plugin's upstream` and `route's upstream` according to the traffic ratio of 3:2, of which 60% of the traffic reaches the upstream of the `1981` port in the plugin, and 40% of the traffic reaches the default `1980` port on the route Upstream.
+## Example usage
+
+The examples below shows different use cases for using the `traffic-split` Plugin.
+
+### Grayscale/Canary release
Review Comment:
Others LGTM
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix] navendu-pottekkat commented on a diff in pull request #7064: docs: update "Traffic" Plugin docs 3
Posted by GitBox <gi...@apache.org>.
navendu-pottekkat commented on code in PR #7064:
URL: https://github.com/apache/apisix/pull/7064#discussion_r876919261
##########
docs/en/latest/plugins/traffic-split.md:
##########
@@ -141,13 +152,27 @@ curl http://127.0.0.1:9080/apisix/admin/routes/1 -H 'X-API-KEY: edd1c9f034335f13
}'
```
->Note: **1.** Use the `upstream_id` to bind the defined upstream, it can reuse upstream health detection, retry and other functions. **2.** Support the two configuration methods of `upstream` and `upstream_id` to be used together.
+:::tip
+
+Configure via `upstream_id` to reuse Upstream's health detection, retires, and other functions.
+
+:::
+
+:::note
-## Example
+You can use both `upstream` configuration and `upstream_id` configuration together.
-### Grayscale Release
+:::
-The `match` rule part is missing, and the traffic is split according to the `weight` value configured by the `weighted_upstreams` in the plugin. Divide `plugin's upstream` and `route's upstream` according to the traffic ratio of 3:2, of which 60% of the traffic reaches the upstream of the `1981` port in the plugin, and 40% of the traffic reaches the default `1980` port on the route Upstream.
+## Example usage
+
+The examples below shows different use cases for using the `traffic-split` Plugin.
+
+### Grayscale/Canary release
Review Comment:
So no need for mentioning Grayscale or Canary?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix] navendu-pottekkat commented on a diff in pull request #7064: docs: update "Traffic" Plugin docs 3
Posted by GitBox <gi...@apache.org>.
navendu-pottekkat commented on code in PR #7064:
URL: https://github.com/apache/apisix/pull/7064#discussion_r876080313
##########
docs/en/latest/plugins/request-id.md:
##########
@@ -23,20 +29,60 @@ title: request-id
## Description
-`request-id` plugin adds a unique ID (UUID) to each request proxied through APISIX. This plugin can be used to track an
-API request. The plugin will not add a request id if the `header_name` is already present in the request.
+The `request-id` Plugin adds a unique ID to each request proxied through APISIX.
+
+This Plugin can be used to track API requests.
+
+:::note
+
+The Plugin will not add a unique ID if the request already has a header with the configured `header_name`.
+
+:::
## Attributes
-| Name | Type | Requirement | Default | Valid | Description |
-| ------------------- | ------- | ----------- | -------------- | ----- | -------------------------------------------------------------- |
-| header_name | string | optional | "X-Request-Id" | | Request ID header name |
-| include_in_response | boolean | optional | true | | Option to include the unique request ID in the response header |
-| algorithm | string | optional | "uuid" | ["uuid", "snowflake", "nanoid"] | ID generation algorithm |
+| Name | Type | Required | Default | Valid values | Description |
+| ------------------- | ------- | -------- | -------------- | ------------------------------- | ---------------------------------------------------------------------- |
+| header_name | string | False | "X-Request-Id" | | Header name for the unique request ID. |
+| include_in_response | boolean | False | true | | When set to `true`, adds the unique request ID in the response header. |
+| algorithm | string | False | "uuid" | ["uuid", "snowflake", "nanoid"] | Algorithm to use for generating the unique request ID. |
-## How To Enable
+### Using snowflake algorithm to generate unique ID
-Create a route and enable the request-id plugin on the route:
+To use the snowflake algorithm, you have to enable it first on your configuration file (`conf/config.yaml`):
+
+```yaml title="conf/config.yaml"
+plugin_attr:
+ request-id:
+ snowflake:
+ enable: true
+ snowflake_epoc: 1609459200000
+ data_machine_bits: 12
+ sequence_bits: 10
+ data_machine_ttl: 30
+ data_machine_interval: 10
+```
+
+:::warning
Review Comment:
Yes, it was supposed to be :::caution WARNING. I missed it. Thank you for pointing it out.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix] juzhiyuan merged pull request #7064: docs: update "Traffic" Plugin docs 3
Posted by GitBox <gi...@apache.org>.
juzhiyuan merged PR #7064:
URL: https://github.com/apache/apisix/pull/7064
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix] juzhiyuan commented on a diff in pull request #7064: docs: update "Traffic" Plugin docs 3
Posted by GitBox <gi...@apache.org>.
juzhiyuan commented on code in PR #7064:
URL: https://github.com/apache/apisix/pull/7064#discussion_r877677769
##########
docs/en/latest/plugins/traffic-split.md:
##########
@@ -141,13 +152,27 @@ curl http://127.0.0.1:9080/apisix/admin/routes/1 -H 'X-API-KEY: edd1c9f034335f13
}'
```
->Note: **1.** Use the `upstream_id` to bind the defined upstream, it can reuse upstream health detection, retry and other functions. **2.** Support the two configuration methods of `upstream` and `upstream_id` to be used together.
+:::tip
+
+Configure via `upstream_id` to reuse Upstream's health detection, retires, and other functions.
+
+:::
+
+:::note
-## Example
+You can use both `upstream` configuration and `upstream_id` configuration together.
-### Grayscale Release
+:::
-The `match` rule part is missing, and the traffic is split according to the `weight` value configured by the `weighted_upstreams` in the plugin. Divide `plugin's upstream` and `route's upstream` according to the traffic ratio of 3:2, of which 60% of the traffic reaches the upstream of the `1981` port in the plugin, and 40% of the traffic reaches the default `1980` port on the route Upstream.
+## Example usage
+
+The examples below shows different use cases for using the `traffic-split` Plugin.
+
+### Grayscale/Canary release
Review Comment:
Yes, only using the Canary release is enough
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix] hf400159 commented on a diff in pull request #7064: docs: update "Traffic" Plugin docs 3
Posted by GitBox <gi...@apache.org>.
hf400159 commented on code in PR #7064:
URL: https://github.com/apache/apisix/pull/7064#discussion_r874809335
##########
docs/en/latest/plugins/proxy-control.md:
##########
@@ -23,19 +29,23 @@ title: proxy-control
## Description
-The `proxy-control` plugin dynamically controls the behavior of Nginx to proxy.
+The proxy-control Plugin dynamically controls the behavior of the Nginx proxy.
Review Comment:
I found on the [NGINX official website](https://www.nginx.com/), it should be `NGINX`.
##########
docs/en/latest/plugins/proxy-control.md:
##########
@@ -23,19 +29,23 @@ title: proxy-control
## Description
-The `proxy-control` plugin dynamically controls the behavior of Nginx to proxy.
+The proxy-control Plugin dynamically controls the behavior of the Nginx proxy.
Review Comment:
```suggestion
The proxy-control Plugin dynamically controls the behavior of the NGINX proxy.
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix] navendu-pottekkat commented on a diff in pull request #7064: docs: update "Traffic" Plugin docs 3
Posted by GitBox <gi...@apache.org>.
navendu-pottekkat commented on code in PR #7064:
URL: https://github.com/apache/apisix/pull/7064#discussion_r875050118
##########
docs/en/latest/plugins/proxy-control.md:
##########
@@ -23,19 +29,23 @@ title: proxy-control
## Description
-The `proxy-control` plugin dynamically controls the behavior of Nginx to proxy.
+The proxy-control Plugin dynamically controls the behavior of the Nginx proxy.
Review Comment:
I guess we started using Nginx and kept using it. We might need to change this everywhere if this is desired.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org