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.

Sign up for GitHub

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

Jump to bottom

forensic mode and concurrent runners : sqlite lock #60

Closed
buixor opened this issue Jun 2, 2020 · 1 comment · Fixed by #61
Closed

forensic mode and concurrent runners : sqlite lock #60

buixor opened this issue Jun 2, 2020 · 1 comment · Fixed by #61
Labels
kind/bug Something isn't working kind/enhancement New feature or request

Comments

@buixor
Copy link
Contributor

buixor commented Jun 2, 2020

When having both crowdsec running in daemon mode and in forensics mode (-file ... -type ...) I sometime encounter sqlite locks :

WARN[02-06-2020 17:03:05] 60.251.xxx.xx triggered a 4h0m0s ip ban remediation for [crowdsecurity/http-crawl-non_statics]  bucket_id=muddy-morning event_time="2019-01-08 21:16:49 +0100 CET" scenario=crowdsecurity/http-crawl-non_statics source_ip=60.251.xxx.xx
FATA[02-06-2020 17:03:06] failed to flush : failed to commit records : database is locked

While it's okay, I don't think we should fatal on this, but rather retry ?
@buixor buixor added kind/bug Something isn't working kind/enhancement New feature or request labels Jun 2, 2020
buixor added a commit that referenced this issue Jun 3, 2020
@buixor
don't fatal is DB is locked. Attempt to fix #60 - but was not able to…
e7a2f32 
… reproduce
@buixor
Copy link
Contributor Author

buixor commented Jun 3, 2020

edit: it's actually when metabase is actively querying the database while crowdsec is trying to write.

@buixor buixor mentioned this issue Jun 3, 2020
Merged
AlteredCoder pushed a commit that referenced this issue Jun 3, 2020
@buixor
fix #60 - sqlite lock + badges (#61)
f9a4ae2 
* don't fatal is DB is locked + Fix #60
registergoofy pushed a commit that referenced this issue Sep 7, 2020
@buixor @registergoofy
fix #60 - sqlite lock + badges (#61)
94d48ea 
* don't fatal is DB is locked + Fix #60
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
kind/bug Something isn't working kind/enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix #60 - sqlite lock + badges crowdsecurity/crowdsec
1 participant
@buixor