Page MenuHomePhabricator

WikiLambda: Translate "not connected" errors from FunctionCall and PerformTest APIs into ZErrors
Closed, ResolvedPublic

Description

When there's a connection error, the returned value is "apierror-wikilambda_function_call-not-connected". We should translate this into a ZErrorType built-in and a Z5 error

Event Timeline

Jdforrester-WMF lowered the priority of this task from Medium to Low.Feb 22 2024, 5:53 PM
Jdforrester-WMF moved this task from To Triage to Backlog on the Abstract Wikipedia team board.

Change #1194726 had a related patch set uploaded (by Arpitajain28; author: Arpitajain28):

[mediawiki/extensions/WikiLambda@master] T305612: Translate connection errors to ZErrorType built-in and Z5 errors

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

I have subscribed your task and tried to resolve the issue mentioned in the description. You can check and tell me the changes required.

Hello, @Arpitajain28! Thank you for your contribution!

We have recently merged significant changes in the files you worked on, so I rebased your patch on top of those changes.

Adding a new builtin ZError requires a bit more work; the existing builtin errors are in function-schemata/data/definitions (https://gitlab.wikimedia.org/repos/abstract-wiki/wikifunctions/function-schemata/-/tree/main/data/definitions) and should be added there first. You can see a table of the current errors in here, too: https://meta.wikimedia.org/wiki/Abstract_Wikipedia/Representation_of_errors

So, the next free slot should be Z576, but I'd suggest using the Z529 one, so that it's contiguous to Z530/Api error (which seems closely related).

The process to add an error would be:

  1. Add the new ZObject to function-schemata/data/definitions, and get it merged
  2. Once that is merged, we would update wikilambda reference of function-schemata in an isolated patch (like this one https://gerrit.wikimedia.org/r/c/mediawiki/extensions/WikiLambda/+/1192975)
  3. Then, your patch could be merged (this would make sure that the object referred to i.e. Z529 exists as a built-in)

I can get function-schemata ready for your changes and follow-up in gerrit.

Change #1203549 had a related patch set uploaded (by Jforrester; author: Jforrester):

[mediawiki/extensions/WikiLambda@master] Update function-schemata sub-module to HEAD (5bf82a1)

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

Change #1203549 merged by jenkins-bot:

[mediawiki/extensions/WikiLambda@master] Update function-schemata sub-module to HEAD (91e3ffe)

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

Change #1194726 had a related patch set uploaded (by Genoveva Galarza; author: Arpitajain28):

[mediawiki/extensions/WikiLambda@master] Translate connection errors to ZErrorType built-in and Z5 errors

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

Change #1194726 merged by jenkins-bot:

[mediawiki/extensions/WikiLambda@master] Translate connection errors to ZErrorType built-in and Z5 errors

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

Change #1206919 had a related patch set uploaded (by Genoveva Galarza; author: Genoveva Galarza):

[mediawiki/extensions/WikiLambda@master] Return Z529/Connection error in other applicable instances

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

Change #1206919 merged by jenkins-bot:

[mediawiki/extensions/WikiLambda@master] Return Z529/Connection error in other applicable instances

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

Change #1206981 had a related patch set uploaded (by Jforrester; author: Jforrester):

[operations/deployment-charts@master] wikifunctions: Upgrade orchestrator from 2025-11-08-223341 to 2025-11-18-175356

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

Change #1206981 merged by jenkins-bot:

[operations/deployment-charts@master] wikifunctions: Upgrade orchestrator from 2025-11-08-223341 to 2025-11-18-175356

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

Change #1211872 had a related patch set uploaded (by Cory Massaro; author: Cory Massaro):

[operations/deployment-charts@master] wikifunctions: Upgrade evaluators from 2025-11-12-122736 to 2025-11-17-175029

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