Change Details

``` :~/puppet/modules/mediawiki$ grep -ir silver * manifests/maintenance/updatequerypages.pp: mediawiki::maintenance::updatequerypages::cronjob { ['s1@11', 's2@12', 's3@13', 's4@14', 's5@15', 's6@16', 's7@17', 's8@18', 'silver@19']: } manifests/maintenance/refreshlinks.pp: mediawiki::maintenance::refreshlinks::cronjob { ['s1@1', 's2@2', 's3@3', 's4@4', 's5@5', 's6@6', 's7@7', 's8@8', 'silver@9']: } ``` I don't at all understand what those are. Probably the references to silver just need to be removed, but it seems like they should then be replaced with references to m5 (or labweb? I'm not clear.) --- After discussion with @jcrespo we have this list of remediation steps: [x] Rename the new wikitech section from `m5` to something sensible like "wikitech". Using the same label for a MediaWiki LBFactory label and a layer 4 proxy causes confusion. [x] Rename silver.dblist to the same name as the section, e.g. wikitech.dblist, including labswiki and maybe labstestwiki (See related issues at T192339). This should be done in two steps: first introduce the new dblist name, then later remove the old one after ensuring it is no longer used. [x] Check and fix old references to silver dblist on mediawiki configuration and puppet [x] In particular, change operations/puppet:modules/mediawiki/manifests/maintenance/{updatequerypages.pp,refreshlinks.pp} to the new dblist name [] Reference wikitech section databases exclusively by `A` DNS entry (db1073 currently) rather than by its `CNAME` alias to avoid double load-balancing (mediawiki AND misc) and make TLS possible

``` :~/puppet/modules/mediawiki$ grep -ir silver * manifests/maintenance/updatequerypages.pp: mediawiki::maintenance::updatequerypages::cronjob { ['s1@11', 's2@12', 's3@13', 's4@14', 's5@15', 's6@16', 's7@17', 's8@18', 'silver@19']: } manifests/maintenance/refreshlinks.pp: mediawiki::maintenance::refreshlinks::cronjob { ['s1@1', 's2@2', 's3@3', 's4@4', 's5@5', 's6@6', 's7@7', 's8@8', 'silver@9']: } ``` I don't at all understand what those are. Probably the references to silver just need to be removed, but it seems like they should then be replaced with references to m5 (or labweb? I'm not clear.) --- After discussion with @jcrespo we have this list of remediation steps: [] Rename the new wikitech section from `m5` to something sensible like "wikitech". Using the same label for a MediaWiki LBFactory label and a layer 4 proxy causes confusion. [x] Rename silver.dblist to the same name as the section, e.g. wikitech.dblist, including labswiki and maybe labstestwiki (See related issues at T192339). This should be done in two steps: first introduce the new dblist name, then later remove the old one after ensuring it is no longer used. [x] Check and fix old references to silver dblist on mediawiki configuration and puppet [x] In particular, change operations/puppet:modules/mediawiki/manifests/maintenance/{updatequerypages.pp,refreshlinks.pp} to the new dblist name [] Reference wikitech section databases exclusively by `A` DNS entry (db1073 currently) rather than by its `CNAME` alias to avoid double load-balancing (mediawiki AND misc) and make TLS possible

``` :~/puppet/modules/mediawiki$ grep -ir silver * manifests/maintenance/updatequerypages.pp: mediawiki::maintenance::updatequerypages::cronjob { ['s1@11', 's2@12', 's3@13', 's4@14', 's5@15', 's6@16', 's7@17', 's8@18', 'silver@19']: } manifests/maintenance/refreshlinks.pp: mediawiki::maintenance::refreshlinks::cronjob { ['s1@1', 's2@2', 's3@3', 's4@4', 's5@5', 's6@6', 's7@7', 's8@8', 'silver@9']: } ``` I don't at all understand what those are. Probably the references to silver just need to be removed, but it seems like they should then be replaced with references to m5 (or labweb? I'm not clear.) --- After discussion with @jcrespo we have this list of remediation steps: [x] Rename the new wikitech section from `m5` to something sensible like "wikitech". Using the same label for a MediaWiki LBFactory label and a layer 4 proxy causes confusion. [x] Rename silver.dblist to the same name as the section, e.g. wikitech.dblist, including labswiki and maybe labstestwiki (See related issues at T192339). This should be done in two steps: first introduce the new dblist name, then later remove the old one after ensuring it is no longer used. [x] Check and fix old references to silver dblist on mediawiki configuration and puppet [x] In particular, change operations/puppet:modules/mediawiki/manifests/maintenance/{updatequerypages.pp,refreshlinks.pp} to the new dblist name [] Reference wikitech section databases exclusively by `A` DNS entry (db1073 currently) rather than by its `CNAME` alias to avoid double load-balancing (mediawiki AND misc) and make TLS possible