enhance: Remove docker acquis internal timer use docker events

[LaurenceJJones]

Instead of polling the api every one second, we can instead use the events hook in docker this is a continuous event stream that notify us of a container event. We filter for start and die which is crate and remove, to which we then trigger a function call to get all containers and update the state.

This means this should be less intensive since we dont have to constantly poll for all containers and only do so when we actually get a notification to do so.

[github-actions]

@LaurenceJJones: There are no 'kind' label on this PR. You need a 'kind' label to generate the release automatically.

• /kind feature
• /kind enhancement
• /kind refactoring
• /kind fix
• /kind chore
• /kind dependencies

Details

I am a bot created to help the crowdsecurity developers manage community feedback and contributions. You can check out my manifest file to understand my behavior and what I can do. If you want to use this for your project, you can check out the BirthdayResearch/oss-governance-bot repository.

[github-actions]

@LaurenceJJones: There are no area labels on this PR. You can add as many areas as you see fit.

• /area agent
• /area local-api
• /area cscli
• /area appsec
• /area security
• /area configuration

Details

I am a bot created to help the crowdsecurity developers manage community feedback and contributions. You can check out my manifest file to understand my behavior and what I can do. If you want to use this for your project, you can check out the BirthdayResearch/oss-governance-bot repository.

[LaurenceJJones]

/kind enhancement
/area agent

[codecov]

Codecov Report

Attention: Patch coverage is 43.96552% with 65 lines in your changes missing coverage. Please review.

Project coverage is 60.81%. Comparing base (5fe038d) to head (35e9e62).
Report is 1 commits behind head on master.

Files with missing lines	Patch %	Lines
pkg/acquisition/modules/docker/docker.go	43.96%	60 Missing and 5 partials ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #3598      +/-   ##
==========================================
- Coverage   60.82%   60.81%   -0.02%     
==========================================
  Files         372      372              
  Lines       40841    40917      +76     
==========================================
+ Hits        24843    24885      +42     
- Misses      13943    13979      +36     
+ Partials     2055     2053       -2     

Flag	Coverage Δ
bats	44.57% <0.00%> (-0.05%)	⬇️
unit-linux	34.37% <43.96%> (+<0.01%)	⬆️
unit-windows	24.33% <43.96%> (+0.03%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[mmetc]

@LaurenceJJones let me know what you think of my change, I didn't test in docker but the only difference should be that it's applied to the initial connection.

[sabban]

lgtm