Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # 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.

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

Successfully merging this pull request may close these issues.

3 participants