Hi Ed et al,
here are my comments:
- usr/src/cmd/initpkg/manifest/Makefile
usr/src/cmd/initpkg/svcmethod/Makefile
* copyright 2008 -> 2009
- usr/src/ipsdefs/ha-cluster/*
usr/src/ipsdefs/Makefile
* I am not sure why we need that. My vote is to not deliver this.
- usr/src/cmd/initpkg/svcmethod/svc_boot_check_enabler
* I see that line 64 says this service should be never enabled.
Wouldn't it be better to have this service deleted once it was run?
I guess the service can't delete itself, but can't this be done within
scinstall?
If this is too complicated to impelemt then I won't like the service
being there, but won't object for the change.
Greets
Thorsten
Ed McKnight wrote:
> I need one more reviewer for this... thx, --emk
>
> Ed McKnight wrote:
>> Sorry for the rush--hoping to putback these changes for build 7.
>>
>> http://galileo.sfbay.sun.com/workspace/emk/sc/ws/webreviews/CO-IPS-10/
>>
>> Please review the changes for:
>>
>> - 6802680 Disabled service cl_boot_check:default prevents node from
>> boot up
>> Under SVR4, postinstall schedules this service for synch enable at
>> next boot; if it's not enabled at boot then console-login can't be
>> enabled. Under IPS there is no postinstall and presently, if this is
>> synch enabled via scinstall then there is a window between pkg install
>> and scinstall where if the user reboots, the login service will not be
>> enabled.
>> The fix is to deliver (only under IPS) a new service called
>> cl_boot_check_enabler which is delivered enabled so it runs at package
>> installation time, which does the necessary scheduling. Now the
>> bad-boot window no longer exists.
>> Note that this fix requires the application of the fix for
>> http://defect.opensolaris.org/bz/show_bug.cgi?id=7264 which
>> unfortunately didn't make it into 109. For people on-SWAN I made a
>> little installer that applies the fix. It can also be done manually
>> very easily.
>>
>> - create default group package: ha-cluster
>> As a placeholder I've set this group package to do ha-cluster-full.
>> Those who believe that default install should be ha-cluster-minimal,
>> sing out now.
>>
>> - 6805657 manual for vxvm in clsetup needs to be removed from colorado
>> Similar to the fix for disabling unavailable data service wizards
>> in clsetup, this fix tests to see if we have clustering support for
>> vxvm, and if not, disables all vxvm-specific options
>>
>> thx, --emk
--
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
Sitz der Gesellschaft:
Sun Microsystems GmbH, Sonnenallee 1, D-85551 Kirchheim-Heimstetten
Amtsgericht Muenchen: HRB 161028
Geschaeftsfuehrer: Thomas Schroeder, Wolfgang Engels, Dr. Roland Boemer
Vorsitzender des Aufsichtsrates: Martin Haering
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
