Page MenuHomePhabricator

move misc mw maintenance scripts into mw puppet module
Closed, ResolvedPublic

Description

we would like to get rid of the entire 'misc' directory in the operations/puppet repo.

misc/maintenance.pp is one of the bigger ones left.

maintenance.pp contains all those mw related scripts and crons. so the idea is to move them into the mediawiki module instead of sitting here in /misc unrelated.

i started a patch for that here https://gerrit.wikimedia.org/r/#/c/178873/

could platform engineering / Ori / Reedy / anyone take a look ?

this feels a bit touchy because it can potentially influence all the maintenance crons at once.

also, is this a good moment to check if any of these scripts are not being used anymore that can be removed?

Event Timeline

Dzahn raised the priority of this task from to Needs Triage.
Dzahn updated the task description. (Show Details)
Dzahn subscribed.
Dzahn set Security to None.
Dzahn triaged this task as Medium priority.Feb 5 2015, 1:23 AM

I merged this:

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

compiler result was here:

http://puppet-compiler.wmflabs.org/891/terbium.eqiad.wmnet/

also checked the run on terbium and compared crontabs of mwdeploy and www-data before and after, identical

all the maintenance jobs are here now:

/puppet/modules/mediawiki/manifests/maintenance$ ls
cleanup_upload_stash.pp  purge_checkuser.pp   translationnotifications.pp  updatetranslationstats.pp
echo_mail_batch.pp       purge_securepoll.pp  update_article_count.pp      wikidata.pp
pagetriage.pp            README.md            update_flaggedrev_stats.pp
parsercachepurging.pp    refreshlinks.pp      updatequerypages.pp
purge_abusefilter.pp     tor_exit_node.pp     update_special_pages.pp

misc/maintenance.pp is gone

Change 238626 had a related patch set uploaded (by Dzahn):
delete misc/maintenance.pp, now empty

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

Change 238626 merged by Dzahn:
delete misc/maintenance.pp, now empty

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

Dzahn removed a project: Patch-For-Review.