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

mof/mounting.mof (Diff revision 1)
23
    "filesystem independent mount options (as listed in mount(8)). Options "
24
    "'auto' and 'user' are ommited, as they only make sense in /etc/fstab." ) ]
Options 'auto' and 'user' are *not* omitted.

mof/mounting.mof (Diff revision 1)
181
               "Size Not Supported", "Method Reserved", "Vendor Specific" } ]
I think 'Size Not Supported' could be removed from the value map, IMHO it makes no sense in this method or in ModifyMount and DeleteMount.

mof/mounting.mof (Diff revision 1)
203
        [ IN, Description (
204
            "Filesystem specification. Specifies the device that should be "
205
            "mounted. Remote filesystems can be specified in their usual "
206
            "form (e.g. 'hostname:/share' for NFS, or '//hostname/share' for "
207
            "CIFS). Non-device filesystems can also be specified "
208
            "(e.g. 'tmpfs' or 'sysfs')." ) ]
209
        string FileSystemSpec,
I would add: 'For bind mounts, this is path to source directory of the bind mount.'

- Jan


On April 8th, 2013, 3:22 p.m. CEST, Jan Synacek wrote:

Review request for OpenLMI Developers.
By Jan Synacek.

Updated April 8, 2013, 3:22 p.m.

Repository: openlmi-storage

Description

Add mounting.mof.

Diffs

  • mof/mounting.mof (PRE-CREATION)

View Diff