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

netflow log rotation parameters #1170

Closed
fichtner opened this issue Sep 2, 2016 · 2 comments
Closed

netflow log rotation parameters #1170

fichtner opened this issue Sep 2, 2016 · 2 comments
Assignees
Labels
support Community support
Milestone

Comments

@fichtner
Copy link
Member

fichtner commented Sep 2, 2016

Hi Ad,

The current flowd log file is turned over 10 times for each 10 MB. In cases where this is larger traffic volume old data points may be lost when recreating the database, going into POLA violation territory because it's not obvious. What about making these two values configurable?

Or changing the defaults, or documenting them. Or something else entirely? :)

Cheers,
Franco

See: https://forum.opnsense.org/index.php?topic=3581.0

@fichtner fichtner added the support Community support label Sep 2, 2016
@fichtner fichtner added this to the 17.1 milestone Sep 2, 2016
@AdSchellevis
Copy link
Member

Hi Franco,

I don't mind adding some configuration parameters for it, but this issue should only happen when you trash the full database and try to recreate it from its source (which may take quite some time if you increase this value).... Personally I haven't seen corruption on the more aggregated datasets only saw the 30 seconds average get corrupted beyond repair, although others can get damaged by a crash as well (as can flowd.log) .

Basically it isn't a very good idea to trash data you actually want to preserve ;) The flowd log files are only meant for staging. You can't keep a lot of history in there, because it would take too much time to parse it back to our structure.

Cheers,

Ad

@fichtner
Copy link
Member Author

Upon further review this seems to work as intended if we can guarantee the log rotation to kick in, which seems to be the case with the database repair now. I will make a note in the GUI and close this.

fichtner added a commit that referenced this issue Sep 23, 2016
While here, remove the RAM disk mention, we can cope with this now.

(cherry picked from commit f85f356)
(cherry picked from commit 02f9143)
(cherry picked from commit 47988b5)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
support Community support
Development

No branches or pull requests

2 participants