Skip to content

fix(inputs.mongodb): actually start plugin correctly#11849

Merged
reimda merged 2 commits intoinfluxdata:masterfrom
powersj:fix/11830
Sep 21, 2022
Merged

fix(inputs.mongodb): actually start plugin correctly#11849
reimda merged 2 commits intoinfluxdata:masterfrom
powersj:fix/11830

Conversation

@powersj
Copy link
Copy Markdown
Contributor

@powersj powersj commented Sep 20, 2022

The start method handler did not match the interface, nor was there a
stop function. As a result, start was never called and the plugin was
never setting up the servers to connect to and collect from correctly.

This was introduced in #11629.

fixes: #11830

The start method handler did not match the interface, nor was there a
stop function. As a result, start was never called and the plugin was
never setting up the servers to connect to and collect from correctly.

This was introduced in influxdata#11629.

fixes: influxdata#11830
@telegraf-tiger telegraf-tiger bot added area/mongodb fix pr to fix corresponding bug plugin/input 1. Request for new input plugins 2. Issues/PRs that are related to input plugins labels Sep 20, 2022
@telegraf-tiger
Copy link
Copy Markdown
Contributor

@hackdefendr
Copy link
Copy Markdown

Tests all pass!
All good here.

@powersj powersj added the ready for final review This pull request has been reviewed and/or tested by multiple users and is ready for a final review. label Sep 21, 2022
Copy link
Copy Markdown
Member

@srebhan srebhan left a comment

Choose a reason for hiding this comment

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

Looks good to me. Thanks for fixing this issue @powersj!

@reimda reimda merged commit f899eb5 into influxdata:master Sep 21, 2022
popey pushed a commit that referenced this pull request Oct 3, 2022
dba-leshop pushed a commit to dba-leshop/telegraf that referenced this pull request Oct 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/mongodb fix pr to fix corresponding bug plugin/input 1. Request for new input plugins 2. Issues/PRs that are related to input plugins ready for final review This pull request has been reviewed and/or tested by multiple users and is ready for a final review.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Telegraf MongoDB plugin doesn't work.

5 participants