Uploaded image for project: 'Observium'
  1. Observium
  2. OBS-293

LLDP discovers wrong links on Juniper MX boxes

Details

    • Bug
    • Resolution: Fixed
    • Minor
    • None
    • None
    • Discovery
    • Observium Version r3977
      Debian Version 6.0

    Description

      LLDP discovery inserts wrong links into the 'links' table when run against Juniper MX (running JunOS 11.4R5).

      LLDP discovery cannot find the right port_id and as a last resort uses the remote MAC-Adress which actually discovers the wrong link on the remote side.

      Solution:

      Juniper returns the SNMP ifIndex of the remote port as lldpRemPortId. We can use this to find the correct port. Patch is attached.

      And as a personal opinion: Perhaps the fallback to the mac-address should be removed. I would rather have no link discovered instead of the wrong one.

      Attachments

        Activity

          [OBS-293] LLDP discovers wrong links on Juniper MX boxes
          landy Mike Stupalov made changes -
          Workflow Original: classic default workflow [ 10502 ] New: Observium workflow [ 13638 ]
          adama Adam Armstrong made changes -
          Status Original: Resolved [ 5 ] New: Closed [ 6 ]
          adama Adam Armstrong made changes -
          Resolution New: Fixed [ 1 ]
          Status Original: Open [ 1 ] New: Resolved [ 5 ]
          adama Adam Armstrong added a comment -

          r4003.

          adama Adam Armstrong added a comment - r4003.
          adama Adam Armstrong added a comment -

          indeed. It wasn't obvious due to the terribly named variables.

          I won't commit it until i've time to go through and rewrite the variables to be less useless.

          adama Adam Armstrong added a comment - indeed. It wasn't obvious due to the terribly named variables. I won't commit it until i've time to go through and rewrite the variables to be less useless.
          sebastianw Sebastian added a comment -

          Why is that? I just added another query.

          sebastianw Sebastian added a comment - Why is that? I just added another query.
          adama Adam Armstrong added a comment -

          Your patch would break matching for every device which returns a ifDescr.

          adama Adam Armstrong added a comment - Your patch would break matching for every device which returns a ifDescr.
          sebastianw Sebastian created issue -

          People

            adama Adam Armstrong
            sebastianw Sebastian
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: