Thank you for review, Niall !
Jan

On 06/15/12 12:25 PM, Niall Power wrote:
Hi Jan,

Looks good  to me.

Niall

On 14/06/12 06:09 PM, Jan Damborsky wrote:
Hi,

could I please ask two pairs of eyes to take a look at fix for

7155952 textui needs better default network adapter
7171775 users.py:home_zfs_parent_exists() confused by "/export/home/" mountpoint

webrev:
https://cr.opensolaris.org/action/browse/caiman/dambi/cr-7171775/webrev-1/

Thank you,
Jan


Testing done:
* regression tests

- built text/AI/LiveCD Sparc/x86 install images.

- tested text (booted from media and network),
  AI,
  LiveCD installations (Sparc and x86).

* 7155952

tested on x4100 containing 4 NICs configured in following way:

net0 - not connected
net1 - not connected
net2 - connected (used as AI boot NIC)
net3 - not connected

- booted text installer from media and verified that
  net2 was selected as default NIC

- booted text installer from AI server and verified that
  net2 was selected as default NIC

* 7171775

tested in non-global zone, verified that

- creation of user account is enabled for both
  rpool/export/home mountpointset to  '/export/home'
  as well as '/export/home/'.

- creation of user account is disabled if rpool/export/home
  dataset set does not exist.

* run related unit tests

_______________________________________________
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

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

Reply via email to