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

Support In-situ Operation Administration and Maintenance (IOAM) #4232

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jingfeihu
Copy link
Contributor

1.support CMCC IOAM over ipv4/ipv6 + tcp/udp/vxlan
2.add CMCC IOAM unit tests

@jingfeihu jingfeihu changed the title Support In-situ Operation Administration and Maintenance Protocol (IOAM) Support In-situ Operation Administration and Maintenance (IOAM) Jan 25, 2024
Copy link

codecov bot commented Jan 25, 2024

Codecov Report

Merging #4232 (f93e029) into master (d71014a) will decrease coverage by 32.65%.
Report is 5 commits behind head on master.
The diff coverage is 46.15%.

Additional details and impacted files
@@             Coverage Diff             @@
##           master    #4232       +/-   ##
===========================================
- Coverage   81.77%   49.12%   -32.65%     
===========================================
  Files         331      332        +1     
  Lines       76721    76752       +31     
===========================================
- Hits        62736    37703    -25033     
- Misses      13985    39049    +25064     
Files Coverage Δ
scapy/contrib/ioam.py 100.00% <100.00%> (ø)
scapy/layers/inet6.py 41.22% <54.54%> (-47.33%) ⬇️
scapy/layers/inet.py 24.09% <0.00%> (-46.53%) ⬇️

... and 232 files with indirect coverage changes

1.support CMCC IOAM over ipv4/ipv6 + tcp/udp/vxlan
2.add CMCC IOAM unit tests
@jingfeihu jingfeihu marked this pull request as ready for review January 25, 2024 09:28
@guedou guedou marked this pull request as draft January 30, 2024 15:48
@guedou
Copy link
Member

guedou commented Jan 30, 2024

@jingfeihu it looks like some tests are failing. I converted your PR to a draft. Feel free to modify it when it is ready.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants