This is an automatically generated e-mail. To reply, visit: http://reviewboard-openlmi.rhcloud.com/r/896/

mof/60_LMI_Software.mof (Diff revision 1)
333
  [Implemented(true), Override("MethodName")]
333
  [ImplEmEnted(true), Override("MethodName")]
I believe, this is a typo.

mof/60_LMI_Software.mof (Diff revision 1)
611
              , "4097", "4098", "4099..32767"
611
              , "4097", "4098", "4099..32767"
612
              , "32768", "32769..65535" },
612
              , "32768", "32769..65535" },
613
    Values    { "Job Completed with No Error", "Not Supported"
613
    Values    { "Job Completed with No Error", "Not Supported"
614
              , "Unspecified Error", "Timeout", "Failed"
614
              , "Unspecified Error", "Timeout", "Failed"
615
              , "Invalid Parameter", "Target In Use", "DMTF Reserved"
615
              , "Invalid Parameter", "Target In Use", "DMTF Reserved"
616
              // 4096
616
              // 4096
617
              , "Method Parameters Checked - Job Started"
617
              , "Method Parameters Checked - Job Started"
618
              , "Unsupported TargetType"
618
              , "Unsupported TargetType"
619
              , "Method Reserved"
619
              , "Method Reserved"
620
              // 32768
620
              , "Reserved"
Please, instead of adding "Reserved", rather remove "4098" from ValueMap and extend the "Method Reserved" range.

- Michal Minar


On September 27th, 2013, 1:44 p.m. UTC, Jan Safranek wrote:

Review request for OpenLMI Developers.
By Jan Safranek.

Updated Sept. 27, 2013, 1:44 p.m.

Repository: openlmi-providers

Description

Make Storage MOF file parseable by openlmi-doc-class2rst

- don't use Linux_ classes
- fix ValueMaps

Diffs

  • mof/60_LMI_Software.mof (1462479dd829affdfef71294d4629ce831eb7b4f)

View Diff