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.

Sign up for GitHub

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

Jump to bottom

Add deprecation notice to inputs.leofs documentation #9927

Open
Hipska opened this issue Oct 14, 2021 · 5 comments · May be fixed by #16154
Open

Add deprecation notice to inputs.leofs documentation #9927

Hipska opened this issue Oct 14, 2021 · 5 comments · May be fixed by #16154
Assignees
@DStrand1
Labels
area/snmp feature request Requests for new plugin and for new features to existing plugins plugin/input 1. Request for new input plugins 2. Issues/PRs that are related to input plugins size/m 2-4 day effort

Comments

@Hipska
Copy link
Contributor

Hipska commented Oct 14, 2021

Feature Request

The inputs.leofs will become deprecated (see #9478), so the docs of this plugin should describe this to the users.

Proposal:

Add example configuration how to achieve the same with inputs.snmp.
@Hipska Hipska added the feature request Requests for new plugin and for new features to existing plugins label Oct 14, 2021
@Hipska Hipska self-assigned this Oct 14, 2021
@Hipska Hipska added the plugin/input 1. Request for new input plugins 2. Issues/PRs that are related to input plugins label Oct 14, 2021
@sjwang90
Copy link
Contributor

@Hipska It would be nice to target this deprecation announcement for the next minor release (1.21 in December) along with all the new deprecation changes.

@Hipska Hipska mentioned this issue Feb 2, 2022
Open
@Hipska Hipska mentioned this issue Feb 9, 2022
Closed
75 tasks
@powersj powersj added the size/m 2-4 day effort label Aug 11, 2022
@Hipska Hipska removed their assignment Jan 24, 2024
@DStrand1
Copy link
Member

@Hipska Should this plugin still be deprecated? If so, are you able to give an example of how to migrate a config to the snmp plugin?

@DStrand1 DStrand1 added the waiting for response waiting for response from contributor label Oct 28, 2024
@Hipska
Copy link
Contributor Author

Hipska commented Oct 29, 2024

It definitely should! It is hard-coded to use the net-snmp commands and "public" community string.

All I can see it is doing a walk on .1.3.6.1.4.1.35450, I could not find a MIB to find out what is available under it, but could find these docs which may help: https://leo-project.net/leofs/docs-old/configuration/configuration_4.html

@telegraf-tiger telegraf-tiger bot removed the waiting for response waiting for response from contributor label Oct 29, 2024
@DStrand1 DStrand1 self-assigned this Oct 29, 2024
@DStrand1 DStrand1 linked a pull request Nov 6, 2024 that will close this issue
Open
1 task
@DStrand1
Copy link
Member

DStrand1 commented Nov 6, 2024

@Hipska are you able to review or test if #16154 properly migrates the config? Thanks!

@Hipska
Copy link
Contributor Author

Hipska commented Nov 6, 2024

Sure, I will try. Let me know when it is ready.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@DStrand1 DStrand1

Labels
area/snmp feature request Requests for new plugin and for new features to existing plugins plugin/input 1. Request for new input plugins 2. Issues/PRs that are related to input plugins size/m 2-4 day effort
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

chore(inputs.leofs): Deprecate plugin and add migration to inputs.snmp DStrand1/telegraf
5 participants
@Hipska @powersj @MyaLongmire @sjwang90 @DStrand1