Nirmal,

Other than what Karen already pointed out, this looks ok to me.

For Karen's comment, you can just use:

zpool_size = self.parent.get("free") to get size of the free space in the pool.

-Drew

On 2/12/12 10:12 PM, Nirmal Agarwal wrote:
Hi all

Could I please get a code review for the following CR:

7070697 size element for zvols should be optional

Webrev :
https://cr.opensolaris.org/action/browse/caiman/nirmal27/7070697/webrev/

slim_test result
----------------
/net/indiana-build.us.oracle.com//export/home/na210770/ai/7070697/slim_test

Manual Tests :

-- created a custom image and tested the fix with the manifest not
   describing the size of zvol.


Thanks
Nirmal
_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss

Reply via email to