Page MenuHomePhabricator

Linting is disabled on beta cluster, but needs to be enabled
Closed, ResolvedPublic

Description

Seen while testing RESTBase. Calling transform/wikitext/to/lint with the body wikitext: '<div>No div ending' produces different results between Parsoid/JS and Parsoid/PHP:

JS
body: [
  { type: 'missing-end-tag', params: [Object], dsr: [Object] }
]
PHP
body: []

Details

Related Gerrit Patches:
operations/mediawiki-config : masterParsoid/PHP: Load the extension on all Parsoid nodes

Event Timeline

mobrovac triaged this task as High priority.Thu, Oct 24, 2:20 PM
mobrovac created this task.
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptThu, Oct 24, 2:20 PM

Linting has been implemented and tested. This is a config issue.
From wmf-config file:

// While we are testing & benchmarking, make production db access read-only
$wgReadOnly = $wmfRealm !== 'labs';
$wgParsoidSettings = [
    'useSelser' => true,
    // Temporarily condition on $wgReadOnly while testing & benchmarking
    // so we don't try to update lint information in the production dbs.
    'linting' => (bool)$wgReadOnly
];

So we need a config fix for beta.

Thinking about this, shouldn't this be

linting => !$wgReadOnly

since we should be linting if MW is not in read-only mode?

It was set that way because scandium was readonly ... and linting was safe because posting of lints to the db was conditioned on $wgReadOnly.
We want the same behavior for production in these next few weeks .. i.e. and lint and find crashers if any .. but don't post.
For now, we probably need a special condition for beta.

Once we are ready to go live, we can change it to what you suggest there and have a special case for scandium (instead of beta).
Does that sound reasonable?

ssastry renamed this task from Parsoid/PHP's wt2lint doesn't detect (simple?) errors to Linting is disabled on beta cluster, but needs to be enabled.Thu, Oct 24, 3:23 PM

Change 544878 had a related patch set uploaded (by Mobrovac; owner: Mobrovac):
[operations/mediawiki-config@master] Parsoid/PHP: Load the extension on all Parsoid nodes

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

Change 544878 merged by jenkins-bot:
[operations/mediawiki-config@master] Parsoid/PHP: Load the extension on all Parsoid nodes

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

Mentioned in SAL (#wikimedia-operations) [2019-10-24T17:26:00Z] <mobrovac@deploy1001> Synchronized wmf-config/CommonSettings.php: Enable Parsoid/PHP in the whole wtp (a.k.a. Parsoid) cluster - T236388 (duration: 00m 53s)

mobrovac closed this task as Resolved.Thu, Oct 24, 6:03 PM
mobrovac claimed this task.