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

On April 9th, 2013, 8:44 a.m. CEST, Radek Novacek wrote:

mof/mounting.mof (Diff revision 1)
107
        "Allow an ordinary user to mount the filesystem. Corresponds to 'auto' "
Shouldn't this be "Corresponds to 'user' mount option" ?
Yes, this is fixed in [3/10] part of the series.

On April 9th, 2013, 8:44 a.m. CEST, Radek Novacek wrote:

mof/mounting.mof (Diff revision 1)
152
        "Method to create and populate an LMI_MountedFileSystemSetting instance. This "
an -> a
I'm pretty sure 'an' is correct here. Try pronouncing 'L' (pronounciation begins with a vowel).

On April 9th, 2013, 8:44 a.m. CEST, Radek Novacek wrote:

mof/mounting.mof (Diff revision 1)
212
            "Filesystem type. If NULL, perform a binding mount. If binding a "
"If binding a local filesystem" - shouldn't it be "If mounting a local filesystem"?
Yes, it should be 'mounting'. I'll fix it.

- 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