Page MenuHomePhabricator

Warn if localSiteID is not a known site ID
Open, NormalPublic

Description

The localSiteID setting must refer to a site that is known to the SiteLookup service. If this is not the case, a warning should be raised - probably a production warning (wfLogWarning), but at least a devlopment warning (wfWarn).

This warnign existed until it was removed by I1a52ce07943 to fix a problem with qunit tests, see T153729 and T153597. However, that problem would better be addressed by fixing the CI setup, instead of removing the warning that indicates a configuration error.

Event Timeline

daniel created this task.Dec 21 2016, 12:21 PM
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptDec 21 2016, 12:21 PM
daniel triaged this task as Unbreak Now! priority.Dec 21 2016, 12:22 PM
Restricted Application added subscribers: Jay8g, TerraCodes. · View Herald TranscriptDec 21 2016, 12:22 PM

Change 328504 had a related patch set uploaded (by Daniel Kinzler):
[WIP] Revert "Remove warning about missing site ID"

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

thiemowmde lowered the priority of this task from Unbreak Now! to Normal.
thiemowmde moved this task from incoming to consider for next sprint on the Wikidata board.

Change 328504 abandoned by Addshore:
[WIP] Revert "Remove warning about missing site ID"

Reason:
Build is dead

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