mbox series

[ovs-dev,v6,0/3] DHCP Relay Agent support for overlay subnets.

Message ID 20240424095607.129155-1-naveen.yerramneni@nutanix.com
Headers show
Series DHCP Relay Agent support for overlay subnets. | expand

Message

Naveen Yerramneni April 24, 2024, 9:56 a.m. UTC 
This patch contains changes to enable DHCP Relay Agent support for overlay subnets.

    USE CASE:
    ----------
      - Enable IP address assignment for overlay subnets from the centralized DHCP server present in the underlay network.

    PREREQUISITES
    --------------
      - Logical Router Port IP should be assigned (statically) from the same overlay subnet which is managed by DHCP server.
      - LRP IP is used for GIADRR field when relaying the DHCP packets and also same IP needs to be configured as default gateway for the overlay subnet.
      - Overlay subnets managed by external DHCP server are expected to be directly reachable from the underlay network.

    EXPECTED PACKET FLOW:
    ----------------------
    Following is the expected packet flow inorder to support DHCP rleay functionality in OVN.
      1. DHCP client originates DHCP discovery (broadcast).
      2. DHCP relay (running on the OVN) receives the broadcast and forwards the packet to the DHCP server by converting it to unicast.
         While forwarding the packet, it updates the GIADDR in DHCP header to its interface IP on which DHCP packet is received and increments hop count.
      3. DHCP server uses GIADDR field to decide the IP address pool from which IP has to be assigned and DHCP offer is sent to the same IP (GIADDR).
      4. DHCP relay agent forwards the offer to the client.
      5. DHCP client sends DHCP request (broadcast) packet.
      6. DHCP relay (running on the OVN) receives the broadcast and forwards the packet to the DHCP server by converting it to unicast.
         While forwarding the packet, it updates the GIADDR in DHCP header to its interface IP on which DHCP packet is received.
      7. DHCP Server sends the ACK packet.
      8. DHCP relay agent forwards the ACK packet to the client.
      9. All the future renew/release packets are directly exchanged between DHCP client and DHCP server.

    OVN DHCP RELAY PACKET FLOW:
    ----------------------------
    To add DHCP Relay support on OVN, we need to replicate all the behavior described above using distributed logical switch and logical router.
    At, highlevel packet flow is distributed among Logical Switch and Logical Router on source node (where VM is deployed) and redirect chassis(RC) node.
      1. Request packet gets processed on the source node where VM is deployed and relays the packet to DHCP server.
      2. Response packet is first processed on RC node (which first recieves the packet from underlay network). RC node forwards the packet to the right node by filling in the dest MAC and IP.

    OVN Packet flow with DHCP relay is explained below.
      1. DHCP client (VM) sends the DHCP discover packet (broadcast).
      2. Logical switch converts the packet to L2 unicast by setting the destination MAC to LRP's MAC
      3. Logical Router receives the packet and redirects it to the OVN controller.
      4. OVN controller updates the required information(GIADDR, HOP count) in the DHCP payload after doing the required checks. If any check fails, packet is dropped.
      5. Logical Router converts the packet to L3 unicast and forwards it to the server. This packets gets routed like any other packet (via RC node).
      6. Server replies with DHCP offer.
      7. RC node processes the DHCP offer and forwards it to the OVN controller.
      8. OVN controller does sanity checks and  updates the destination MAC (available in DHCP header), destination IP (available in DHCP header) and reinjects the packet to datapath.
         If any check fails, packet is dropped.
      9. Logical router updates the source IP and port and forwards the packet to logical switch.
      10. Logical switch delivers the packet to the DHCP client.
      11. Similar steps are performed for Request and Ack packets.
      12. All the future renew/release packets are directly exchanged between DHCP client and DHCP server

    NEW OVN ACTIONS
    ---------------
      1. dhcp_relay_req_chk(<relay-ip>, <server-ip>)
          - This action executes on the source node on which the DHCP request originated.
          - This action relays the DHCP request coming from client to the server. Relay-ip is used to update GIADDR in the DHCP header.
      2. dhcp_relay_resp_chk(<relay-ip>, <server-ip>)
          - This action executes on the first node (RC node) which processes the DHCP response from the server.
          - This action updates  the destination MAC and destination IP so that the response can be forwarded to the appropriate node from which request was originated.
          - Relay-ip, server-ip are used to validate GIADDR and SERVER ID in the DHCP payload.

    FLOWS
    -----
    Following are the flows added when DHCP Relay is configured on one overlay subnet, one additonal flow is added in ls_in_l2_lkup table for each VM part of the subnet.

      1. table=27(ls_in_l2_lkup      ), priority=100  , match=(inport == <vm_port> && eth.src == <vm_mac> && ip4.src == 0.0.0.0 && ip4.dst == 255.255.255.255 && udp.src == 68 && udp.dst == 67),
         action=(eth.dst=<lrp_mac>;outport=<lrp>;next;/* DHCP_RELAY_REQ */)
      2. table=3 (lr_in_ip_input     ), priority=110  , match=(inport == <lrp> && ip4.src == 0.0.0.0 && ip4.dst == 255.255.255.255 && ip.frag == 0 && udp.src == 68 && udp.dst == 67),
         action=(reg9[7] = dhcp_relay_req_chk(<lrp_ip>, <dhcp_server_ip>);next; /* DHCP_RELAY_REQ */)
      3. table=3 (lr_in_ip_input     ), priority=110  , match=(ip4.src == <dhcp_server> && ip4.dst == <lrp> && udp.src == 67 && udp.dst == 67), action=(next;/* DHCP_RELAY_RESP */)
      4. table=4 (lr_in_dhcp_relay_req), priority=100  , match=(inport == "lrp1" && ip4.src == 0.0.0.0 && ip4.dst == 255.255.255.255 && udp.src == 68 && udp.dst == 67 && reg9[7]),
         action=(ip4.src=<lrp>;ip4.dst=<dhcp_server>;udp.src=67;next; /* DHCP_RELAY_REQ */)
      5. table=4 (lr_in_dhcp_relay_req), priority=1    , match=(inport == <lrp> && ip4.src == 0.0.0.0 && ip4.dst == 255.255.255.255 && udp.src == 68 && udp.dst == 67 && reg9[7] == 0),
         action=(drop; /* DHCP_RELAY_REQ */)
      6. table=18(lr_in_dhcp_relay_resp_chk), priority=100  , match=(ip4.src == <dhcp_server> && ip4.dst == <lrp> && ip.frag == 0 && udp.src == 67 && udp.dst == 67),
         action=(reg2 = ip4.dst;reg9[8] = dhcp_relay_resp_chk(<lrp_ip>, <dhcp_server_ip>);next;/* DHCP_RELAY_RESP */)
      7. table=19(lr_in_dhcp_relay_resp), priority=100  , match=(ip4.src == <dhcp_server> && reg2 == <lrp_ip> && udp.src == 67 && udp.dst == 67 && reg9[8]),
         action=(ip4.src=<lrp>;udp.dst=68;outport=<lrp>;output; /* DHCP_RELAY_RESP */)
      8. table=19(lr_in_dhcp_relay_resp), priority=1    , match=(ip4.src == <dhcp_server> && reg2 == <lrp_ip> && udp.src == 67 && udp.dst == 67 && reg9[8] == 0), action=(drop; /* DHCP_RELAY_RESP */)

    NEW PIPELINE STAGES
    -------------------
    Following stage is added for DHCP relay feature. Some of the flows are fitted into the existing pipeline tages.
      1. lr_in_dhcp_relay_req
          - This stage process the DHCP request packets coming from DHCP clients.
          - DHCP request packets for which dhcp_relay_req_chk action (which gets applied in ip input stage) is successful are forwarded to DHCP server.
          - DHCP request packets for which dhcp_relay_req_chk action is unsuccessful gets dropped.
      2. lr_in_dhcp_relay_resp_chk
          - This stage applied the dhcp_relay_resp_chk action for  DHCP response packets coming from the DHCP server.
      3. lr_in_dhcp_relay_resp
          - DHCP response packets for which dhcp_relay_resp_chk is sucessful are forwarded to the DHCP clients.
          - DHCP response packets for which dhcp_relay_resp_chk is unsucessful gets dropped.

    REGISTRY USAGE
    ---------------
      - reg9[7] : To store the result of dhcp_relay_req_chk action.
      - reg9[8] : To store the result of dhcp_relay_resp_chk action.
      - reg2 : To store the original dest ip for DHCP response packets.
               This is required to properly match the packets in
               lr_in_dhcp_relay_resp stage since dhcp_relay_resp_chk action
               changes the dest ip.

    NB SCHEMA CHANGES
    ----------------
      1. New DHCP_Relay table
          "DHCP_Relay": {
                "columns": {
                    "name": {"type": "string"},
                    "servers": {"type": {"key": "string",
                                           "min": 0,
                                           "max": 1}},
                    "external_ids": {
                        "type": {"key": "string", "value": "string",
                                "min": 0, "max": "unlimited"}}},
                    "options": {"type": {"key": "string", "value": "string",
                                "min": 0, "max": "unlimited"}},
                "isRoot": true},
      2. New column to Logical_Router_Port table
          "dhcp_relay": {"type": {"key": {"type": "uuid",
                                "refTable": "DHCP_Relay",
                                "refType": "strong"},
                                "min": 0,
                                "max": 1}},

    Commands to enable the feature:
    ------------------------------
     ovn-nbctl create DHCP_Relay name=<name> servers=<dhcp_server_ip>
     ovn-nbctl set Logical_Router_port <lrp> dhcp_relay=<relay_uuid>
     ovn-nbctl set Logical_Switch <ls> other_config:dhcp_relay_port=<router_patch_port>

    Example:
    -------
     ovn-nbctl ls-add ls0
     ovn-nbctl lsp-add ls0 vif0
     ovn-nbctl lsp-set-addresses vif0 <MAC> #Only MAC address has to be specified when logical ports are created.
     ovn-nbctl lsp-add ls0 lrp1-attachment
     ovn-nbctl lsp-set-type lrp1-attachment router
     ovn-nbctl lsp-set-addresses lrp1-attachment
     ovn-nbctl lsp-set-options lrp1-attachment router-port=lrp1
     ovn-nbctl lr-add lr0
     ovn-nbctl lrp-add lr0 lrp1 <MAC> <GATEWAY_IP/Prefix> #GATEWAY IP is set in GIADDR field when relaying the DHCP requests to server.
     ovn-nbctl lrp-add lr0 lrp-ext <MAC> <GATEWAY_IP/Prefix>
     ovn-nbctl ls-add ls-ext
     ovn-nbctl lsp-add ls-ext lrp-ext-attachment
     ovn-nbctl lsp-set-type lrp-ext-attachment router
     ovn-nbctl lsp-set-addresses lrp-ext-attachment
     ovn-nbctl lsp-set-options lrp-ext-attachment router-port=lrp-ext
     ovn-nbctl lsp-add ls-ext ln_port
     ovn-nbctl lsp-set-addresses ln_port unknown
     ovn-nbctl lsp-set-type ln_port localnet
     ovn-nbctl lsp-set-options ln_port network_name=physnet1
     # Enable DHCP Relay feature
     ovn-nbctl create DHCP_Relay name=dhcp_relay_test servers=<dhcp_server_ip>
     ovn-nbctl set Logical_Router_port lrp1 dhcp_relay=<relay_uuid>
     ovn-nbctl set Logical_Switch ls0 other_config:dhcp_relay_port=lrp1-attachment

    Limitations:
    ------------
      - All OVN features that needs IP address to be configured on logical port (like proxy arp, etc) will not be supported for overlay subnets on which DHCP relay is enabled.

    References:
    ----------
      - rfc1541, rfc1542, rfc2131

