-
Notifications
You must be signed in to change notification settings - Fork 13
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
kola/tests: Replace nmap's ncat with openbsd ncat #539
Conversation
3dab3fd
to
8fb4cd1
Compare
8fb4cd1
to
979c8a2
Compare
Ready for review. The changes in crio and podman tests are untested - we are not running those on Flatcar. |
Just occurred to me that the docker network test will need a split - old version with nmap ncat, new version with openbsd nc. |
979c8a2
to
755b614
Compare
CI on older versions (to test if nmap ncat docker.network still works):
Also rekicked test-dispatcher jobs in CI for the new versions (at http://jenkins.infra.kinvolk.io:8080/job/container/job/packages_all_arches/4414/cldsv/) |
CI passed. |
Will drop the |
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.
I don't know Go, but it LGTM.
Nmap's ncat will be replaced in newer versions with OpenBSD nc. But keep the old test intact, as the older channels still have nmap's ncat. The CRIO and podman are untested - they do not run on Flatcar.
755b614
to
a602f87
Compare
OpenBSD netcat has no long flags and has no idle timeout flag, so these need to be replaced with short flags and using a timeout utility, respectively.