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

ICMP errors generated by tracked flows treated as related traffic #2247

Merged
merged 21 commits into from Apr 21, 2020

Commits on Apr 15, 2020

  1. Configuration menu
    Copy the full SHA
    b63ddac View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    ee18268 View commit details
    Browse the repository at this point in the history
  3. bpf: icmp_skb_get_hdr() returns the icmp header

    Can be reused in other locations.
    tomastigera committed Apr 15, 2020
    Configuration menu
    Copy the full SHA
    a684d88 View commit details
    Browse the repository at this point in the history
  4. bpf: icmp related for NAT

    tomastigera committed Apr 15, 2020
    Configuration menu
    Copy the full SHA
    1c8652b View commit details
    Browse the repository at this point in the history
  5. bpf: ut for NAT related from the host

    This is a prerequisite for letting host to handle TTL exceeded.
    tomastigera committed Apr 15, 2020
    Configuration menu
    Copy the full SHA
    c49f205 View commit details
    Browse the repository at this point in the history
  6. fv: allow creating inactive workloads

    Sets up endpoints, routes etc, but does not run
    tomastigera committed Apr 15, 2020
    Configuration menu
    Copy the full SHA
    290081d View commit details
    Browse the repository at this point in the history
  7. fv: allow to start inactive workload

    to complement the fact that we can create an inactive workload
    tomastigera committed Apr 15, 2020
    Configuration menu
    Copy the full SHA
    57cf27a View commit details
    Browse the repository at this point in the history
  8. Configuration menu
    Copy the full SHA
    910c0f6 View commit details
    Browse the repository at this point in the history
  9. bpf/fv: fix the IP size in makeICMPError() and test ports

    Check that the ports are fixed up after NATing the ICMP related back
    tomastigera committed Apr 15, 2020
    Configuration menu
    Copy the full SHA
    7fc04a8 View commit details
    Browse the repository at this point in the history
  10. Configuration menu
    Copy the full SHA
    b483d5e View commit details
    Browse the repository at this point in the history
  11. bpf: tunneling from host and csum of icmp related

    Needs to update csum in the inner IP header instead of the outer.
    
    When icmp generated by the next hop node, it needs to be placed in the
    tunnel.
    tomastigera committed Apr 15, 2020
    Configuration menu
    Copy the full SHA
    cbf1998 View commit details
    Browse the repository at this point in the history

Commits on Apr 16, 2020

  1. fv: tcpdump fails test if it never listened

    This is triggered by missing tcpdump or bad filter
    tomastigera committed Apr 16, 2020
    Configuration menu
    Copy the full SHA
    8739e6f View commit details
    Browse the repository at this point in the history
  2. fv: tcpdump for containers without tcpdump installed

    only felix-test image has tcpdump installed, extrnal client has not
    tomastigera committed Apr 16, 2020
    Configuration menu
    Copy the full SHA
    43c8738 View commit details
    Browse the repository at this point in the history
  3. bpf: nodeports and icmp related

    if SNAT, we also need to fix up the source in the outer IP
    
    FV tests when ICMP is returned from host and from the backing workload
    tomastigera committed Apr 16, 2020
    Configuration menu
    Copy the full SHA
    5dc2d48 View commit details
    Browse the repository at this point in the history
  4. Configuration menu
    Copy the full SHA
    65314c5 View commit details
    Browse the repository at this point in the history
  5. bpf: SNAT of outer IP only if returning to outer client

    When we generate an ICMP in a workload or at a host in response to
    traffic that originated through a NP tunnel ammend the outer source IP
    as if the reponse was from the original node as all the rest is internal
    to the cluster.
    tomastigera committed Apr 16, 2020
    Configuration menu
    Copy the full SHA
    e4759c1 View commit details
    Browse the repository at this point in the history
  6. Configuration menu
    Copy the full SHA
    14b7721 View commit details
    Browse the repository at this point in the history
  7. Configuration menu
    Copy the full SHA
    5c6a24e View commit details
    Browse the repository at this point in the history

Commits on Apr 17, 2020

  1. Configuration menu
    Copy the full SHA
    7cbe463 View commit details
    Browse the repository at this point in the history
  2. bpf: icmp related retunign from tunnel is fwd approved

    We need to treat related ICMP from tunnel the same way as we do the
    original traffic, otherwise it would go through the conntrack on the way
    out and would create ICMP tracking record.
    tomastigera committed Apr 17, 2020
    Configuration menu
    Copy the full SHA
    aa28d96 View commit details
    Browse the repository at this point in the history

Commits on Apr 20, 2020

  1. Configuration menu
    Copy the full SHA
    0364cf6 View commit details
    Browse the repository at this point in the history