V1:
  - First patch.

V2:
  - Addressed review comments from Numan.

V3:
  - Split the patch into series.
  - Addressed review comments from Numan.
  - Updated the match condition for DHCP Relay flows.

V4:
  - Fix sparse errors.
  - Reorder patch series.

V5:
  - Fix test failures.

V6:
  - Addressed review comments from Numan.
  - Increment NB schema version.

Naveen Yerramneni (3):
  actions: DHCP Relay Agent support for overlay IPv4 subnets.
  controller: DHCP Relay Agent support for overlay IPv4 subnets.
  northd, tests: DHCP Relay Agent support for overlay IPv4 subnets.

 controller/pinctrl.c    | 597 +++++++++++++++++++++++++++++++++++-----
 include/ovn/actions.h   |  27 ++
 lib/actions.c           | 116 ++++++++
 lib/ovn-l7.h            |   2 +
 northd/northd.c         | 271 +++++++++++++++++-
 northd/northd.h         |  41 +--
 northd/ovn-northd.8.xml | 211 ++++++++++++--
 ovn-nb.ovsschema        |  21 +-
 ovn-nb.xml              |  39 +++
 ovn-sb.xml              |  62 +++++
 tests/atlocal.in        |   3 +
 tests/ovn-northd.at     |  38 +++
 tests/ovn.at            | 258 ++++++++++++++++-
 tests/system-ovn.at     | 148 ++++++++++
 utilities/ovn-trace.c   |  67 +++++
 15 files changed, 1784 insertions(+), 117 deletions(-)