Harold,

On 08/07/11 15:22, Harold Shaw wrote:
Hi Ethan,
The changes look fine but I am wondering how the issue that you raised in the evaluation of 7069337 is going to be tracked or does it need to be?

Thanks for the review. A separate bug will be filed for that if that's what we need to do. At this point though, I'm not sure yet if we ought to remove that ICT altogether.

thanks,
-ethan

The relevant section shown below:
However, that property will not be left set on the installed image
(non-global or global zone).  The installer has an explicit ICT to set the
flush-content-cache-on-success back to false after install completes no
matter what it was set to during the installation.

Now that we have support in the AI manifest to specify this, I am wondering
if AI needs to run that ICT any longer.  It would seem we should leave it
set as whatever the user specified in the manifest.  Any default should
otherwise be set as by pkg'ing system itself.

Harold

On 08/ 7/11 04:05 PM, Ethan Quach wrote:
Can I get a review for the following bugfixes:


Webrev:
----------
https://cr.opensolaris.org/action/browse/caiman/equach/7069338.7069337/webrev/

Bugs
-------
7069338 <http://monaco.us.oracle.com/detail.jsf?cr=7069338> Need zfs compression set for ROOT dataset in zones 7069337 <http://monaco.us.oracle.com/detail.jsf?cr=7069337> Set flush-content-cache-on-success to true in zone default manifest


thanks,
-ethan


_______________________________________________
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