Hi Swati,
Couple of nits:
disk_screen.py, line 657: Can the comma be moved to the line above
so it follows the word "file". It's kinda odd to see a line starting
with a comma.
Also, can you make sure the the open double quote in this line lines up
with the
one above. It doesn't line up in the webrev.
disk_screen.py, line 664: The indentation of his line looks kinda
random. It should
be indented 4 spaces compared to the line above.
Thanks,
--Karen
On 07/17/12 11:47, Swati Sarraf wrote:
Hi All,
Can I have a code review for the following bug fix:
7183018 Gui livecd is not using size from image.info but using fallback image
size
http://monaco.us.oracle.com/detail.jsf?cr=7183018
Webrev link:
https://cr.opensolaris.org/action/browse/caiman/ssarraf/7183018/
Testing is done as follow:
1. Slim test done
Ran 1901 tests in 353.236s, FAILED (SKIP=1, failures=1)
failed test: Tests package with understood add option
Slim test pointer: file:///net/osol-bldx/datapool/ssarraf/7183018/
2. Pylint and pep8 test done , result OK
pylint result pointer: file:///net/osol-bldx/datapool/ssarraf/7183018/
3 Created DC image and tried installation with minimum size (4.7GB).
Installation finished successfully.
-Thanks
Swati Sarraf
_______________________________________________
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
