Bug 560302

Summary:	WebYaST init scripts are confused by generic lighttpd
Product:	[openSUSE] openSUSE 11.2	Reporter:	Martin Vidner <mvidner>
Component:	WebYaST	Assignee:	Martin Vidner <mvidner>
Status:	RESOLVED FIXED	QA Contact:	E-mail List <qa-bugs>
Severity:	Critical
Priority:	P2 - High	CC:	kkaempf, mrueckert, mvidner, schubi
Version:	Final
Target Milestone:	---
Hardware:	Other
OS:	Other
Whiteboard:
Found By:	Development	Services Priority:
Business Priority:		Blocker:	---
Marketing QA Status:	---	IT Deployment:	---

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.