Fix up the UploadWizard-api for beta cluster
Closed, ResolvedPublic
Actions

Description

UploadWizard-api smoke test is broken. Should be enhanced / unbroke and have it ping release engineering on IRC whenever it is broken.

While at it, move the JJB definition out of browsertests.yaml things, since that is going away soonish ™

Details

	Subject	Repo	Branch	Lines +/-
	tests: fix PIL requirement definition	mediawiki/extensions/UploadWizard	master	+1 -3

Customize query in gerrit

Related Objects

Mentioned In: T271666: UploadWizard api test requirements.txt is out of date
T254149: upload-wizard_tests.py does not work with Python3
Mentioned Here: T115465: Stop running the UploadWizard API smoke tests in production

Event Timeline

hashar created this task.May 9 2016, 8:24 AM

Restricted Application added a project: Multimedia. · View Herald TranscriptMay 9 2016, 8:24 AM

Restricted Application added subscribers: Zppix, Steinsplitter, Aklapper. · View Herald Transcript

hashar moved this task from To Triage to Next: Feature on the Beta-Cluster-Infrastructure board.May 9 2016, 8:25 AM

Restricted Application added a subscriber: Matanya. · View Herald TranscriptMay 9 2016, 8:25 AM

hashar triaged this task as Low priority.May 9 2016, 8:25 AM

hashar moved this task from Next: Feature to Next: Maintenance on the Beta-Cluster-Infrastructure board.

hashar added a project: Essential-Work.

Where would I even find the UploadWizard-api smoke test? I was under the impression that we deleted all the broken browser tests.

It is not an a browser tests / Selenium based job. The code is some python scripts in mediawiki/extensions/UploadWizard which is more of an integration tests targeting the production commons.wikimedia.org

Was/is meant to assert that UploadWizard works properly on production. Turned out to be broken because the Python module PIL got relocated to some other place.

Ideally, the script should be ported from PIL to pillow.

hashar removed hashar as the assignee of this task.May 10 2016, 12:52 PM

Oh, I see it now. But please don't restore running it for production. We agreed previously not to do this (T115465).

matmarex removed projects: Multimedia, UploadWizard.May 12 2016, 5:38 PM

Change 289634 had a related patch set uploaded (by Hashar):
tests: fix PIL requirement definition

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

gerritbot added a project: Patch-For-Review.May 19 2016, 11:44 AM

Change 289634 merged by jenkins-bot:
tests: fix PIL requirement definition

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