Hi Nirmal,

I have a question about your changes.

usr/src/lib/install_target/logical.py, line 711-712, why do you need to create
another Filesystem object in 711 for getting the size?
self.parent is a Zpool object, can you get the size from that?

Thanks,

--Karen


On 02/12/12 09: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