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

Disable modules enabled by default to improve discovery/poller speed. #7539

Merged
merged 1 commit into from Oct 25, 2017
Merged

Disable modules enabled by default to improve discovery/poller speed. #7539

merged 1 commit into from Oct 25, 2017

Conversation

alangregory
Copy link

DO NOT DELETE THIS TEXT

Please note

Please read this information carefully. You can run ./scripts/pre-commit.php to check your code before submitting.

Testers

If you would like to test this pull request then please run: ./scripts/github-apply <pr_id>, i.e ./scripts/github-apply 5926

@alangregory alangregory changed the title Disable modules enabled by default do improve discovery/poller speed. Disable modules enabled by default to improve discovery/poller speed. Oct 24, 2017
@scrutinizer-notifier
Copy link

The inspection completed: No new issues

@laf
Copy link
Member

laf commented Oct 24, 2017

Do these devices genuinely not support these modules or is this personal choice?

@alangregory
Copy link
Author

No support, except the port module, but it is unusable at this time.
Each Onu returns they interfaces without data, and then we receive an interface table entry of 6k to 10k. I am in contact with the manufacturer, when it is fixed, I will implement the correct reading.
Example:
https://gist.github.com/alangregory/c9409815fb1f3d52f9986eb88a157cb0

@laf laf merged commit 383557f into librenms:master Oct 25, 2017
@alangregory alangregory deleted the fiberhome-modules branch April 11, 2018 19:50
@lock lock bot locked as resolved and limited conversation to collaborators Jun 10, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants