Skip to content

Add s7comm external input plugin#9360

Merged
reimda merged 1 commit intoinfluxdata:masterfrom
nicolasme:master
Jun 15, 2021
Merged

Add s7comm external input plugin#9360
reimda merged 1 commit intoinfluxdata:masterfrom
nicolasme:master

Conversation

@nicolasme
Copy link
Copy Markdown
Contributor

Required for all PRs:

  • Updated associated README.md.
  • Wrote appropriate unit tests.

resolves #7728

Added external input plugin for Siemens PLC using golang library gos7

First ever contribution to open source project. All advice are welcome.

@telegraf-tiger telegraf-tiger bot added the feat Improvement on an existing feature such as adding a new setting/mode to an existing plugin label Jun 14, 2021
@srebhan srebhan added the external plugin Plugins that would be ideal external plugin and expedite being able to use plugin w/ Telegraf label Jun 14, 2021
@srebhan srebhan self-assigned this Jun 14, 2021
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.

@nicolasme thanks for submitting this PR. I also skimmed over your plugin code and it looks quite nice and lean. Good job!

One thing that would be nice is to sort the list of external plugins alphabetically, but that might be a task for your next PR!?!? ;-P

@srebhan srebhan 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 Jun 14, 2021
@reimda reimda merged commit 07778c5 into influxdata:master Jun 15, 2021
@reimda
Copy link
Copy Markdown
Contributor

reimda commented Jun 15, 2021

Thanks @nicolasme for the external plugin link!

reimda pushed a commit that referenced this pull request Jul 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

external plugin Plugins that would be ideal external plugin and expedite being able to use plugin w/ Telegraf feat Improvement on an existing feature such as adding a new setting/mode to an existing plugin 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.

S7comm external, input plugin

3 participants