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

Stalls when using port #6

Closed
nomasprime opened this issue May 22, 2016 · 5 comments · Fixed by #7
Closed

Stalls when using port #6

nomasprime opened this issue May 22, 2016 · 5 comments · Fixed by #7

Comments

@nomasprime
Copy link

nomasprime commented May 22, 2016

it { is_expected.to be_reachable } works as expected but it { is_expected.to be_reachable.tcp.dest_port(80) } outputs via firewall and a blinking cursor indefinitely.

This is the repo I'm working on - https://github.com/habitullence/ansible-demo. Should be able to reproduce by adding a test to existing spec file.

@nomasprime nomasprime changed the title Stalls when adding port Stalls when using port May 22, 2016
@otahi
Copy link
Owner

otahi commented May 22, 2016

Thank you for your reporting!
Can you describe more details?

  • which test code do you use?
  • can you ssh to the VM without password or yes/no questions?
  • is_expected.to be_reachable.tcp.dest_port(80).ack(:only) works?

@nomasprime
Copy link
Author

nomasprime commented May 23, 2016

No problem, thanks for the plugin.

  • I'm using the integration tests
  • Yes
  • The test fails

It's stalling somewhere in https://github.com/otahi/infrataster-plugin-firewall/blob/master/lib/infrataster/plugin/firewall/capture.rb#L50.

@nomasprime
Copy link
Author

@otahi
Copy link
Owner

otahi commented Oct 11, 2016

@habitullence I am very sorry about my late response.
I have found the test case can be passed with net-ssh 3.0.2. But the test fails with net-ssh 3.1.1.

I will dive into net-ssh.

@otahi
Copy link
Owner

otahi commented Oct 25, 2016

@habitullence Can you try to check #7?
If it is OK for you, I will merge and release this modification.
Thank you.

@otahi otahi closed this as completed in #7 Oct 25, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants