mmodell (Mukunda Modell)Administrator
Release Engineer, Phabricator Admin

User Details

User Since
Sep 19 2014, 9:45 AM (131 w, 4 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

Yesterday

mmodell committed rPHUTIL278d2b87ad39: Merge remote-tracking branch 'upstream/stable' into wmf/stable (authored by mmodell).
Merge remote-tracking branch 'upstream/stable' into wmf/stable
Tue, Mar 28, 11:31 PM
mmodell committed rPHDEP33b26060f9e9: Submodule updates pulled upstream 2016 week 38 (authored by mmodell).
Submodule updates pulled upstream 2016 week 38
Tue, Mar 28, 11:29 PM

Mon, Mar 27

mmodell closed T159169: Add Chelsy and Mikhail as admins to Discovery's Analysis repos on Diffusion as "Resolved".

I guess there isn't anything more to do here.

Mon, Mar 27, 10:45 AM · Project-Admins, Discovery-Analysis, Repository-Admins
mmodell added a comment to T125357: /maniphest/report/project/ : Maximum execution time of 10 seconds exceeded.

It's because the report is very computationally expensive to generate, so it can only complete within the time limit when all of the circumstances align perfectly such as the exceedingly rare event when the moon is full while simultaneously there is unusually little activity on all of phabricator's servers (web app + database).

Mon, Mar 27, 10:42 AM · Patch-For-Review, Performance, Phabricator
mmodell added a comment to T153742: "Security" field is invisible, but still exists, and the value can't be changed, causing Herald problems.

Since the Security tag doesn't have any real meaning to the software (and never has) we should probably just stop fooling ourselves and remove the herald rule.

Mon, Mar 27, 10:36 AM · Phabricator

Fri, Mar 24

mmodell added a comment to T106924: Consider using the Badges application for a few special roles to highlight those users' comments.

@MZMcBride: I suspect you're already aware of https://userstyles.org/

Fri, Mar 24, 10:01 PM · Phabricator

Thu, Mar 23

mmodell added a comment to T161208: Create a Phabricator badge for persons involved in CI.

Neat! How does one create a badge? Id like to do one for mobile wizards.

Thu, Mar 23, 9:10 PM · Phabricator
mmodell awarded Volunteer to recipient: EddieGP.
Thu, Mar 23, 8:54 PM
mmodell added a comment to T161208: Create a Phabricator badge for persons involved in CI.

Please remove the badge from me, thanks.

Thu, Mar 23, 5:39 PM · Phabricator
mmodell revoked Continuous Integrator from recipient: Legoktm.
Thu, Mar 23, 5:39 PM
mmodell awarded Repository Admin to recipient: Luke081515.
Thu, Mar 23, 5:37 PM
mmodell created Repository Admin.
Thu, Mar 23, 5:33 PM
mmodell added a comment to T106924: Consider using the Badges application for a few special roles to highlight those users' comments.

I like the idea of a possible mentors badge.

Thu, Mar 23, 5:24 PM · Phabricator
mmodell added a comment to T161208: Create a Phabricator badge for persons involved in CI.

I've added the names you called out, other than the release engineering team. It seems more appropriate to me that we give them out to people outside Release-Engineering-Team rather than awarding ourselves with more badges ;)

Thu, Mar 23, 5:20 PM · Phabricator
mmodell awarded Continuous Integrator to recipients: Addshore, awight, chasemp, Ejegg, JanZerebecki, Jdforrester-WMF, Jdlrobson, Krinkle, Legoktm, Paladox, Sniedzielski, Tobi_WMDE_SW, Umherirrender.
Thu, Mar 23, 5:20 PM
mmodell added a comment to T161208: Create a Phabricator badge for persons involved in CI.

https://phabricator.wikimedia.org/badges/view/8/

Thu, Mar 23, 5:17 PM · Phabricator
mmodell created Continuous Integrator.
Thu, Mar 23, 5:17 PM

Wed, Mar 22

mmodell committed rPHDEP74179efbff4c: Merge branch 'wmf/stable' into debian (authored by mmodell).
Merge branch 'wmf/stable' into debian
Wed, Mar 22, 10:59 AM
mmodell committed rPHDEPb3e1706914a7: Update submodules (authored by mmodell).
Update submodules
Wed, Mar 22, 10:54 AM
mmodell closed T158997: MW-1.29.0-wmf.16 deployment blockers as "Resolved".
Wed, Mar 22, 7:20 AM · Release, Release-Engineering-Team (Deployment-Blockers)

Tue, Mar 21

mmodell closed T87772: Resizing Title entry field in "Create New Task" form does not update position of "Required" hint next to field as "Invalid".

so I guess that this is no longer valid due to upstream changes.

Tue, Mar 21, 9:08 PM · Upstream, Phabricator (Upstream)

Mon, Mar 20

mmodell added a member for Security: Jrbranaa.
Mon, Mar 20, 4:10 PM
mmodell added a comment to D600: Create a wrapper around conftool for our pooling/depooling needs.

Build error:

Mon, Mar 20, 12:57 PM · Release-Engineering-Team
mmodell requested changes to D601: Create version subcommand, cleans up weird argparse voodoo.

We need to either fix tag_release or implement the version command in a separate module file.

Mon, Mar 20, 12:54 PM · Release-Engineering-Team
mmodell edited the description of T160763: Draft clarifying statement email re: Tooling Working Group.
Mon, Mar 20, 12:46 PM · Team-Practices (This-Week)
mmodell closed T160715: Single task appearing in two columns on a workboard as "Resolved".

Indeed, moving it on the workboard cleaned up the double entry. I'm marking this as resolved with the assumption that it's an isolated incident.

Mon, Mar 20, 12:44 PM · Phabricator
mmodell moved T140289: Investigate Improvements and Confidence Measures for TextCat Language Detection from Needs triage to Later on the Discovery-Search board.
Mon, Mar 20, 12:42 PM · Discovery-Search, Epic, CirrusSearch, Discovery
mmodell moved T140289: Investigate Improvements and Confidence Measures for TextCat Language Detection to Needs triage on the Discovery-Search board.
Mon, Mar 20, 12:41 PM · Discovery-Search, Epic, CirrusSearch, Discovery

Fri, Mar 17

mmodell added a member for acl*Project-Admins: Jrbranaa.
Fri, Mar 17, 1:13 AM

Thu, Mar 16

mmodell added a comment to T144111: Allow self-service creation of Maniphest projects for Tools.

IMO this is the perfect use case for sub-projects.

Thu, Mar 16, 9:28 PM · Project-Admins, Striker
mmodell added a comment to T144713: Add a web shell allowing people to perform actions as their tool from striker.

This is something I've spend a lot of time thinking about in the context of offering a web interface to scap.

Thu, Mar 16, 9:26 PM · Striker
mmodell awarded T144713: Add a web shell allowing people to perform actions as their tool from striker a Love token.
Thu, Mar 16, 9:24 PM · Striker
mmodell added a comment to T106924: Consider using the Badges application for a few special roles to highlight those users' comments.

@Peachey88: no, actually I enabled the application to try it out and see if the policy controls were indeed fixed. I don't think that the concern of dividing the community applies here because we don't have a "wmf staff" icon. I would envision having badges that acknowledge unique contributions or specific privileges / roles.

Thu, Mar 16, 8:30 PM · Phabricator
mmodell added a comment to T155180: codfw: mw2251-mw2260 rack/setup.

this host is logging a ton of noise in logstash... can we shut down hhvm until the ram problem is fixed?

Thu, Mar 16, 7:56 PM · Patch-For-Review, User-Elukey, Operations, ops-codfw
mmodell updated the title for P5063 make ci great again from Masterwork From Distant Lands to make ci great again.
Thu, Mar 16, 5:40 PM
mmodell edited P5063 make ci great again.
Thu, Mar 16, 5:39 PM
mmodell added a member for acl*Project-Admins: thcipriani.
Thu, Mar 16, 5:18 PM
mmodell committed rMSCAab7d4446c43e: Fix FancyProgressReporter (authored by mmodell).
Fix FancyProgressReporter
Thu, Mar 16, 3:31 PM
mmodell closed D589: Fix FancyProgressReporter by committing rMSCAab7d4446c43e: Fix FancyProgressReporter.
Thu, Mar 16, 3:31 PM · Release-Engineering-Team
mmodell closed T160044: Tasks show "This task has been locked." when not being logged in (should show option to log in instead?) as "Resolved".
Thu, Mar 16, 3:23 PM · Phabricator (Upstream), Upstream

Wed, Mar 15

mmodell added a project to T146055: Improve privilege separation for phabricator's config files and mysql credentials: DBA.

should this ticket have DBA tag?

Wed, Mar 15, 11:42 PM · DBA, Phabricator, Security
mmodell closed T160578: Fatal error: Couldn't find constant Elastica\Search::OPTION_SEARCH_TYPE_COUNT in ApiFeatureUsageQueryEngineElastica.php on line 88 as "Resolved".
Wed, Mar 15, 11:36 PM · MW-1.29-release (WMF-deploy-2017-03-14_(1.29.0-wmf.16)), Patch-For-Review, Discovery-Search, Discovery, CirrusSearch, Wikimedia-log-errors, ApiFeatureUsage, Elasticsearch
mmodell closed T160578: Fatal error: Couldn't find constant Elastica\Search::OPTION_SEARCH_TYPE_COUNT in ApiFeatureUsageQueryEngineElastica.php on line 88, a subtask of T158997: MW-1.29.0-wmf.16 deployment blockers, as "Resolved".
Wed, Mar 15, 11:36 PM · Release, Release-Engineering-Team (Deployment-Blockers)
mmodell added a comment to T160569: Variable wgCirrusSearchCompletionGeoContextSettings is not set in getConfiguration.php on line 105.

Thanks @EBernhardson!

Wed, Mar 15, 10:57 PM · Patch-For-Review, Discovery-Search, Discovery, MediaWiki-Configuration, CirrusSearch, Wikimedia-log-errors
mmodell added a comment to T160569: Variable wgCirrusSearchCompletionGeoContextSettings is not set in getConfiguration.php on line 105.

But now I'm getting this same error for wgCirrusSearchCompletionSuggesterGeoContext

Wed, Mar 15, 10:05 PM · Patch-For-Review, Discovery-Search, Discovery, MediaWiki-Configuration, CirrusSearch, Wikimedia-log-errors
mmodell added a comment to T160569: Variable wgCirrusSearchCompletionGeoContextSettings is not set in getConfiguration.php on line 105.

submitted separate task for T160578: Fatal error: Couldn't find constant Elastica\Search::OPTION_SEARCH_TYPE_COUNT in ApiFeatureUsageQueryEngineElastica.php on line 88

Wed, Mar 15, 10:04 PM · Patch-For-Review, Discovery-Search, Discovery, MediaWiki-Configuration, CirrusSearch, Wikimedia-log-errors
mmodell added a parent task for T160578: Fatal error: Couldn't find constant Elastica\Search::OPTION_SEARCH_TYPE_COUNT in ApiFeatureUsageQueryEngineElastica.php on line 88: T158997: MW-1.29.0-wmf.16 deployment blockers.
Wed, Mar 15, 9:59 PM · MW-1.29-release (WMF-deploy-2017-03-14_(1.29.0-wmf.16)), Patch-For-Review, Discovery-Search, Discovery, CirrusSearch, Wikimedia-log-errors, ApiFeatureUsage, Elasticsearch
mmodell added a subtask for T158997: MW-1.29.0-wmf.16 deployment blockers: T160578: Fatal error: Couldn't find constant Elastica\Search::OPTION_SEARCH_TYPE_COUNT in ApiFeatureUsageQueryEngineElastica.php on line 88.
Wed, Mar 15, 9:59 PM · Release, Release-Engineering-Team (Deployment-Blockers)
mmodell created T160578: Fatal error: Couldn't find constant Elastica\Search::OPTION_SEARCH_TYPE_COUNT in ApiFeatureUsageQueryEngineElastica.php on line 88.
Wed, Mar 15, 9:58 PM · MW-1.29-release (WMF-deploy-2017-03-14_(1.29.0-wmf.16)), Patch-For-Review, Discovery-Search, Discovery, CirrusSearch, Wikimedia-log-errors, ApiFeatureUsage, Elasticsearch
mmodell added a comment to T160569: Variable wgCirrusSearchCompletionGeoContextSettings is not set in getConfiguration.php on line 105.

not sure if this is related or separate issue:

Wed, Mar 15, 9:57 PM · Patch-For-Review, Discovery-Search, Discovery, MediaWiki-Configuration, CirrusSearch, Wikimedia-log-errors
mmodell changed the status of T146055: Improve privilege separation for phabricator's config files and mysql credentials from "Open" to "Stalled".

unassigning as there is nothing more I can do here.

Wed, Mar 15, 9:52 PM · DBA, Phabricator, Security
mmodell changed the status of T146055: Improve privilege separation for phabricator's config files and mysql credentials, a subtask of T143969: Unable to mirror repository from git.legoktm.com into diffusion, from "Open" to "Stalled".
Wed, Mar 15, 9:52 PM · Security-Reviews, Patch-For-Review, Striker, Phabricator
mmodell changed the status of T146055: Improve privilege separation for phabricator's config files and mysql credentials, a subtask of T143971: Allow easy replication of existing github/bitbucket repos, from "Open" to "Stalled".
Wed, Mar 15, 9:52 PM · Striker
mmodell added a comment to T160569: Variable wgCirrusSearchCompletionGeoContextSettings is not set in getConfiguration.php on line 105.

Now I'm getting the same error for $wgCirrusSearchCacheWarmers so I submitted https://gerrit.wikimedia.org/r/342914

Wed, Mar 15, 9:27 PM · Patch-For-Review, Discovery-Search, Discovery, MediaWiki-Configuration, CirrusSearch, Wikimedia-log-errors
mmodell added a comment to T160569: Variable wgCirrusSearchCompletionGeoContextSettings is not set in getConfiguration.php on line 105.

Ah so the problem is mismatched versions across wikis due to multiversion. Maybe just catch the exception?

Wed, Mar 15, 8:39 PM · Patch-For-Review, Discovery-Search, Discovery, MediaWiki-Configuration, CirrusSearch, Wikimedia-log-errors
mmodell edited the description of T160569: Variable wgCirrusSearchCompletionGeoContextSettings is not set in getConfiguration.php on line 105.
Wed, Mar 15, 8:36 PM · Patch-For-Review, Discovery-Search, Discovery, MediaWiki-Configuration, CirrusSearch, Wikimedia-log-errors
mmodell added a commit to T158997: MW-1.29.0-wmf.16 deployment blockers: rMW2c9cc18f30db: Revert "Improve sorting on SpecialWanted*-Pages".
Wed, Mar 15, 8:29 PM · Release, Release-Engineering-Team (Deployment-Blockers)
mmodell added a parent task for T160569: Variable wgCirrusSearchCompletionGeoContextSettings is not set in getConfiguration.php on line 105: T158997: MW-1.29.0-wmf.16 deployment blockers.
Wed, Mar 15, 8:28 PM · Patch-For-Review, Discovery-Search, Discovery, MediaWiki-Configuration, CirrusSearch, Wikimedia-log-errors
mmodell added a subtask for T158997: MW-1.29.0-wmf.16 deployment blockers: T160569: Variable wgCirrusSearchCompletionGeoContextSettings is not set in getConfiguration.php on line 105.
Wed, Mar 15, 8:28 PM · Release, Release-Engineering-Team (Deployment-Blockers)
mmodell created T160569: Variable wgCirrusSearchCompletionGeoContextSettings is not set in getConfiguration.php on line 105.
Wed, Mar 15, 8:27 PM · Patch-For-Review, Discovery-Search, Discovery, MediaWiki-Configuration, CirrusSearch, Wikimedia-log-errors
mmodell added a comment to T138429: Notice: Undefined index in /srv/mediawiki/php-1.28.0-wmf.7/extensions/SpamBlacklist/SpamBlacklist_body.php.

Seeing it again in wmf.16:

Wed, Mar 15, 7:22 PM · SpamBlacklist, Wikimedia-log-errors

Tue, Mar 14

mmodell added a reviewer for D589: Fix FancyProgressReporter: demon.
Tue, Mar 14, 11:37 PM · Release-Engineering-Team
mmodell accepted D597: Move utils.lock() and related code to its own module.
Tue, Mar 14, 11:36 PM · Release-Engineering-Team
mmodell added a comment to T158997: MW-1.29.0-wmf.16 deployment blockers.

wmf.16 uses elastic search 5 (hosted in codfw). Will be keeping an eye out for any search-related issues in this branch. See T157479: Put together a production migration plan for ES 2 -> ES 5

Tue, Mar 14, 7:22 PM · Release, Release-Engineering-Team (Deployment-Blockers)
mmodell added a watcher for Release-Engineering-Team (Deployment-Blockers): mmodell.
Tue, Mar 14, 4:37 PM
mmodell raised the priority of T158997: MW-1.29.0-wmf.16 deployment blockers from "Normal" to "High".
Tue, Mar 14, 4:36 PM · Release, Release-Engineering-Team (Deployment-Blockers)
mmodell edited Description on Wikimedia Phabricator Extensions.
Tue, Mar 14, 3:32 PM
mmodell created Wikimedia Phabricator Extensions.
Tue, Mar 14, 3:21 PM
mmodell added a comment to T160125: The star for the "favorites" menu is non-obvious.

@ksmith: resize your browser window to < 1280px (width) to see the mobile ui

Tue, Mar 14, 2:58 PM · Upstream, Phabricator (Upstream)
mmodell added a commit to T160044: Tasks show "This task has been locked." when not being logged in (should show option to log in instead?): rPHABd0c648dfa53b: Make "Can Interact" and logged-out users interact more gracefully.
Tue, Mar 14, 2:56 PM · Phabricator (Upstream), Upstream
mmodell changed the edit policy for Janitorial.
Tue, Mar 14, 2:42 PM
mmodell created Janitorial.
Tue, Mar 14, 2:34 PM
mmodell added a comment to T106924: Consider using the Badges application for a few special roles to highlight those users' comments.

It's a per-badge edit policy which controls who can award or remove each badge. Currently that should be set to phabricator admins for all of the active badges.

Tue, Mar 14, 2:24 PM · Phabricator
mmodell changed the badge icon for Volunteer from Average Person to Superstar.
Tue, Mar 14, 2:16 PM
mmodell disabled the badge Tea Lover.
Tue, Mar 14, 2:15 PM
mmodell added a comment to T106924: Consider using the Badges application for a few special roles to highlight those users' comments.

@MZMcBride: badges do not require acceptance by the recipient.

Tue, Mar 14, 2:13 PM · Phabricator
mmodell added a comment to T106924: Consider using the Badges application for a few special roles to highlight those users' comments.

The permissions checks are fixed and badges is (I believe) close to being 'un-prototyped' upstream (meaning: brought out of beta)

Tue, Mar 14, 2:11 PM · Phabricator
mmodell added a comment to T160408: Upgrading python-phabricator in trusty.

It might be comforting to know that the old version quite probably does not work with our modern version of phabricator. Authentication tokens changed quite a long time ago in a backwards incompatible way. I believe 0.6.x is actually required for a functioning conduit connection. If someone was using 0.4.x, I believe that their code has already broken and the updated package is more likely to fix something than it is to break anything.

Tue, Mar 14, 2:09 PM · Operations
mmodell closed T136015: thoroughly document the new branch cutting plan / strategy as "Declined".
Tue, Mar 14, 1:30 PM · Release-Engineering-Team (Long-Lived-Branches), Deployment-Systems
mmodell closed T136015: thoroughly document the new branch cutting plan / strategy, a subtask of T89945: Merge to deployed branches instead of cutting a new deployment branch every week., as "Declined".
Tue, Mar 14, 1:30 PM · Release-Engineering-Team (Long-Lived-Branches), releng-201617-q1, Deployment-Systems, Epic
mmodell added a comment to T160408: Upgrading python-phabricator in trusty.

Unless chase remembers something that I am forgetting, then I'd say go ahead with the new version. Anything that depends on the old version can be found and fixed if it breaks.

Tue, Mar 14, 1:26 PM · Operations
mmodell updated subscribers of T160408: Upgrading python-phabricator in trusty.

I don't know of anything on iridium that's actually using that. @chasemp might remember?

Tue, Mar 14, 1:25 PM · Operations

Sat, Mar 11

mmodell awarded Volunteer to recipient: Paladox.
Sat, Mar 11, 5:11 AM
mmodell changed the badge icon for Chaos Monkey from In Debt to Rebel.
Sat, Mar 11, 5:04 AM

Fri, Mar 10

mmodell added a comment to rMSCAa6b5f470270f: New scap sayings....

I keep having more ideas...

Fri, Mar 10, 10:33 PM
mmodell awarded Chaos Monkey to recipients: bd808, ori.
Fri, Mar 10, 10:15 PM
mmodell created Chaos Monkey.
Fri, Mar 10, 10:14 PM
mmodell updated Phabricator Admin flavor text from This person is a phabricator admin on Wikimedia's Phabricator to This person is an admin of Wikimedia's Phabricator.
Fri, Mar 10, 10:09 PM
mmodell awarded Phabricator Admin to recipients: tstarling, greg, Jalexander.
Fri, Mar 10, 10:08 PM
mmodell changed the edit policy for SWAT Deployer.
Fri, Mar 10, 10:06 PM
mmodell awarded SWAT Deployer to recipients: Addshore, hashar, thcipriani, aude, mmodell, Catrope, Dereckson, Anomie, MaxSem, demon.
Fri, Mar 10, 10:05 PM
mmodell created SWAT Deployer.
Fri, Mar 10, 10:02 PM
mmodell changed the badge icon for Phabricator Admin from Superstar to Serious Business.
Fri, Mar 10, 9:54 PM
mmodell updated Volunteer quality from Common to Epic.
Fri, Mar 10, 9:53 PM
mmodell committed rPHUTILae26170f5a12: Merge 'upstream/stable' into wmf/stable (authored by mmodell).
Merge 'upstream/stable' into wmf/stable
Fri, Mar 10, 9:07 PM
mmodell committed rMSCAa6b5f470270f: New scap sayings... (authored by mmodell).
New scap sayings...
Fri, Mar 10, 6:49 PM
mmodell closed D595: New scap sayings... by committing rMSCAa6b5f470270f: New scap sayings....
Fri, Mar 10, 6:49 PM · Release-Engineering-Team
mmodell updated the diff for D595: New scap sayings....
  • rebase
  • someone called about a party?
Fri, Mar 10, 6:30 PM · Release-Engineering-Team
mmodell created D595: New scap sayings....
Fri, Mar 10, 6:25 PM · Release-Engineering-Team
mmodell updated the diff for D589: Fix FancyProgressReporter.

imported but unused...

Fri, Mar 10, 5:22 PM · Release-Engineering-Team