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

Fix WMI storage poller

Details

    • Bug
    • Resolution: Fixed
    • Major
    • None
    • None
    • Poller

    Description

      RRD name changed and WMI storage poller was not updated.

      Attached patch fixes this.

      Attachments

        Issue Links

          relates to

          Improvement - An improvement or enhancement to an existing feature or task. OBS-1335 Storage snmp/wmi mislabeled volumes on Windows

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

          Activity

            [OBS-938] Fix WMI storage poller
            landy Mike Stupalov added a comment -

            Fixed in r6542.

            landy Mike Stupalov added a comment - Fixed in r6542.
            robje Rob J. Epping added a comment -

            I probably wasn't clear enough.
            This patch has been on our production system for months without anyone noticing a problem.

            You never committed any oopses in the SVN repo?

            robje Rob J. Epping added a comment - I probably wasn't clear enough. This patch has been on our production system for months without anyone noticing a problem. You never committed any oopses in the SVN repo?
            robje Rob J. Epping added a comment -

            Same patch with typo fixed

            robje Rob J. Epping added a comment - Same patch with typo fixed
            adama Adam Armstrong added a comment -

            You submitted a patch you hadn't tested?

            I think that kinda disqualifies you from having any of your stuff committed...

            adama Adam Armstrong added a comment - You submitted a patch you hadn't tested? I think that kinda disqualifies you from having any of your stuff committed...
            robje Rob J. Epping added a comment -

            typo / fat fingers / cut-n-paste too much.

            Patch is working with the random ). Should not be in there though.
            Need a new patch?

            robje Rob J. Epping added a comment - typo / fat fingers / cut-n-paste too much. Patch is working with the random ). Should not be in there though. Need a new patch?
            adama Adam Armstrong added a comment -

            What's that random ) for?

            adama Adam Armstrong added a comment - What's that random ) for?

            People

              landy Mike Stupalov
              robje Rob J. Epping
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: