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

          mentions

          Improvement - An improvement or enhancement to an existing feature or task. OBS-3608 Improvements to is_valid_parm function

          • Minor - Minor loss of function, or other problem where easy workaround is present.
          • Closed

          Activity

            [OBS-3605] Some serials removed after changes in r10900
            kentbjoh Kent Johannessen added a comment -

            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 added a comment -

            Ahh, derp.
            Fixed in r10930.

            kentbjoh

            landy Mike Stupalov added a comment - Ahh, derp. Fixed in r10930. kentbjoh
            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

             

             

            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 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).

            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).
            landy Mike Stupalov added a comment -

            Ohh, too simple validator for serials...

            landy Mike Stupalov added a comment - Ohh, too simple validator for serials...
            dklimek Denis Klimek added a comment -

            I can confirm this. We saw this too on very many devices happening.

            dklimek Denis Klimek added a comment - I can confirm this. We saw this too on very many devices happening.
            bot Observium Bot added a comment -

            Please make and attach additional information about the device:

            • full snmp dump from device: 

              snmpwalk -v2c -c <community>  --hexOutputLength=0 -ObentxU <hostname> .1 > myagent.snmpwalk
              		 
              snmpwalk -v2c -c <community>  --hexOutputLength=0 -ObentxU <hostname> .1.3.6.1.4.1 >> myagent.snmpwalk

              If device not support SNMP version 2c, replace -v2c with -v1.

            • If you have problems with discovery or poller processes, please do and attach these debugs: 

              ./discovery.php -d -h <device>
              		 
              ./poller.php -d -h <device>

            • additionally attach device and/or vendor specific MIB files

            Note, this comment is added automatically.

            bot Observium Bot added a comment - Please make and attach additional information about the device: full snmp dump from device: snmpwalk -v2c -c <community> --hexOutputLength=0 -ObentxU <hostname> .1 > myagent.snmpwalk snmpwalk -v2c -c <community> --hexOutputLength=0 -ObentxU <hostname> .1.3.6.1.4.1 >> myagent.snmpwalk If device not support SNMP version 2c, replace -v2c with -v1. If you have problems with discovery or poller processes, please do and attach these debugs: ./discovery.php -d -h <device> ./poller.php -d -h <device> additionally attach device and/or vendor specific MIB files Note, this comment is added automatically.

            People

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

              Dates

                Created:
                Updated:
                Resolved: