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

Log count of processed messages configurable #974

Merged
3 commits merged into from
Jun 7, 2017
Merged

Log count of processed messages configurable #974

3 commits merged into from
Jun 7, 2017

Conversation

ghost
Copy link

@ghost ghost commented May 15, 2017

It's now configurable how often the "Processed .. messages" is logged. You can give a count and seconds.

See #743

@ghost ghost added component: core feature Indicates new feature requests or new features labels May 15, 2017
@ghost ghost added this to the v1.0 Stable Release milestone May 15, 2017
@ghost ghost requested a review from dmth May 15, 2017 14:15
@codecov-io
Copy link

codecov-io commented May 15, 2017

Codecov Report

Merging #974 into master will increase coverage by 0.06%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #974      +/-   ##
==========================================
+ Coverage   77.98%   78.04%   +0.06%     
==========================================
  Files         221      221              
  Lines        9047     9070      +23     
==========================================
+ Hits         7055     7079      +24     
+ Misses       1992     1991       -1
Impacted Files Coverage Δ
intelmq/lib/bot.py 70.11% <100%> (+0.77%) ⬆️
intelmq/tests/lib/test_parser_bot.py 100% <100%> (ø) ⬆️
intelmq/tests/lib/test_bot.py 100% <100%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 893aa04...326314d. Read the comment docs.

Copy link
Contributor

@dmth dmth left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for implementing this.

I did not run the code, but it looks ok.

Have you implemented your idea from #743 (comment), also?

@ghost
Copy link
Author

ghost commented May 29, 2017

Have you implemented your idea from #743 (comment), also?

Thanks for the reminder. Did that including tests and also merged master.

@ghost ghost merged commit 326314d into master Jun 7, 2017
@ghost ghost deleted the fix-743 branch June 7, 2017 14:04
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: core feature Indicates new feature requests or new features
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants