Closed varkey98 closed 4 months ago
Observability mode is still WiP, @tyxia
Thank you @htuch for the prompt reply :)
@varkey98 Hi. Thanks for trying it out.
The actual "send and go" mode are recently added in https://github.com/envoyproxy/envoy/pull/34392 and https://github.com/envoyproxy/envoy/pull/34491
It seems that your Envoy binary is release in April 18th which doesn't include actual implementation yet.
Thanks @tyxia, let me try building it from main. Would there be a release soon with this feature?
SG, let me know how is your testing going with actual ob11y support.
Regarding the release: It will be included in upcoming release https://www.envoyproxy.io/docs/envoy/latest/version_history/v1.31/v1.31.0
I think the docs need an update. The docs say that everything except async
mode is implemented and there's no mention about observability mode being the new async mode. Also in the configuratio, observability mode is mentioned without specifying that its not implemented yet. I'm guessing this would've made many like me try this out without realising it wasn't implemented
@varkey98 https://github.com/envoyproxy/envoy/pull/34996 fix the release note that saying observability mode deprecate the async mode.
If you still have issues with latest Envoy image that enables ob11y mode, please feel free to reopen
Sure, thank you
External Processing filter still waits for response from the server when observability_mode is set to true
Description: According to the docs, when observability_mode is turned on, the filter shouldnt be waiting for responses from the server as the requests are just "Send and go". I've added a sleep in my server impl, and even with this mode turned on, my client gets the response only after my ext_proc server receives all parts of the request.
Repro steps: Running on Mac OS Sonoma 14.5 Envoy Version: 816188b86a0a52095b116b107f576324082c7c02/1.30.1/Distribution/RELEASE/BoringSSL Any server implementation of ext_proc filter which has a delay added and with observability_mode set to true will have this issue. Will upload my code somewhere and link it here for reference
Admin and Stats Output:
Config:
Logs: