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

Sorting the SNMP's Lookup instead of assume the ifIndex … #939

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

manassesferreira
Copy link

…of the interfaces are ordered. Related with samilliken/openDCIM #938 issue.

…aces are ordered. Related with samilliken/openDCIM

 opendcim#938 issue.
@wilpig
Copy link
Collaborator

wilpig commented Jul 31, 2017

Don't think this is the best route to be going down for this. Before we put something like this in place we should instead look at expanding the port model to include the indexid from the snmp device and let the end user match ports up.

@mvfcva
Copy link

mvfcva commented Feb 12, 2018

@wilpig I agree it would be great to add ifIndex to the port model.
I am planning to add more switchport features to opendcim (change VLAN...) and this would be a prerequisite
Do you plan to add this anytime soon ?

@wilpig
Copy link
Collaborator

wilpig commented Feb 12, 2018

I don't have a lot of time to work on this right now and I don't have access to some of the offending equipment which makes it difficult to test as well. Adding a vlan column seems out of scope. We have typically told people to put this type of information into the notes field.

@vkristian
Copy link

Is there anything I can do to help with expanding the port model to include the indexid from the snmp source?
although I'm not a programmer, I have access to new stacked C3850s I can test on...
we also have one student on summer internship in our team, if you'd describe approach you want to take, we might be able to do at least some of the work... (?)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants