Thanks, Harold. Looks good now.
Sue

On 08/05/11 15:11, Harold Shaw wrote:
I have made the changes that Sue suggested. There is an updated webrev at:
https://cr.opensolaris.org/action/browse/caiman/hshaw/7066261/webrev/

The output from the install server is now:
[ Aug 5 15:01:43 Executing start method ("/lib/svc/method/svc-install-server 
start"). ]
The svc:/network/tftp/udp6 service has been configured to use the /tftpboot
directory; The Automated Installer is incompatible with this setting.
Please use svccfg to change the inetd_start/exec property of the
svc:/network/tftp/udp6 service to migrate to the /etc/netboot directory.

[ Aug 5 15:01:43 Method "start" exited with status 95. ]


On 08/ 5/11 03:48 PM, Sue Sohn wrote:
Hi Harold,

Just a couple of minor things, in svc-install-server.

You are adding /etc/netboot at line 44, but installadm-common which is 
included, already has a
NETBOOTDIR which is also /etc/netboot.

Also, line 62, you have "AutoInstall is incompatible...". I believe that we are 
supposed to refer
to AI as the "Automated Installer".

Sue

On 08/05/11 14:35, Harold Shaw wrote:
Thanks. I fixed that and created a new webrev if anyone wants to take a look.

webrev: https://cr.opensolaris.org/action/browse/caiman/hshaw/7081868_1

Harold

On 08/ 5/11 03:20 PM, Drew Fisher wrote:
Harold,

About the only thing I see is lines 135-144 in svc-install-server appear to be 
at the wrong
indentation level. Otherwise, LGTM.

-Drew

On 8/5/11 3:04 PM, Harold Shaw wrote:
Please review my changes for the following changes:
7066261 <http://monaco.us.oracle.com/detail.jsf?cr=7066261> Need a way to force 
the migration
from /tftpboot to /etc/netboot when upgrading to ISIM
7071868 <http://monaco.us.oracle.com/detail.jsf?cr=7071868> Default service not 
created if SMF
services fail to start

webrev:
https://cr.opensolaris.org/action/browse/caiman/hshaw/7081868/webrev/

I ran the following tests:
- Created an initial service (create default service alias) with nothing in 
/tftpboot
- Created additional service with nothing in /tftpboot
- Created an alias of an existing service with nothing in /tftpboot
- Created an initial service (create default service alias) with stuff in 
/tftpboot
- Created additional service with stuff in /tftpboot
- Created an alias of an existing service with stuff in /tftpboot
- Tried enabling install/server manually with stuff in /tftpboot
- Cleared out /tftpboot and created a service to verify that the service would 
start up clean



_______________________________________________
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

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

Reply via email to