Bug 560302 - WebYaST init scripts are confused by generic lighttpd
Summary: WebYaST init scripts are confused by generic lighttpd
Status: RESOLVED FIXED
Alias: None
Product: openSUSE 11.2
Classification: openSUSE
Component: WebYaST (show other bugs)
Version: Final
Hardware: Other Other
: P2 - High : Critical (vote)
Target Milestone: ---
Assignee: Martin Vidner
QA Contact: E-mail List
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-12-03 11:31 UTC by Martin Vidner
Modified: 2009-12-04 14:45 UTC (History)
4 users (show)

See Also:
Found By: Development
Services Priority:
Business Priority:
Blocker: ---
Marketing QA Status: ---
IT Deployment: ---

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Martin Vidner 2009-12-03 11:31:10 UTC 
In bug 559534 comment 31 (updating lighttpd crashes webyast) I note:
> Actually there is a bug where rcyastw[sc] think they are running when in fact
only the generic lighttpd is running.

It turns out that both checkproc (status) and startproc (start) are invoked with -p $PIDFILE_OF_THIS_SERVICE -i $PIDFILE_OF_THE_OTHER_WEBYAST_HALF, but the program still picks up on the running copy of the generic lighttpd and therefore wrongly reports webyast as already running and does not start it.
Comment 1 Martin Vidner 2009-12-03 12:00:06 UTC 
If I read the man page right, startproc does not even write a pid file.
So we should be just fine forgetting checkproc and startproc and simply checking the pid file and starting lighttpd directly.
Comment 2 Martin Vidner 2009-12-04 14:45:54 UTC 
In bug 559534 it turned out that startproc had a bug where it ignored -p sometimes.
Fixing that, -i is not necessary anymore, yay!

http://git.opensuse.org/?p=projects/yast/rest-service.git;a=commitdiff;h=4ab3fbcea1b5aaf42edfdf7a26a1c94fa1d218a0
http://git.opensuse.org/?p=projects/yast/web-client.git;a=commitdiff;h=c827dbff4be050721d48aae7497a3aacf6146c20

yast2-webservice-0.0.19 and yast2-webclient-0.0.28 submitted to YaST:Web:STABLE

I will find the fixes for lighttpd ans sysvinit and link them to Devel:WebYaST, then create a separate small repo from their binaries. I will not forward them to SUSE:SLE-11:GA:Products:Test.