-
Notifications
You must be signed in to change notification settings - Fork 29
[Zeta] Data path integration #189
Comments
Initial findings:
Minli may want to take a look before we come back tomorrow Eric's note: two extra bytes 0x0000 between outer source MAC and ethertype 0x0800 was not seen on the compute host with the latest run, @zhangml don't need to investigate that yet. Please focus on the issue (1). |
for reference: |
Non-zeta packet can be communicated with OVS fine and able to use tcpdump to capture the valid IPv4 tunnelled packet. zeta packet is sent to zgc and the packet capture by @liangbin-pub show that it has extra "0000" before "0800" which make it an invalid packet. I am seeing difference in how the Zeta openflow rule is constructed and understood by OVS: int ACA_Zeta_Programming::_create_zeta_group_entry(zeta_config *zeta_cfg) // adding group table rule root@28abfb290c2e:/mnt/host/code/aca-zeta# ovs-ofctl dump-flows br-tun -O OpenFlow13 ovs-ofctl dump-groups br-tun -O OpenFlow13 for non-zeta ports: int ACA_Vlan_Manager::create_l2_neighbor(string virtual_ip, string virtual_mac, ovs-ofctl dump-flows br-tun -O OpenFlow13 |
@er1cthe0ne Similar to CTSL::HashMap, is there a library file for hashSet? |
@zhangml - please note that the mac change (number 1 above) is highest priority and we need the change pretty quickly.
This is second priority. You can still leverage CTSL::HashMap and use it similar to hashSet, please see: namespace aca_vlan_manager // list of ovs_ports names on this host in the same VPC to share the same internal vlan_id string auxGateway_id; |
Update as of 1/13/21:
Next: debug if KVM host doesn't forward some packets |
Complete ARP-Ping-OAM round trip test passed! Pseudo controller test result:
Ping requester (ACA parent) packet capture:
br-tun flow table dump after test should have dynamic flow added by OAM packet (4th packet above), Eric and Minli please check:
|
@liangbin-pub ACA should install the rules on table20 after receiving oam packets, but I did not see it.
|
Track and communicate data path integration findings
The text was updated successfully, but these errors were encountered: