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

WIP - adding support for Juniper BTI 7000 #9979

Open
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
4 participants
@spencerbutler
Copy link
Contributor

spencerbutler commented Mar 16, 2019

Very much a work in progress at the moment. The MIB structure for this device is non-standard, and proving to be quite a challenge to integrate into the lnms ecosystem. Any constructive feedback welcome.

Fixes #9964
#9964

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
After you are done testing, you can remove the changes with ./scripts/github-remove. If there are schema changes, you can ask on discord how to revert.

@spencerbutler spencerbutler referenced this pull request Mar 16, 2019

Open

New Device: BTI 7000 #9964

3 of 3 tasks complete
@czilian

This comment has been minimized.

Copy link
Contributor

czilian commented Mar 17, 2019

Thx guys for helping. Got stuck for the same reasons.

@czilian

This comment has been minimized.

Copy link
Contributor

czilian commented Mar 17, 2019

Regarding OS name: I have another BTI device in the making with totally different OS. Hence, maybe you could call it "juniper-bti7xxx" because the other would be then "juniper-bti8xxx" (BTI-SA-805 is model I am testing it against).

@laf

This comment has been minimized.

Copy link
Member

laf commented Apr 9, 2019

Unless a huge difference exists between the 7k and 8k series then try and keep it in one.

@spencerbutler Anything you need from us?

@laf laf added the User-Pending label Apr 9, 2019

@spencerbutler

This comment has been minimized.

Copy link
Contributor Author

spencerbutler commented Apr 17, 2019

FWIW, I have code almost ready to push that adds lots of sensors. When I return in a week or two I will commit it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.