Hi Dave,
Thanks for working on these changes. The SMF and IPS changes look fine to me.

auto-installer:
line 174-176:
        * Could you update the comment to mention you're looking for
          the dataset (not dataset name(?)) and the method to your nawk
          script (e.g. looking for dataset active on reboot and return
          dataset field) since not all are fluent in (n)awk.

        * Just to confirm, I believe what you're doing is equivalent to
          the following too?
          beadm list -Hd|grep ';R;'|cut -d ';' -f 2

        * You might want to comment that reboot -f should be fine and
          will just warn about no dataset (or whatever the behavior is) if
          for whatever reason $dset is null?

                                                Thank you,
                                                Clay

On Tue, 14 Sep 2010, Dave Miner wrote:

I need two reviewers for some small fixes:

6982854 support nfs file based repos with auto-install
11087 auto-install should at least try to do a fast-reboot after install
13196 should turn off fastreboot_onpanic in microroot environment
8396 var/installadm/services should be removed from install/installadm

Webrev is at

http://cr.opensolaris.org/~dminer/slim_6982854/

Thanks,
Dave
_______________________________________________
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