mmodell (Mukunda Modell)Administrator
Release Engineer, Phabricator Admin

Projects (28)

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

User Details

User Since
Sep 19 2014, 9:45 AM (148 w, 3 d)
Roles
Administrator
Availability
Available
IRC Nick
twentyafterfour
LDAP User
20after4
MediaWiki User
MModell (WMF)

I am a Release Engineer on the the Release-Engineering-Team.


Current projects: Phabricator, Release-Engineering-Team (Long-Lived-Branches) and Scap


Besides working on Deployment-Systems, I am responsible for Phabricator maintenance & operations. I also help out with other phabricator-related tasks as a member of Project-Admins and Repository-Admins.

Disclaimer: I work for or provide services to the Wikimedia Foundation. However, the Foundation does not vet all my activity, so edits, statements, or other contributions made by this account may not reflect the views of the Foundation.`

Recent Activity

Today

mmodell closed T170458: Set up continuous integration for wikistats 2.0 UI as Resolved.
Mon, Jul 24, 7:49 PM · Release-Engineering-Team (Kanban), Continuous-Integration-Config, Analytics-Wikistats, Analytics-Kanban
mmodell closed T170458: Set up continuous integration for wikistats 2.0 UI, a subtask of T170457: Define, Document (and test) Desktop and Mobile browser support for wikistats 2.0, as Resolved.
Mon, Jul 24, 7:49 PM · Analytics-Wikistats, Analytics-Kanban
mmodell accepted D725: MediaWiki: only fail if 1/4 canaries fail.
Mon, Jul 24, 6:21 PM · Release-Engineering-Team
mmodell moved T170458: Set up continuous integration for wikistats 2.0 UI from Backlog to In-progress on the Release-Engineering-Team (Kanban) board.

I've created the meta-ci-jessie tag so CI should be happening now.

Mon, Jul 24, 4:41 PM · Release-Engineering-Team (Kanban), Continuous-Integration-Config, Analytics-Wikistats, Analytics-Kanban
mmodell added a comment to T170458: Set up continuous integration for wikistats 2.0 UI.

Are you using differential for code review? If so, I just need to set up the repo with harbormaster so that we run npm test on new differential revisions.

Mon, Jul 24, 4:24 PM · Release-Engineering-Team (Kanban), Continuous-Integration-Config, Analytics-Wikistats, Analytics-Kanban
mmodell added a comment to T170793: Create Phame blog for Readers department.

@mmodell: Would you be able to edit the policy such that @Jdlrobson and @Jhernandez (Senior Engineers, Reading Web) could edit the bloggy blog?

Mon, Jul 24, 3:57 PM · Release-Engineering-Team (Kanban), Phabricator, Reading-Web-Backlog (Tracking)
mmodell changed the edit policy for Leave it to the prose.
Mon, Jul 24, 3:57 PM · Reading-Community-Engagement

Yesterday

mmodell added a comment to T167244: Rename and update Cloud Services Phabricator projects.

It shouldn't be too difficult to rename them in phab while retaining the old git urls for compatability.

Sun, Jul 23, 2:39 AM · User-bd808, Project-Admins, cloud-services-team (Kanban)

Fri, Jul 21

mmodell added a comment to T167244: Rename and update Cloud Services Phabricator projects.

Even after removing the two subprojects it still refuses. I guess I'll have to do some manual fiddling with the database.

Fri, Jul 21, 8:14 PM · User-bd808, Project-Admins, cloud-services-team (Kanban)
mmodell added a comment to T167244: Rename and update Cloud Services Phabricator projects.

One of them didn't work though:

Fri, Jul 21, 5:46 PM · User-bd808, Project-Admins, cloud-services-team (Kanban)

Wed, Jul 19

mmodell updated subscribers of T76094: Assign due date to task and have tasks with due dates displayed on a calendar.

I would also like to add some more advanced functionality related to this task

Wed, Jul 19, 10:21 PM · Upstream, Phabricator (Upstream)
mmodell added a comment to T76094: Assign due date to task and have tasks with due dates displayed on a calendar.

@Aklapper: I don't really have a good idea of timeline, I want to get a bit of feedback and experience with the functionality before making any firm commitments ;)

Wed, Jul 19, 10:11 PM · Upstream, Phabricator (Upstream)
mmodell awarded rMSCA6c36daed0976: Create a wrapper around conftool for our pooling/depooling needs a Mountain of Wealth token.
Wed, Jul 19, 10:02 PM
mmodell created an object: Phabricator (2017-07-19).
Wed, Jul 19, 9:56 PM
mmodell added a comment to T116627: Include 5xx numbers in fluorine fatalmonitor.

Interesting...

Wed, Jul 19, 6:28 PM · Operations

Tue, Jul 18

mmodell closed T170793: Create Phame blog for Readers department as Resolved.

Leave it to the prose

Tue, Jul 18, 8:47 PM · Release-Engineering-Team (Kanban), Phabricator, Reading-Web-Backlog (Tracking)
mmodell created Leave it to the prose.
Tue, Jul 18, 8:41 PM · Reading-Community-Engagement
mmodell added a comment to T170793: Create Phame blog for Readers department.

So who should be able to post to this blog?

Tue, Jul 18, 3:46 AM · Release-Engineering-Team (Kanban), Phabricator, Reading-Web-Backlog (Tracking)
mmodell committed rPHDEP3d728e1f6bb6: submodule bump (authored by mmodell).
submodule bump
Tue, Jul 18, 3:39 AM
mmodell added a comment to D716: Lower number of MetaMTA failures from 250 to 100.

@demon: I've landed this but this repo has autoclose disabled so you'll have to close the diff manually.

Tue, Jul 18, 3:37 AM
mmodell accepted D716: Lower number of MetaMTA failures from 250 to 100.
Tue, Jul 18, 3:32 AM
mmodell added a comment to T170444: On beta commons, thumbnailing of 3D files is broken still.

@MarkTraceur: does this resolve the issue for you? If so then we just need to get the patch merged so that it doesn't have to remain cherry picked forever.

Tue, Jul 18, 3:29 AM · Multimedia, Thumbor, Beta-Cluster-Infrastructure
mmodell triaged T170444: On beta commons, thumbnailing of 3D files is broken still as Normal priority.
Tue, Jul 18, 3:28 AM · Multimedia, Thumbor, Beta-Cluster-Infrastructure
mmodell added a comment to T170444: On beta commons, thumbnailing of 3D files is broken still.

fixed (or rather, worked around) T166013 by cherry picking https://gerrit.wikimedia.org/r/#/c/365891/

Tue, Jul 18, 3:28 AM · Multimedia, Thumbor, Beta-Cluster-Infrastructure
mmodell triaged T166013: deployment-imagescaler01 has no mwdeploy user as Normal priority.
Tue, Jul 18, 3:27 AM · Release-Engineering-Team (Kanban), Patch-For-Review, Deployment-Systems, Beta-Cluster-Infrastructure
mmodell added a comment to T166013: deployment-imagescaler01 has no mwdeploy user.

cherry picked https://gerrit.wikimedia.org/r/#/c/365891/ on beta puppetmaster. This works around the problem and may actually be the more correct solution.

Tue, Jul 18, 3:27 AM · Release-Engineering-Team (Kanban), Patch-For-Review, Deployment-Systems, Beta-Cluster-Infrastructure
mmodell added a comment to T166013: deployment-imagescaler01 has no mwdeploy user.

So the user's home directory is set in scap::target. So we either fix it in ldap or remove that line from scap::target (I think the former as scap::target is correct for production)

Tue, Jul 18, 2:50 AM · Release-Engineering-Team (Kanban), Patch-For-Review, Deployment-Systems, Beta-Cluster-Infrastructure
mmodell added a comment to T166013: deployment-imagescaler01 has no mwdeploy user.

@hashar: Nothing wrong with the patch in gerrit, as far as I can see. I don't know where puppet is trying to change the homedir but I will try to track that down. I also don't know how to edit the user in ldap but that might be the right thing to do.

Tue, Jul 18, 2:39 AM · Release-Engineering-Team (Kanban), Patch-For-Review, Deployment-Systems, Beta-Cluster-Infrastructure
mmodell moved T93499: Add support for task types from Backlog to Solved upstream on the Phabricator (Upstream) board.
Tue, Jul 18, 2:25 AM · Release-Engineering-Team (Kanban), Phabricator (Upstream), Upstream
mmodell claimed T93499: Add support for task types.
Tue, Jul 18, 2:24 AM · Release-Engineering-Team (Kanban), Phabricator (Upstream), Upstream
mmodell added a comment to T93499: Add support for task types.

@NHarateh_WMF: I don't have a clear plan for rolling out new subtypes. The UI for managing the types is pretty primitive and it's time consuming to add them because existing forms must be updated to account for all the new types.

Tue, Jul 18, 2:24 AM · Release-Engineering-Team (Kanban), Phabricator (Upstream), Upstream

Mon, Jul 17

mmodell closed T167221: Phabricator max exec time exceeded when going to https://phabricator.wikimedia.org/guides/quickstart/ as Resolved.

The guides application is now uninstalled.

Mon, Jul 17, 10:31 PM · Release-Engineering-Team (Kanban), User-Zppix, Phabricator
mmodell added a comment to T167221: Phabricator max exec time exceeded when going to https://phabricator.wikimedia.org/guides/quickstart/.

There is no reason for you to visit the quickstart setup guide. That page is for administrators.

Mon, Jul 17, 6:30 PM · Release-Engineering-Team (Kanban), User-Zppix, Phabricator
mmodell added a comment to E206: Code Review Office Hours.

There are now next / previous links in the recurring event section above.

Mon, Jul 17, 4:55 PM · events, Release-Engineering-Team, Developer-Relations, IRC-Meetings, Patch-For-Review
mmodell added a comment to T168699: Verify that the codfw lvs is configured correctly for Phabricator.

Still waiting on merge of https://gerrit.wikimedia.org/r/#/c/355869/

Mon, Jul 17, 4:43 PM · Traffic, Release-Engineering-Team (Kanban), Phabricator, Operations
mmodell added a comment to T114488: Automate the recurring management of wikitech:Deployments and phab:#train_deployments.

This is still progressing: Current status is I got down a rabbit hole learning lua + scribunto.

Mon, Jul 17, 4:41 PM · Release-Engineering-Team (Kanban), Deployment-Systems
mmodell edited projects for T143175: Configure phabricator clustering for daemons and repositories, added: Release-Engineering-Team (Backlog); removed Release-Engineering-Team (Kanban).
Mon, Jul 17, 4:40 PM · Release-Engineering-Team (Backlog), WorkType-NewFunctionality, Availability, Phabricator
mmodell placed T143175: Configure phabricator clustering for daemons and repositories up for grabs.
Mon, Jul 17, 4:40 PM · Release-Engineering-Team (Backlog), WorkType-NewFunctionality, Availability, Phabricator
mmodell placed T168556: Custom fields for task subtypes fatal on pages > 1 with: Query "ManiphestTaskQuery" failed to return a value from getPagingValueMap() up for grabs.
Mon, Jul 17, 4:32 PM · Upstream, Phabricator (Upstream), Release-Engineering-Team (Kanban)
mmodell updated the task description for T137928: Deploy phabricator to phab2001.codfw.wmnet.
Mon, Jul 17, 4:28 PM · Release-Engineering-Team (Kanban), Patch-For-Review, WorkType-NewFunctionality, Availability, Phabricator
mmodell updated the task description for T137928: Deploy phabricator to phab2001.codfw.wmnet.
Mon, Jul 17, 4:28 PM · Release-Engineering-Team (Kanban), Patch-For-Review, WorkType-NewFunctionality, Availability, Phabricator

Sat, Jul 15

mmodell committed rPHEXd10a629d17ac: delete gitreview (authored by mmodell).
delete gitreview
Sat, Jul 15, 2:43 AM
mmodell committed rPHEXc697f4327138: Add missing function AddActionLinksToBottom (authored by Paladox).
Add missing function AddActionLinksToBottom
Sat, Jul 15, 2:40 AM
mmodell closed D721: Add missing function AddActionLinksToBottom by committing rPHEXc697f4327138: Add missing function AddActionLinksToBottom (authored by Paladox).
Sat, Jul 15, 2:40 AM
mmodell accepted D721: Add missing function AddActionLinksToBottom.
Sat, Jul 15, 2:36 AM
mmodell changed the edit policy for Clouds & Unicorns.
Sat, Jul 15, 12:19 AM · cloud-services-team

Fri, Jul 14

mmodell committed rPHDEP49678012a992: submodule updates (authored by mmodell).
submodule updates
Fri, Jul 14, 8:35 PM
mmodell committed rPHEX3c06822e3b84: Add support for the new redesgned diffusion (authored by Paladox).
Add support for the new redesgned diffusion
Fri, Jul 14, 8:08 PM
mmodell closed D720: Add support for the new redesgned diffusion by committing rPHEX3c06822e3b84: Add support for the new redesgned diffusion (authored by Paladox).
Fri, Jul 14, 8:08 PM
mmodell updated the diff for D720: Add support for the new redesgned diffusion.

use a dropdown menu for the download links

Fri, Jul 14, 1:22 AM
mmodell updated the diff for D718: Add back support for CustomGithubDownloadLinks.

use dropdown menu for download links

Fri, Jul 14, 1:21 AM

Thu, Jul 13

mmodell updated the diff for D718: Add back support for CustomGithubDownloadLinks.

fixed lint for naming conventions and line length.

Thu, Jul 13, 11:23 PM
mmodell accepted D718: Add back support for CustomGithubDownloadLinks.

looks good.

Thu, Jul 13, 11:17 PM
mmodell added a comment to T163938: replace sdb and then setup/install phab1001.eqiad.wmnet.

@Dzahn hmm, I thought it was working before? I think I even logged into it but maybe I am remembering incorrectly.

Thu, Jul 13, 9:30 PM · Release-Engineering-Team (Kanban), Phabricator, Operations
mmodell added inline comments to D718: Add back support for CustomGithubDownloadLinks.
Thu, Jul 13, 9:29 PM
mmodell added inline comments to D718: Add back support for CustomGithubDownloadLinks.
Thu, Jul 13, 9:27 PM
mmodell created T170637: 1.30.0-wmf.19 deployment blockers.
Thu, Jul 13, 8:46 PM · Train Deployments, Release
mmodell created T170636: 1.30.0-wmf.18 deployment blockers.
Thu, Jul 13, 8:46 PM · Train Deployments, Release
mmodell created T170635: 1.30.0-wmf.17 deployment blockers.
Thu, Jul 13, 8:45 PM · Train Deployments, Release
mmodell created T170634: 1.30.0-wmf.16 deployment blockers.
Thu, Jul 13, 8:44 PM · Train Deployments, Release
mmodell created T170633: 1.30.0-wmf.15 deployment blockers.
Thu, Jul 13, 8:44 PM · Train Deployments, Release-Engineering-Team (Kanban), Release
mmodell created T170632: 1.30.0-wmf.14 deployment blockers.
Thu, Jul 13, 8:43 PM · Train Deployments, Release-Engineering-Team (Kanban), Release
mmodell created T170631: 1.30.0-wmf.13 deployment blockers.
Thu, Jul 13, 8:43 PM · Train Deployments, Release-Engineering-Team (Kanban), Release
mmodell requested changes to D719: Add support for the new redesgned diffusion.
Thu, Jul 13, 8:41 PM
mmodell added inline comments to D718: Add back support for CustomGithubDownloadLinks.
Thu, Jul 13, 8:39 PM
mmodell requested changes to D718: Add back support for CustomGithubDownloadLinks.
Thu, Jul 13, 8:37 PM
mmodell accepted D718: Add back support for CustomGithubDownloadLinks.
Thu, Jul 13, 8:34 PM
mmodell edited Description on Phabricator.
Thu, Jul 13, 8:27 PM
mmodell edited Description on Phabricator.
Thu, Jul 13, 8:24 PM
mmodell accepted D717: Add support for the new redesgned diffusion.
Thu, Jul 13, 8:14 PM

Wed, Jul 12

mmodell committed rPHDEP33b2f45b17b5: Merge upstream/master into rPHAB/wmf/stable (authored by mmodell).
Merge upstream/master into rPHAB/wmf/stable
Wed, Jul 12, 6:20 PM
mmodell triaged T150510: Automatically add "patch-for-review" tag when `arc diff` as Low priority.
Wed, Jul 12, 6:05 PM · Developer-Wishlist (2017), Phabricator, Differential
mmodell added a comment to T102550: Use semantic versioning for services (for consistency with mediawiki core).

So this should be resolved?

Wed, Jul 12, 5:58 PM · Services (watching), Deployment-Systems
mmodell added a comment to T170444: On beta commons, thumbnailing of 3D files is broken still.

Nobody from Release-Engineering-Team are familiar with Thumbor, maybe @Gilles can take a look at it?

Wed, Jul 12, 5:54 PM · Multimedia, Thumbor, Beta-Cluster-Infrastructure
mmodell added a project to T170444: On beta commons, thumbnailing of 3D files is broken still: Thumbor.
Wed, Jul 12, 5:51 PM · Multimedia, Thumbor, Beta-Cluster-Infrastructure
mmodell added a comment to T98326: Search by commenter on Phabricator.

I don't think it would be very difficult to implement this. It would also be nice to have a way to search the full-text of comments directly as well as a feature to browse all comments by user. Browsing comments would be a little more difficult and searching comments is currently possible but only indirectly, as the comment text is indexed with the task so you can find tasks by words used in a comment but you can't directly find the comment its self.

Wed, Jul 12, 3:27 AM · Upstream, Phabricator (Upstream)
mmodell closed T113446: Impossible to unsubscribe from calendar event as Resolved.

I suspect this has been resolved since we haven't heard further complaints. Please reopen if you are still experiencing trouble.

Wed, Jul 12, 3:23 AM · Release-Engineering-Team (Kanban), Upstream, Phabricator (Upstream)
mmodell closed T883: Duplicate tasks are not listed in or near the description of the target task as Resolved.
Wed, Jul 12, 3:19 AM · Release-Engineering-Team (Kanban), Developer-Wishlist (2017), Upstream, Phabricator (Upstream)
mmodell updated the image for Diffusion from F4064353: profile to F8709254: profile.
Wed, Jul 12, 3:17 AM
mmodell committed rPHDEP8039e54eb72b: Merged upstream/stable in /phabricator (authored by mmodell).
Merged upstream/stable in /phabricator
Wed, Jul 12, 3:05 AM

Mon, Jul 10

mmodell awarded D708: I mean really, this is what happens 99% of the time a Love token.
Mon, Jul 10, 5:06 PM · Release-Engineering-Team
mmodell lowered the priority of T168692: Blocking an account on wikitech should disable LDAP logins from Normal to Low.
Mon, Jul 10, 4:58 PM · Release-Engineering-Team (Backlog), LDAP, wikitech.wikimedia.org, Wikimedia-Incident, Cloud-Services, Phabricator
mmodell moved T168699: Verify that the codfw lvs is configured correctly for Phabricator from In-progress to Blocked on the Release-Engineering-Team (Kanban) board.
Mon, Jul 10, 4:23 PM · Traffic, Release-Engineering-Team (Kanban), Phabricator, Operations
mmodell awarded T103886: Translation cache exhaustion caused by changes to PHP code in file scope a Orange Medal token.
Mon, Jul 10, 4:20 PM · Release-Engineering-Team (Watching / External), Patch-For-Review, Operations, Performance-Team, Deployment-Systems, HHVM

Thu, Jul 6

mmodell accepted D704: Configure arcanist, linters, and unit engine.
Thu, Jul 6, 7:23 PM · Release-Engineering-Team
mmodell closed T137354: Unable to browse refs/meta/config in Diffusion as Resolved.
Thu, Jul 6, 8:02 AM · Phabricator (2017-07-05), Release-Engineering-Team (Kanban), Regression, Diffusion
mmodell archived Phabricator (2017-07-05).
Thu, Jul 6, 8:02 AM
mmodell closed T168108: Unhandled Exception ("InvalidArgumentException") error when clicking on search link in "Activity Feed" panel as Resolved.
Thu, Jul 6, 12:20 AM · Phabricator (2017-07-05), Upstream, Release-Engineering-Team (Kanban)
mmodell added a comment to P5686 (An Untitled Masterwork).

I see:

phabricator    6236775b1ff6a3020397d4cf03407a9c27f081a5 (Wed, Jul 5) 
arcanist       1c0b8a10b8e27446cbab805b069a73233d97d1c5 (Tue, Jun 13) 
phutil         a5cb19867b08496fd76dc7bb0dba741278f30233 (Wed, Jul 5) 
sprint         7a7368cd216297a257a5d1409c751e70d19d230a (Feb 9 2017) 
security       25c3f70dca558922ac22bb1b664b88208ae2ec37 (Feb 17 2016) 
wmf-ext-misc   cea9396a8ba3ad02063074a1001a237a8e03f834 (Wed, Jul 5)
Thu, Jul 6, 12:12 AM
mmodell set the image for Phabricator (2017-07-05) to F8626885: profile.
Thu, Jul 6, 12:01 AM

Wed, Jul 5

mmodell edited projects for T168108: Unhandled Exception ("InvalidArgumentException") error when clicking on search link in "Activity Feed" panel, added: Phabricator (2017-07-05); removed Phabricator (Upstream).
Wed, Jul 5, 11:58 PM · Phabricator (2017-07-05), Upstream, Release-Engineering-Team (Kanban)
mmodell raised the priority of T114488: Automate the recurring management of wikitech:Deployments and phab:#train_deployments from Normal to High.
Wed, Jul 5, 11:52 PM · Release-Engineering-Team (Kanban), Deployment-Systems
mmodell added a comment to T169788: Improve Phabricator Print function so I can print a ticket out and read it.

I don't think Greasemonkey is an acceptable answer to usability problems :)

Wed, Jul 5, 11:51 PM · Release-Engineering-Team (Kanban), Upstream, Phabricator (Upstream)
mmodell reassigned T137354: Unable to browse refs/meta/config in Diffusion from mmodell to Paladox.

Awesome, thanks @Paladox

Wed, Jul 5, 11:49 PM · Phabricator (2017-07-05), Release-Engineering-Team (Kanban), Regression, Diffusion
mmodell closed T169673: Form 33 should be modified to have the archiving of the git repository in the checklist as Resolved.

Done

Wed, Jul 5, 11:47 PM · Release-Engineering-Team (Kanban), Phabricator
mmodell added a comment to T169788: Improve Phabricator Print function so I can print a ticket out and read it.

The workaround seems completely valid to me. If someone wants to write the css rules I'll be happy to accept it as a local patch to our fork of phabricator.

Wed, Jul 5, 11:38 PM · Release-Engineering-Team (Kanban), Upstream, Phabricator (Upstream)
mmodell committed rPHDEPcea9396a8ba3: Updated submodules for release/2017-07-05/1 (authored by mmodell).
Updated submodules for release/2017-07-05/1
Wed, Jul 5, 11:36 PM
mmodell created an object: Phabricator (2017-07-05).
Wed, Jul 5, 11:34 PM
mmodell committed rPHUTILa5cb19867b08: Merge 'upstream/stable' into wmf/stable (authored by mmodell).
Merge 'upstream/stable' into wmf/stable
Wed, Jul 5, 10:13 PM
mmodell placed T169788: Improve Phabricator Print function so I can print a ticket out and read it up for grabs.
Wed, Jul 5, 9:56 PM · Release-Engineering-Team (Kanban), Upstream, Phabricator (Upstream)