Convert wasm to dual filter about envoy HOT 8 CLOSED

@mpwarres @alyssawilk

from envoy.

I'm not familar with the filter but we have documented steps on conversion here:
https://github.com/envoyproxy/envoy/blob/main/source/docs/upstream_filters.md
that filter owners can use to assess and convert!

from envoy.

Thanks @alyssawilk I try to convert it following your instructions but I see at the moment to get the clusterManager in this line https://github.com/envoyproxy/envoy/blob/main/source/extensions/filters/http/wasm/wasm_filter.cc#L26 it is not available and the assert broke here https://github.com/envoyproxy/envoy/blob/main/source/server/server.cc#L144

I'm creating the config as static. Not sure if the issue is that inserting the filter during the cluster initialization could be the reason this config is still not available.

What I did is to create a mock of the clusterManager to let me create the wasm filter and I could run a simple wasm. but I don't know if it is possible to use the real clusterManager. populate it later in wasm filter, of having a proxy object with lazy initialization. I guess this cluster manager is used later in the execution when the wasm needs to make a call, for example.

from envoy.

from envoy.

The workaround is not something we can use, I can upload a PR to make it easy understand what the problem is.

from envoy.

@alyssawilk If you have a momento to take a look #33119

from envoy.

This issue has been automatically marked as stale because it has not had activity in the last 30 days. It will be closed in the next 7 days unless it is tagged "help wanted" or "no stalebot" or other activity occurs. Thank you for your contributions.

from envoy.

This issue has been automatically closed because it has not had activity in the last 37 days. If this issue is still valid, please ping a maintainer and ask them to label it as "help wanted" or "no stalebot". Thank you for your contributions.

from envoy.