277210 - Testserver.pl should find apache2/httpd2 as well when it looks for webservers

Status: RESOLVED FIXED

(Bugzilla :: Testing Suite, defect)

Description

[Joel Peshkin]

testserver.pl misses "apache2" when it looks for "httpd" or "apache"

Comment 1

[Dave Miller [:justdave]]

[Sun 22:31:10] <kieran> Hi, I changed the testserver.pl script to have
httpd|apache|apache2 and it worked fine.
[Sun 22:31:22] <kieran> thanks

Comment 2

[Kieran Lal]

Attachment: testserver.pl with apache2 added to OR statement for server daemon check.

This is a simple check to see if the Web Server is working.  Currently it
doesn't check for Apache2 and it probably should.  I added an additional OR
statement so the test passes.

Comment 3

[Kieran Lal]

Attachment: Added OR statement to include Apache2 as a webserver to be checked for to see if Bugzilla is running

Removed the whole file, and added a diff.

Comment 4

[Colin Ogilvie [:cso]]

I'd also recommend adding httpd2 as thats what my install of Apache from
Mandrake 10.0 seems to be running as.

Comment 5

[Marc Schumann [:Wurblzap]]

Comment on attachment 170792 [details] [diff] [review]
Added OR statement to include Apache2 as a webserver to be checked for to see if Bugzilla is running

r=wurblzap by inspection.

It's new to me that Apache uses apache2, but what do I know :)

Comment 6

[Marc Schumann [:Wurblzap]]

(In reply to comment #4)
> I'd also recommend adding httpd2 as thats what my install of Apache from
> Mandrake 10.0 seems to be running as.

Whoa, I missed that comment. Please do indeed so, and move r+ forward.

Comment 7

[Vlad Dascalu]

Targeting bug to 2.20 since the 2.20 feature freeze was canceled.

Comment 8

[Colin Ogilvie [:cso]]

Attachment: Detect apache2/httpd2 as well

This hopefully (it detects my httpd2 and an apache on knoppix) allows it to
detect both httpd2 and apache2 as well as the original.

Comment 9

[Vlad Dascalu]

Comment on attachment 171378 [details] [diff] [review]
Detect apache2/httpd2 as well

Looks good upon my review, but also carrying over the previous review :-)

Comment 10

[Vlad Dascalu]

Checking in testserver.pl;
/cvsroot/mozilla/webtools/bugzilla/testserver.pl,v  <--  testserver.pl
new revision: 1.4; previous revision: 1.3
done

Comment 11

[Vlad Dascalu]

Comment on attachment 170792 [details] [diff] [review]
Added OR statement to include Apache2 as a webserver to be checked for to see if Bugzilla is running

Marking this patch obsolete.

Comment 12

[Vlad Dascalu]

Checking in testserver.pl;
/cvsroot/mozilla/webtools/bugzilla/testserver.pl,v  <--  testserver.pl
new revision: 1.1.2.3; previous revision: 1.1.2.2
done