Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

udp: support udp proxy to use original_dst_cluster #26285

Closed
skiptomyliu opened this issue Mar 23, 2023 · 3 comments
Closed

udp: support udp proxy to use original_dst_cluster #26285

skiptomyliu opened this issue Mar 23, 2023 · 3 comments
Labels
enhancement Feature requests. Not bugs or questions. stale stalebot believes this issue/PR has not been touched recently

Comments

@skiptomyliu
Copy link
Contributor

skiptomyliu commented Mar 23, 2023

Title: support udp proxy to use original_dst_cluster

Description:
Looking to forward UDP traffic that has been redirected via iptables to use the original_dst_cluster, identical to how TCP is supported via the original_dst filter

Attempted the following config but looks like UDP only supports a single listener filter, so seems like we need to extend the udp_proxy listener similarly what was done with use_original_src_ip [pr]

error adding listener '0.0.0.0:10005': Only 1 UDP listener filter per listener supported

admin:
  access_log_path: admin_access.log
  address:
    socket_address:
      address: 127.0.0.1
      protocol: TCP
      port_value: 9901

static_resources:
  listeners:
    - name: original_dst_cluster_listener
      address:
        socket_address:
          protocol: UDP
          address: 0.0.0.0
          port_value: 10005
      listener_filters:
      - name: envoy.filters.listener.original_dst
        typed_config:
            "@type": "type.googleapis.com/envoy.extensions.filters.listener.original_dst.v3.OriginalDst"
      - name: envoy.filters.udp_listener.udp_proxy
        typed_config:
            "@type": "type.googleapis.com/envoy.extensions.filters.udp.udp_proxy.v3.UdpProxyConfig"
            stat_prefix: udp_egress
            cluster: original_destination_cluster
  clusters:
  - name: original_dst_cluster
    connect_timeout: 5s
    lb_policy: CLUSTER_PROVIDED
    type: ORIGINAL_DST
    dns_lookup_family: V4_ONLY
@skiptomyliu skiptomyliu added enhancement Feature requests. Not bugs or questions. triage Issue requires triage labels Mar 23, 2023
@alyssawilk alyssawilk removed the triage Issue requires triage label Mar 24, 2023
@github-actions
Copy link

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.

@github-actions github-actions bot added the stale stalebot believes this issue/PR has not been touched recently label Apr 23, 2023
@github-actions
Copy link

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.

@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Apr 30, 2023
@ohadvano
Copy link
Contributor

It looks like support for multiple UDP filters was added here: #18573
What version are you using?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Feature requests. Not bugs or questions. stale stalebot believes this issue/PR has not been touched recently
Projects
None yet
Development

No branches or pull requests

3 participants