Last modified: 2014-09-10 04:43:15 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T52912, the corresponding Phabricator task for complete and up-to-date bug report information.

Bug 50912 - Fatal PHP errors can prompt Vagrant to re-configure MediaWiki


Summary:	Fatal PHP errors can prompt Vagrant to re-configure MediaWiki

Status:	RESOLVED FIXED

Product:	MediaWiki-Vagrant
Classification:	Unclassified
Component:	General (Other open bugs)
Version:	unspecified
Hardware:	All All

Importance:	Normal major
Target Milestone:	---
Assigned To:	Dan Duvall

URL:
Whiteboard:
Keywords:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2013-07-07 23:08 UTC by Ori Livneh
Modified:	2014-09-10 04:43 UTC (History)
CC List:	3 users (show)

See Also:
Web browser:	---
Mobile Platform:	---
Assignee Huggle Beta Tester:	---

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

Description Ori Livneh 2013-07-07 23:08:30 UTC

To reproduce, introduce a syntax error to LocalSettings.php and run 'vagrant provision'. The error causes the 'check settings' exec task in puppet/modules/mediawiki/manifests/init.pp to fail, which prompts Puppet to delete mediawiki/LocalSettings.php and re-install MediaWiki.

Comment 1 Andre Klapper 2013-09-11 22:45:33 UTC

[mass-moving from Tools>MediaWiki-Vagrant to separate product. See bug 54041. Filter bugmail on this comment.]

Comment 2 Yuvi Panda 2013-10-16 09:16:59 UTC

Hmm, replace a php run with just a file exists check, perhaps?

Comment 3 Bryan Davis 2014-09-05 23:59:59 UTC

Dan and I chatted about this a bit today and he's going to take a stab at alternate behavior.

Comment 4 Gerrit Notification Bot 2014-09-08 23:12:49 UTC

Change 159217 had a related patch set uploaded by Dduvall:
Wiki installer now conditional on existence of database

https://gerrit.wikimedia.org/r/159217

Comment 5 Gerrit Notification Bot 2014-09-10 04:27:03 UTC

Change 159217 merged by jenkins-bot:
Wiki installer now conditional on existence of database

https://gerrit.wikimedia.org/r/159217

Comment 6 Bryan Davis 2014-09-10 04:43:15 UTC

Tested fix by deliberately introducing a syntax error in LocalSettings.php. In my particular configuration, the next puppet run failed with an error while checking for the existence of the Selenium_user account and included a php stacktrace that pointed to the php syntax error. Much nicer than having my whole wiki messed up.

I then dropped the database for the wiki and ran `vagrant provision` again. The installer was triggered and the database was recreated.

Wikimedia Bugzilla is closed!

Search

Personal tools

Navigation

Links