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

ss used in quick_start.sh not available on MacOS #3151

Closed
geekysquirrel opened this issue Oct 19, 2020 · 2 comments · Fixed by #4448
Closed

ss used in quick_start.sh not available on MacOS #3151

geekysquirrel opened this issue Oct 19, 2020 · 2 comments · Fixed by #4448
Labels
bug Something isn't working quick-start-script

Comments

@geekysquirrel
Copy link

When running FlowKit via the quick_start.sh script, ss -tuna is used to check for the status of FlowMachine.

Version
c4dc74f

To Reproduce
Steps to reproduce the behavior:

  1. Run quick_start.sh on MacOS

Expected behavior

[...]
FlowDB ready.
./quick_start.sh: line 141: ss: command not found
[...]

Additional context
A quick search revealed that ss (a netstat replacement included in the iproute package on linux) is apparently not included in MacOS, allegedly because of dependencies on the linux kernel. Workarounds exist.

@jc-harrison
Copy link
Member

Hmm. It should already be using a workaround if ss is not found - the script actually runs (ss -tuna || netstat -an).

So either we're just not suppressing the error message from attempting to use ss (and actually the script carries on fine), or we're not handling the error properly so the workaround never gets tried. Either way something could do with fixing, but with different degrees of severity.

Does the script go on to output "FlowMachine ready" after the error message?

@geekysquirrel
Copy link
Author

Yes it does, eventually.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working quick-start-script
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants