Page MenuHomePhabricator

Remove all usages of $::mw_primary on puppet
Closed, ResolvedPublic

Description

Currentl, $::mw_primary is just used to alert and a few secondary changes on databases, remove it and use etcd instead.

This was the patch: https://gerrit.wikimedia.org/r/#/c/operations/puppet/+/345346/

Event Timeline

jcrespo triaged this task as High priority.Jul 9 2018, 5:08 PM
jcrespo moved this task from Triage to Pending comment on the DBA board.

@Volans @Joe @Marostegui I have prepared 2 approaches to achieve this, one is to call conftool on the perl check script and return a CRIT or WARNING based on the host dc: https://gerrit.wikimedia.org/r/345346

The other is to use the conftool puppet function (does it work, I don't see it being in use?) to decide it beforehand and configuring if it is paging or not: https://gerrit.wikimedia.org/r/449742 Please give me any feedback you can about what is the best (or possible) approach to remove all uses of $::mw_primary puppet variable.

jcrespo moved this task from Pending comment to In progress on the DBA board.

$::mw_primary is removed from puppet now. Resolving this.

Change 461012 had a related patch set uploaded (by Dzahn; owner: Dzahn):
[operations/puppet@production] mediawiki_maintenance: remove mw_primary remnant

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

Change 461012 merged by Dzahn:
[operations/puppet@production] mediawiki_maintenance: remove mw_primary remnant

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

Change 461013 had a related patch set uploaded (by Dzahn; owner: Dzahn):
[operations/puppet@production] mediawiki_maintenance: reverse absent/present for inactive motd

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

Change 461013 merged by Dzahn:
[operations/puppet@production] mediawiki_maintenance: reverse absent/present for inactive motd

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