Clay,

ai-httpd.conf
-------------
36,37 - Shouldn't these paths be somewhere in /var/run instead of
somewhere on the disk?


delete_service.py
-----------------
809-812, Should we execute lines 814-819 first before we transition
to MAINTENANCE?


thanks,
-ethan


Clay Baenziger wrote:
> Hi all,
>     Could Ethan (or if someone else wants to instead) code review the 
> changes for:
> 11214 - AI image web server won't run if apache22 is already enabled
> 12016 - Need to use SIGTERM and check for last service in delete_service
> 
> The -- seven line -- webrev is at:
> http://cr.opensolaris.org/~clayb/11214_12016/
> 
> The bugs are at:
> http://defect.opensolaris.org/bz/show_bug.cgi?id=11214
> http://defect.opensolaris.org/bz/show_bug.cgi?id=12016
>                             Thank you,
>                             Clay

Reply via email to