Re: [zones-discuss] Webrev for CR 6909222

2010-01-04 Thread Jerry Jelinek
Jordan Vaughan wrote: Maybe I'm not understanding the bug's evaluation but it seems to say that the problem is caused by the presence of boot archive files. Jerry Jerry, It is. However, bootadm(1M) infers the existence of boot archives from the existence of /boot/solaris/bin/create_ramdisk

Re: [zones-discuss] Webrev for CR 6909222

2010-01-04 Thread Jordan Vaughan
On 01/ 4/10 10:25 AM, Jerry Jelinek wrote: Jordan Vaughan wrote: On 01/ 4/10 09:54 AM, Jerry Jelinek wrote: Jordan Vaughan wrote: On 01/ 4/10 07:26 AM, Jerry Jelinek wrote: Jordan Vaughan wrote: I need someone to review my fix for 6909222 reboot of system upgraded from 128 to build 129 gene

Re: [zones-discuss] Webrev for CR 6909222

2010-01-04 Thread Jerry Jelinek
Jordan Vaughan wrote: On 01/ 4/10 09:54 AM, Jerry Jelinek wrote: Jordan Vaughan wrote: On 01/ 4/10 07:26 AM, Jerry Jelinek wrote: Jordan Vaughan wrote: I need someone to review my fix for 6909222 reboot of system upgraded from 128 to build 129 generated error from an s10 zone due to boot-ar

Re: [zones-discuss] Webrev for CR 6909222

2010-01-04 Thread Jordan Vaughan
On 01/ 4/10 09:54 AM, Jerry Jelinek wrote: Jordan Vaughan wrote: On 01/ 4/10 07:26 AM, Jerry Jelinek wrote: Jordan Vaughan wrote: I need someone to review my fix for 6909222 reboot of system upgraded from 128 to build 129 generated error from an s10 zone due to boot-archive My webrev is ac

Re: [zones-discuss] Webrev for CR 6909222

2010-01-04 Thread Jordan Vaughan
On 01/ 4/10 09:57 AM, Enda O'Connor wrote: Hi Jordan Not being too familiar with the underlying technology, I assume s10_boot.ksh is run every boot, as otherwise a patch can deliver create_ramdisk to the system after it's removed. Enda Correct. Thanks, Jordan _

Re: [zones-discuss] Webrev for CR 6909222

2010-01-04 Thread Enda O'Connor
Jordan Vaughan wrote: On 12/23/09 12:40 AM, Frank Batschulat (Home) wrote: On Tue, 22 Dec 2009 00:46:00 +0100, Jordan Vaughan wrote: I need someone to review my fix for 6909222 reboot of system upgraded from 128 to build 129 generated error from an s10 zone due to boot-archive My webrev

Re: [zones-discuss] Webrev for CR 6909222

2010-01-04 Thread Jerry Jelinek
Jordan Vaughan wrote: On 01/ 4/10 07:26 AM, Jerry Jelinek wrote: Jordan Vaughan wrote: I need someone to review my fix for 6909222 reboot of system upgraded from 128 to build 129 generated error from an s10 zone due to boot-archive My webrev is accessible via http://cr.opensolaris.org/~fli

Re: [zones-discuss] Webrev for CR 6909222

2010-01-04 Thread Jordan Vaughan
On 12/23/09 12:40 AM, Frank Batschulat (Home) wrote: On Tue, 22 Dec 2009 00:46:00 +0100, Jordan Vaughan wrote: I need someone to review my fix for 6909222 reboot of system upgraded from 128 to build 129 generated error from an s10 zone due to boot-archive My webrev is accessible via http:/

Re: [zones-discuss] Webrev for CR 6909222

2010-01-04 Thread Jordan Vaughan
On 01/ 4/10 07:26 AM, Jerry Jelinek wrote: Jordan Vaughan wrote: I need someone to review my fix for 6909222 reboot of system upgraded from 128 to build 129 generated error from an s10 zone due to boot-archive My webrev is accessible via http://cr.opensolaris.org/~flippedb/onnv-s10c Jorda

Re: [zones-discuss] Webrev for CR 6909222

2010-01-04 Thread Jerry Jelinek
Jordan Vaughan wrote: I need someone to review my fix for 6909222 reboot of system upgraded from 128 to build 129 generated error from an s10 zone due to boot-archive My webrev is accessible via http://cr.opensolaris.org/~flippedb/onnv-s10c Jordan, This looks ok to me but don't we need to

Re: [zones-discuss] Webrev for CR 6909222

2009-12-23 Thread Frank Batschulat (Home)
> On Tue, 22 Dec 2009 00:46:00 +0100, Jordan Vaughan > wrote: > >> I need someone to review my fix for >> >> 6909222 reboot of system upgraded from 128 to build 129 generated error >> from an s10 zone due to boot-archive >> >> My webrev is accessible via >> >> http://cr.opensolaris.org/~flippedb/

Re: [zones-discuss] Webrev for CR 6909222

2009-12-22 Thread Jordan Vaughan
Hi Frank, Thanks for reviewing my fix. Native-branded zones will disappear when Solaris Express dies (which should happen in a few builds); therefore, it isn't worthwhile to fix this problem for native-branded zones. No special script code is needed for ipkg-branded zones because IPS package

Re: [zones-discuss] Webrev for CR 6909222

2009-12-22 Thread Frank Batschulat (Home)
On Tue, 22 Dec 2009 00:46:00 +0100, Jordan Vaughan wrote: > I need someone to review my fix for > > 6909222 reboot of system upgraded from 128 to build 129 generated error > from an s10 zone due to boot-archive > > My webrev is accessible via > > http://cr.opensolaris.org/~flippedb/onnv-s10c Jo

Re: [zones-discuss] Webrev for CR 6909222

2009-12-21 Thread Jordan Vaughan
Excellent suggestion. I'll update CR 6912451. Meanwhile, I removed the XXX comments and regenerated the webrev. Thanks, Jordan On 12/21/09 06:12 PM, Edward Pilatowicz wrote: i'd probably leave out the XXX comment and instead update 6912451 to mention that part of the fix for 6912451 would i

Re: [zones-discuss] Webrev for CR 6909222

2009-12-21 Thread Edward Pilatowicz
i'd probably leave out the XXX comment and instead update 6912451 to mention that part of the fix for 6912451 would involve removing the fix for 6909222 (since it would essentially be obsoleting this fix.) ed On Mon, Dec 21, 2009 at 03:46:00PM -0800, Jordan Vaughan wrote: > I need someone to revi

Re: [zones-discuss] Webrev for CR 6909222

2009-12-21 Thread Steve Lawrence
The bug mentions that this can also impact a nevada zone that was p2v'ed. Should you fix usr/src/lib/brand/native/zone as well? -Steve On Mon, Dec 21, 2009 at 03:46:00PM -0800, Jordan Vaughan wrote: > I need someone to review my fix for > > 6909222 reboot of system upgraded from 128 to build 129

[zones-discuss] Webrev for CR 6909222

2009-12-21 Thread Jordan Vaughan
I need someone to review my fix for 6909222 reboot of system upgraded from 128 to build 129 generated error from an s10 zone due to boot-archive My webrev is accessible via http://cr.opensolaris.org/~flippedb/onnv-s10c Thanks, Jordan ___ zones-disc