Last modified: 2013-05-08 23:06:44 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T50090, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 48090 - Concurrent uploading is broken
Concurrent uploading is broken
Status: RESOLVED FIXED
Product: MediaWiki extensions
Classification: Unclassified
UploadWizard (Other open bugs)
unspecified
All All
: High major (vote)
: ---
Assigned To: Nobody - You can work on this!
: browser-test-bug
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-05-05 01:39 UTC by Erik Moeller
Modified: 2013-05-08 23:06 UTC (History)
5 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---
greg: Backport_WMF+


Attachments

Description Erik Moeller 2013-05-05 01:39:49 UTC
Although I have "3" as my maximum number of concurrent uploads, Upload Wizard now uploads files one at a time. (Chrome/Ubuntu)
Comment 1 Gerrit Notification Bot 2013-05-05 06:30:47 UTC
Related URL: https://gerrit.wikimedia.org/r/62320 (Gerrit Change I0d9f13c6b7698056d4f9ed113b9b23b2f266263d)
Comment 2 Mark Holmquist 2013-05-06 16:52:41 UTC
Merged - marking as fixed, as it worked locally, but we should revisit when it's deployed and maybe set up some browser testing with Chris.
Comment 3 Chris McMahon 2013-05-06 17:03:10 UTC
Adding Zeljko to this, also made a note on test backlog page for browser test coverage of concurrent uploads in UW.
Comment 4 Erik Moeller 2013-05-06 17:09:51 UTC
Mark, since this is a pretty major regression, do you think you'd be able to arrange for a quick deploy this week? Otherwise we'll have to wait til May 15. 

If we set up a deploy window this week, we should try to get a fix for Opera in as well, which appears completely broken right now (bug 48091).
Comment 5 Mark Holmquist 2013-05-06 17:15:32 UTC
That's possible, though I haven't deployed to the cluster yet, so it would be best to have someone ride along.
Comment 6 Mark Holmquist 2013-05-06 20:32:41 UTC
kaldari and I are planning on deploying on Wednesday.
Comment 7 Greg Grossmeier 2013-05-07 17:20:24 UTC
Scheduled for Wednesday May 8th at 1:30 Pacific (20:30 UTC).

(btw, yes, this bug is now "resolved" and I've set the Backport flag to "+" (accepting it) but it really isn't out there and deployed yet, unfortunately there is currently no way to indicate that in BZ... yet)
Comment 8 Mark Holmquist 2013-05-08 21:50:19 UTC
Deployed to Commons today. Please test and confirm.
Comment 9 Erik Moeller 2013-05-08 22:56:16 UTC
Confirmed fixed in FF and Chrome. (Pref now shows "0" as an option for concurrent uploads, we should fix that, but that's trivial and can wait.)
Comment 10 Mark Holmquist 2013-05-08 23:06:44 UTC
Yeah, I guess I should have left in the unset and added a 'default' => 'default' member instead. I'll submit a separate enh bug.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links