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

Add integration test for dovecot #9435

Closed
reimda opened this issue Jun 24, 2021 · 1 comment · Fixed by #11996
Closed

Add integration test for dovecot #9435

reimda opened this issue Jun 24, 2021 · 1 comment · Fixed by #11996
Labels
testing request An issue that requires the tests to be updated

Comments

@reimda
Copy link
Contributor

reimda commented Jun 24, 2021

#9415 happened because of a linter fix in code that wasn't covered with tests. A docker integration test would make sense here and would have caught the regression.

We also need to check if there's a better api to use to gather dovecot stats. The current one we use "old_statistics" requires us to ignore a timeout.

https://doc.dovecot.org/configuration_manual/stats/old_statistics/#old-statistics

@reimda reimda added the feature request Requests for new plugin and for new features to existing plugins label Jun 24, 2021
@sspaink sspaink added testing request An issue that requires the tests to be updated and removed feature request Requests for new plugin and for new features to existing plugins labels Jun 24, 2021
@pabl-o-ce
Copy link

Hey @reimda how config stats on dovecot? I'm trying but don't know where to put the metrics? con dovecot.conf? if you can guide I really appreciate :)

powersj added a commit to powersj/telegraf that referenced this issue Oct 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
testing request An issue that requires the tests to be updated
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants