Flow shouldn't assume Parsoid is source of all VRS errors
Closed, ResolvedPublic
Actions

Assigned To

Authored By

	cscott
	Sep 14 2015, 6:41 PM

Description

Since VRS backend could be either Parsoid or RESTBase, the error messages should reflect this. See T112330.

Details

	Subject	Repo	Branch	Lines +/-
	Clarify that both Parsoid and RESTBase can respond to VRS requests	mediawiki/extensions/Flow	master	+19 -16

Customize query in gerrit

Related Objects
Search...

		Status	Subtype	Assigned	Task
		Resolved		• GWicke	T112330 In current VRS deployment scenarios, error logging should not assume Parsoid is the source of all errors.
		Resolved		• Catrope	T112566 Flow shouldn't assume Parsoid is source of all VRS errors

Event Timeline

cscott created this task.Sep 14 2015, 6:41 PM

cscott raised the priority of this task from to Needs Triage.

cscott updated the task description. (Show Details)

cscott added a project: StructuredDiscussions.

cscott added subscribers: Aklapper, cscott.

Restricted Application added a project: Collaboration-Team-Triage. · View Herald TranscriptSep 14 2015, 6:41 PM

cscott added a parent task: T112330: In current VRS deployment scenarios, error logging should not assume Parsoid is the source of all errors..Sep 14 2015, 6:41 PM

Which error messages are you talking about exactly?

• Catrope triaged this task as Medium priority.Sep 23 2015, 11:35 PM

Change 242929 had a related patch set uploaded (by Cscott):
Clarify that both Parsoid and RESTBase can respond to VRS requests

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

gerritbot added a project: Patch-For-Review.Oct 1 2015, 5:34 PM

@Catrope hopefully the patch answers your question. Flow doesn't have quite the same proliferation of "parsoiderror" types, but there are still some things that could be clarified.

Change 242929 merged by jenkins-bot:
Clarify that both Parsoid and RESTBase can respond to VRS requests

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

Diffusion mentioned this in rMEXT332977245ef5: Updated mediawiki/extensions Project: mediawiki/extensions/Flow….Oct 2 2015, 2:56 PM

cscott mentioned this in rEFLWf17e620354ff: Clarify that both Parsoid and RESTBase can respond to VRS requests.Oct 2 2015, 2:57 PM

ReleaseTaggerBot added a project: MW-1.27-release (WMF-deploy-2015-10-06_(1.27.0-wmf.2)).Oct 2 2015, 3:00 PM

• Catrope closed this task as Resolved.Feb 13 2016, 12:26 AM

• Catrope claimed this task.