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

Some serials removed after changes in r10900

Details

    • Bug
    • Resolution: Fixed
    • Minor
    • None
    • Professional Edition
    • Poller

    Description

      Hi,

      Seems serials coming from "entPhysicalSerialNum" is removed after changes in r10900.

      I think maybe line 131 in /observium/trunk/includes/polling/os.inc.php

      Relevant things from os debugging

      Results:
      [entPhysicalDescr] => string(24) "Dell Inc. PowerEdge R740"
      [entPhysicalModelName] => string(14) "PowerEdge R740"
      [entPhysicalSerialNum] => string(7) "60XXXV2"
      [entPhysicalMfgName] => string(9) "Dell Inc."

      Output:
      Detected empty value for param 'serial'.
      Added OS param from ENTITY-MIB: 'hardware' = 'PowerEdge R740'
      Added OS param from ENTITY-MIB: 'vendor' = 'Dell Inc.'

      Attachments

        Issue Links

          Activity

            [OBS-3605] Some serials removed after changes in r10900
            bot Observium Bot made changes -
            Status Original: Resolved [ 5 ] New: Closed [ 6 ]
            kentbjoh Kent Johannessen made changes -
            Link New: This issue mentions OBS-3608 [ OBS-3608 ]

            Excellent, thanks
            Made a new Jira issue with some improvements here: OBS-3608

            kentbjoh Kent Johannessen added a comment - Excellent, thanks Made a new Jira issue with some improvements here:  OBS-3608
            landy Mike Stupalov made changes -
            Resolution New: Fixed [ 1 ]
            Status Original: In Progress [ 3 ] New: Resolved [ 5 ]

            Ahh, derp.
            Fixed in r10930.

            kentbjoh

            landy Mike Stupalov added a comment - Ahh, derp. Fixed in r10930. kentbjoh
            landy Mike Stupalov made changes -
            Status Original: In Review [ 10101 ] New: In Progress [ 3 ]
            landy Mike Stupalov made changes -
            Status Original: Pending Response [ 10000 ] New: In Review [ 10101 ]

            yes, it's still the same.

            If you look at the code i referred to. anything "entPhysicalSerialNum" is just set to empty string

             

                elseif ($oid === 'entPhysicalSerialNum')
                {
                  $value = '';
                }
            

            Unless I am missing something, that seems wrong

             

             

            kentbjoh Kent Johannessen added a comment - yes, it's still the same. If you look at the code i referred to. anything "entPhysicalSerialNum" is just set to empty string   elseif ($oid === 'entPhysicalSerialNum' ) { $value = '' ; } Unless I am missing something, that seems wrong    
            landy Mike Stupalov made changes -
            Status Original: In Progress [ 3 ] New: Pending Response [ 10000 ]

            Hrm, but can you check if issue still exist for latest revisions (after r10904)?

            If yes, please show me full (original) serial string(s).

            landy Mike Stupalov added a comment - Hrm, but can you check if issue still exist for latest revisions (after r10904)? If yes, please show me full (original) serial string(s).

            People

              landy Mike Stupalov
              kentbjoh Kent Johannessen
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: