-
Notifications
You must be signed in to change notification settings - Fork 173
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
passt file transfer tests via tcp/udp #4150
base: master
Are you sure you want to change the base?
Conversation
b0b1fda
to
c6c2d1f
Compare
Hi @PaulYuuu Could you please help me review this patch? It can test pass on my side, thanks in advance. |
5c1dadf
to
7944bac
Compare
Add a new case to support passt file transfer test via tcp/udp Signed-off-by: Lei Yang <[email protected]>
Hello @PaulYuuu Could you please help review it again? I have update it based on your last times comment. And the corrent code can test pass on my side. Thanks in advance. |
test.log.info) | ||
if dst_md5 != src_md5: | ||
test.fail("File changed after transfer host -> %s" % vm.name) | ||
host2guest_port += 1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Both host2guest_port and guest2guest_port should from vim.virtnet, not simple calculations, if the vm ports are 1001 and 1003, the current not work.
Add a new case to support passt file transfer test via tcp/udp
ID:2853
Signed-off-by: Lei Yang [email protected]