j2gg0s

> @j2gg0s: The following test **failed**, say `/retest` to rerun all failed tests or `/retest-required` to rerun all mandatory failed tests: > > Test name Commit Details Required Rerun command...

Any other thing than i need to fix?

Hi @x1nchen , Is my understanding correct? Do we need one pr?

> this seems related to #39521 Yes, it is related but not same. This PR don't change current logic and just fix obvious unexpected behavior.

> Yes, two PR is conflict But if we want to change Telemetry's AccessLogging behavior, Maybe it's better to define what we expected before coding. one Telemetry has many AccessLoging，one...

We configured the same as the previous example, but encountered the error. ```No artifact with ID 'gid-0' in group 'null' was found.; error code: 40401``` The message is writen by...

> Anyone taking this forward? @akashjain971 I switched to EnvoyFilters to manage logs, such as: ```yaml apiVersion: networking.istio.io/v1alpha3 kind: EnvoyFilter metadata: name: log namespace: istio-system spec: configPatches: - applyTo: NETWORK_FILTER...

> > For now the only way is `outlierDetection: {}`. I cannot promise 100%, but I would imagine this will not likely go away even if it was unintended originally....

Do you mean https://istio.io/latest/docs/reference/config/networking/destination-rule/#LocalityLoadBalancerSetting?

Current, we define ourself Library to replace the StdLib. ```go type filterLibrary struct { functions map[string]bool } func (filterLibrary) LibraryName() string { return "cel.j2gg0s.filter" } func (lib filterLibrary) CompileOptions() []cel.EnvOption...