On Mon, 08 Feb 2010 11:23:37 -0800
Renee Danson Sommerfeld <renee.sommerfeld at sun.com> wrote:

> On Mon, Feb 08, 2010 at 07:15:48AM -0800, Michael Hunter wrote:
> > For these bugs found during zones testing:
> > 
> > 14302 -  netcfgd needs refresh specified to reduce log noise
> > 14305 -  netcfgd/nwamd have issues starting up in a zone
> > 14311 -  svcs -xv output dated
> > 14423 -  pfail() outputs trash
> > 14424 -  /lib/svc/method/net-nwam shouldn't set UMEM_DEBUG
> > 14425 -  error message for fattach in nwam_door_init() is busted
> > 
> > /net/coupe.eng/builds/mph/nwam1_bug/webrev
> 
> nwamd/util.c: when we're dropping unneeded privileges, why don't we
>   need to change the effective set?  That's what we give everything
>   to when we escalate, seems like it would need to change when we
>   de-escalate.

ACCEPT

> 
> Regarding the doc links: the hub.opensolaris.org links are every bit
> as stable as the opensolaris.org links you just ripped out.  :-)
> The main project page link *should* redirect, I thought; but it isn't
> working for me now.

ACCEPT

> 
> So, I would say go ahead and put in a link for our main project page
> at hub.opensolaris.org.  I would say we would have solid grounds for
> yelling at the website people if they did another move and redirects
> for that didn't work.  The spec page, or anything buried within the
> project page, I could see being problematic; so I think we should
> omit that link.  We should have the main page organized such that
> the phase 1 spec is easy to find.
> 
> Otherwise, looks good.

New webrev up.

        Michael

> 
> -renee

Reply via email to