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

Sostat cleanup output: Suricata output when no packet drops received and sostat-interface formatting #7

Merged
merged 2 commits into from Jul 14, 2016

Conversation

Projects
None yet
2 participants
@weslambert
Contributor

weslambert commented Jul 14, 2016

Modifies sostat to include output for Suricata when no packet drops are received.

Modifies formatting in sostat-interface-delta for "Packets received during monitoring interval..."

Thanks,
Wes

@dougburks dougburks merged commit 1d99277 into Security-Onion-Solutions:master Jul 14, 2016

dougburks added a commit that referenced this pull request Jul 14, 2016

@dougburks

This comment has been minimized.

Show comment
Hide comment
@dougburks

dougburks Jul 20, 2016

Contributor

It looks like we take the last 50 lines of the stats.log ($i) and pipe that into a grep of the same file. Is that correct?

Contributor

dougburks commented on bin/sostat in b63e3d9 Jul 20, 2016

It looks like we take the last 50 lines of the stats.log ($i) and pipe that into a grep of the same file. Is that correct?

This comment has been minimized.

Show comment
Hide comment
@weslambert

weslambert Jul 20, 2016

Contributor
Contributor

weslambert replied Jul 20, 2016

This comment has been minimized.

Show comment
Hide comment
@dougburks

dougburks Jul 20, 2016

Contributor

Should the grep be searching the tail output instead of searching the file directly? Meaning that we should take tail -n 50 $i | grep -c drop $i and remove the second $i, making it just tail -n 50 $i | grep -c drop?

Contributor

dougburks replied Jul 20, 2016

Should the grep be searching the tail output instead of searching the file directly? Meaning that we should take tail -n 50 $i | grep -c drop $i and remove the second $i, making it just tail -n 50 $i | grep -c drop?

This comment has been minimized.

Show comment
Hide comment
@weslambert

weslambert Jul 20, 2016

Contributor
Contributor

weslambert replied Jul 20, 2016

This comment has been minimized.

Show comment
Hide comment
@dougburks

dougburks Jul 20, 2016

Contributor

Sounds good, thanks Wes!

Contributor

dougburks replied Jul 20, 2016

Sounds good, thanks Wes!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment