wu-sheng commented on a change in pull request #6322:
URL: https://github.com/apache/skywalking/pull/6322#discussion_r585292260



##########
File path: docs/en/setup/envoy/als_setting.md
##########
@@ -87,3 +88,20 @@ this analyzer requires Istio to enable the metadata exchange 
plugin (you can ena
 or if you're using Istio 1.7+ and installing it with profile `demo`/`preview`, 
it should be enabled then).
 
 The 
[blog](https://skywalking.apache.org/blog/obs-service-mesh-vm-with-sw-and-als/) 
illustrates the detail of how it works, and a step-by-step tutorial to apply it 
into the [Online 
Boutique](https://github.com/GoogleCloudPlatform/microservices-demo) system.
+
+### `persistence`
+
+`persistence` analyzer adapts the Envoy access log format to
+SkyWalking's [native log 
format](https://github.com/apache/skywalking-data-collect-protocol/blob/master/logging/Logging.proto)
+, and forwards the formatted logs to [LAL](../../concepts-and-designs/lal.md), 
where you can configure persistent
+conditions, such as `sampler`, only persist error logs, etc. SkyWalking 
provides a default configuration
+file 
[`envoy-als.yaml`](../../../../oap-server/server-bootstrap/src/main/resources/lal/envoy-als.yaml)
 that you can
+adjust as per your needs. Please make sure to activate this rule via adding 
the rule name `envoy-als`
+into config item `log-analyzer/default/lalFiles` (or environment variable 
`SW_LOG_LAL_FILES`,
+e.g. `SW_LOG_LAL_FILES=envoy-als`).
+
+**Attention**: because `persistence` analyzer also needs a mechanism to map 
the logs into responding services, hence,
+you need to configure at least one of `k8s-mesh` or `mx-mesh` as its 
antecedent so that `persistence` analyzer knows
+which service the logs belong to. For example, you should set 
`envoy-metric/default/alsHTTPAnalysis` (or environment
+variable `SW_ENVOY_METRIC_ALS_HTTP_ANALYSIS`) to something like 
`k8s-mesh,persistence`, `mx-mesh,persistence`
+or `k8s-mesh,mx-mesh,persistence`.

Review comment:
       ```suggestion
   or `mx-mesh,k8s-mesh,persistence`.
   ```
   I think this is a recommended order if you add both analyzers?




----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to