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

[SDFAB-1084] Wipe out state from BESS on restart #572

Merged
merged 2 commits into from Mar 17, 2022
Merged

Conversation

osinstom
Copy link
Contributor

@osinstom osinstom commented Mar 16, 2022

TODO:

  • test with real BESS

@osinstom
Copy link
Contributor Author

Report from testing with real BESS:

  1. Run BESS with ./scripts/docker_setup.sh in sim mode
  2. Setup PFCP sessions: docker exec -ti bess-pfcpiface pfcpiface -config /conf/upf.json -simulate create
  3. Check that rules have been installed:
$ ./bessctl show module pdrLookup
  pdrLookup::WildcardMatch(8 fields, 6244 rules)
  1. Restart pfcpiface Docker container with docker restart

  2. Check rules have been cleared.

$ ./bessctl show module pdrLookup
  pdrLookup::WildcardMatch(8 fields, 0 rules)
    Per-packet metadata fields:
      src_iface: read   1 bytes at offset 12
tunnel_ipv4_dst: read   4 bytes at offset 44
           teid: read   4 bytes at offset 40
         src_ip: read   4 bytes at offset 28
         dst_ip: read   4 bytes at offset 32
       src_port: read   2 bytes at offset 14
       dst_port: read   2 bytes at offset 36
       ip_proto: read   1 bytes at offset 13
         pdr_id: write  4 bytes (no downstream reader)
          fseid: write  8 bytes at offset 16
         ctr_id: write  4 bytes (no downstream reader)
         qer_id: write  4 bytes at offset 24
         far_id: write  4 bytes at offset 8
         
 $ ./bessctl show module farLookup
  farLookup::ExactMatch(2 fields, 0 rules)
    Per-packet metadata fields:
         far_id: read   4 bytes at offset 8
          fseid: read   8 bytes at offset 16
         action: write  1 bytes at offset 36
tunnel_out_type: write  1 bytes at offset 12
tunnel_out_src_ip4addr: write  4 bytes at offset 32
tunnel_out_dst_ip4addr: write  4 bytes at offset 24
tunnel_out_teid: write  4 bytes at offset 28
tunnel_out_udp_port: write  2 bytes at offset 14

$ ./bessctl show module appQERLookup
  appQERLookup::Qos(3 fields, 0 rules)
    Per-packet metadata fields:
      src_iface: read   1 bytes at offset 12
         qer_id: read   4 bytes at offset 24
          fseid: read   8 bytes at offset 16
            qfi: write  1 bytes at offset 13
            
$ ./bessctl show module sessionQERLookup
  sessionQERLookup::Qos(2 fields, 0 rules)
    Per-packet metadata fields:
      src_iface: read   1 bytes at offset 12
          fseid: read   8 bytes at offset 16                    

@osinstom osinstom requested a review from pudelkoM March 16, 2022 10:32
@osinstom
Copy link
Contributor Author

@ccascone just to confirm, is that in line with your understanding of what should be done in SDFAB-1084?

Copy link
Member

@pudelkoM pudelkoM left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code looks good, can't comment on the jira ticket.

@osinstom osinstom merged commit 026d1ff into master Mar 17, 2022
@osinstom osinstom deleted the bess-wipe-out branch March 17, 2022 11:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants