Page MenuHomePhabricator

mmodell (Mukunda Modell)
Release Engineer, Phabricator AdminAdministrator

Projects (30)

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

Saturday

  • Clear sailing ahead.

User Details

User Since
Sep 19 2014, 9:45 AM (234 w, 5 d)
Roles
Administrator
Availability
Available
IRC Nick
twentyafterfour
LDAP User
20after4
MediaWiki User
MModell (WMF) [ Global Accounts ]

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


Current projects: Wikimedia-Phabricator-Extensions, Developer Productivity, Release Pipeline

Besides working on Deployments, 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 committed rPHUTIL0806e00b5e60: Merge remote-tracking branch 'upstream/stable' into wmf/stable (authored by mmodell).
Merge remote-tracking branch 'upstream/stable' into wmf/stable
Thu, Mar 21, 12:47 AM
mmodell committed rPHDEP221e08d75558: Updated submodules for release/2019-03-20/1` (authored by mmodell).
Updated submodules for release/2019-03-20/1`
Thu, Mar 21, 12:47 AM

Yesterday

mmodell edited Description on Phabricator (2019-03-20).
Wed, Mar 20, 11:59 PM
mmodell created Phabricator (2019-03-20).
Wed, Mar 20, 11:37 PM

Tue, Mar 19

mmodell added a project to T218683: extract throttling filter form wdqs so that it can be reused in other projects: Restricted Project.
Tue, Mar 19, 6:07 PM · Restricted Project, Wikidata, Wikidata-Query-Service
mmodell added a comment to T218683: extract throttling filter form wdqs so that it can be reused in other projects.

ref {T218636}

Tue, Mar 19, 6:03 PM · Restricted Project, Wikidata, Wikidata-Query-Service
mmodell added a comment to T218041: Hide archived projects on subproject page.

@Aklapper: I like the idea of simply sorting the active ones first. I'll see if that is difficult to implement.

Tue, Mar 19, 11:42 AM · Phabricator (Upstream), Upstream, WMSE-Bug-Reporting-and-Translation-2019
mmodell added a comment to T218597: Work out how we are going to cut deploy branches without gerrit HTTP Token.

Just to be clear, is the removal of the Gerrit HTTP token feature going to be a permanent change or just temporary? If this is a permanent thing I'll have to figure out how to modify libraryupgrader as well...

Tue, Mar 19, 11:40 AM · Release-Engineering-Team (Kanban), User-zeljkofilipin, Release, Train Deployments
mmodell removed projects from T168692: Blocking an account on wikitech should disable LDAP logins: Release-Engineering-Team (Backlog), Phabricator.
Tue, Mar 19, 9:48 AM · Patch-For-Review, wikitech.wikimedia.org, LDAP, Wikimedia-Incident
mmodell added a comment to T165795: Ldap auth extension vs. ldap vs. username Case.

It's worse than just ldap, gerrit also creates a ton of dupes with differing case.

Tue, Mar 19, 9:43 AM · Patch-For-Review, MediaWiki-Authentication-and-authorization, wikitech.wikimedia.org, MediaWiki-extensions-LdapAuthentication

Mon, Mar 18

mmodell changed the edit policy for Continuous-Integration-Config.
Mon, Mar 18, 10:40 PM
mmodell added a comment to T217901: Evaluate Phabricator Harbormaster.

@zeljkofilipin Harbormaster can do more than that, however, not without some customization of the code (which isn't possible on their free cloud instances)

Mon, Mar 18, 10:34 PM · User-zeljkofilipin, Release-Engineering-Team (Kanban)
mmodell awarded T217938: Beta Cluster does not have php7.0-redis available a Orange Medal token.
Mon, Mar 18, 2:12 PM · Patch-For-Review, PHP 7.0 support, Beta-Cluster-Infrastructure

Fri, Mar 15

mmodell committed rPHTR75c247234a8d: Added new translation strings for phabricator-ext (authored by mmodell).
Added new translation strings for phabricator-ext
Fri, Mar 15, 12:22 AM
mmodell committed rPHEX0ea04f9c4726: Make release details strings translatable (authored by mmodell).
Make release details strings translatable
Fri, Mar 15, 12:17 AM

Thu, Mar 14

mmodell added a project to T217938: Beta Cluster does not have php7.0-redis available: PHP 7.0 support.
Thu, Mar 14, 4:12 PM · Patch-For-Review, PHP 7.0 support, Beta-Cluster-Infrastructure
mmodell added a comment to T217938: Beta Cluster does not have php7.0-redis available.

on deployment-deploy01 in /usr/local/bin/sql we have php=php7.0 but apt doesn't have a php7.0-redis package

Thu, Mar 14, 4:12 PM · Patch-For-Review, PHP 7.0 support, Beta-Cluster-Infrastructure
mmodell added a comment to T217938: Beta Cluster does not have php7.0-redis available.

mariadb no longer puts its socket in /tmp/mysql.sock, it seems that the socket is in a 'systemd-private-xxx' directory within /tmp. See:

Thu, Mar 14, 7:58 AM · Patch-For-Review, PHP 7.0 support, Beta-Cluster-Infrastructure
mmodell reassigned T88005: Task created via Conduit is reported in mail as "Reopened" from mmodell to epriestley.
Thu, Mar 14, 7:48 AM · Upstream, Phabricator (Upstream)
mmodell closed T88005: Task created via Conduit is reported in mail as "Reopened" as Resolved.
Thu, Mar 14, 7:48 AM · Upstream, Phabricator (Upstream)

Tue, Mar 12

mmodell closed T217795: Remove mentioning rPHES in description of rPHAB as Resolved.

Updated the description.

Tue, Mar 12, 1:18 AM · Repository-Admins, Phabricator
mmodell closed T207525: Remove/Reduce large bug report header for "Edit" form as Resolved.
Tue, Mar 12, 1:03 AM · Phabricator

Fri, Mar 8

Krinkle awarded Blog Post: Projects, Forms and Subtypes oh my! a Orange Medal token.
Fri, Mar 8, 7:53 PM · Phabricator (2019-02-27)

Thu, Mar 7

mmodell added a comment to T216867: @Phabricator_maintenance is sending email notifications.

@greg: yes it should but the usefulness might be limited by T205258: Mass-edits via @Phabricator_maintenance account stop after 11 tasks (I still need to test that more thoroughly, that's tracked in T215079: test bulk editing in phabricator)

Thu, Mar 7, 3:09 AM · Phabricator
mmodell committed rPHDEP1a42721bf037: more release script fixes (authored by mmodell).
more release script fixes
Thu, Mar 7, 1:11 AM
mmodell archived Phabricator (2019-02-27).
Thu, Mar 7, 1:02 AM
mmodell accepted rPHESbbd8f8f2b8ef: Moved README content.
Thu, Mar 7, 12:59 AM
mmodell added 1 auditor(s) for rPHESbbd8f8f2b8ef: Moved README content: mmodell.
Thu, Mar 7, 12:59 AM
mmodell edited Description on Phabricator (2019-03-07).
Thu, Mar 7, 12:58 AM
mmodell committed rPHDEPd8e718b5dd6f: Updated submodules + release scripts (authored by mmodell).
Updated submodules + release scripts
Thu, Mar 7, 12:56 AM
mmodell edited Description on Phabricator (2019-03-07).
Thu, Mar 7, 12:55 AM
mmodell created Phabricator (2019-03-07).
Thu, Mar 7, 12:52 AM
mmodell committed rPHEXceaa62689cc0: Add user.transactions conduit method (authored by mmodell).
Add user.transactions conduit method
Thu, Mar 7, 12:47 AM

Wed, Mar 6

mmodell updated the task description for T171940: Create an easier way to manage SWAT deploys on (Deployments calendar).
Wed, Mar 6, 11:06 PM · wikitech.wikimedia.org, Deployments, Release-Engineering-Team (Backlog), User-MModell
mmodell updated the task description for T171940: Create an easier way to manage SWAT deploys on (Deployments calendar).
Wed, Mar 6, 11:06 PM · wikitech.wikimedia.org, Deployments, Release-Engineering-Team (Backlog), User-MModell
mmodell added a comment to T171940: Create an easier way to manage SWAT deploys on (Deployments calendar).

So while I was doing SWAT last night, I had a bit of inspiration:

Wed, Mar 6, 11:04 PM · wikitech.wikimedia.org, Deployments, Release-Engineering-Team (Backlog), User-MModell
mmodell updated the summary of D1144: Add user.transactions conduit method.
Wed, Mar 6, 8:09 PM
mmodell updated the test plan for D1144: Add user.transactions conduit method.
Wed, Mar 6, 8:06 PM
mmodell updated the diff for D1144: Add user.transactions conduit method.

Only allow admins to call the method

Wed, Mar 6, 8:02 PM
mmodell reclaimed D1144: Add user.transactions conduit method.

Turns out that the upstream method (transaction.search) doesn't work as expected. It only operates on a single object at a time (e.g. one task) so it doesn't give us a way to search for all transactions by a given user.

Wed, Mar 6, 7:46 PM

Tue, Mar 5

Dzahn awarded Blog Post: Phab Phebruary a Like token.
Tue, Mar 5, 2:31 PM · Phabricator (2019-02-20)

Mon, Mar 4

mmodell closed T51392: Make make-wmf-branch able to branch extensions with replaced substring of the version of mediawiki being branched, a subtask of T89945: Merge to deployed branches instead of cutting a new deployment branch every week., as Invalid.
Mon, Mar 4, 8:10 PM · RelEng-Archive-FY201718-Q2, Release-Engineering-Team (Long-Lived-Branches), releng-201617-q1, Deployments, Epic
mmodell closed T51392: Make make-wmf-branch able to branch extensions with replaced substring of the version of mediawiki being branched as Invalid.

This doesn't apply anymore since we have removed all support for special snowflakes.

Mon, Mar 4, 8:10 PM · MediaWiki-Release-Tools, Release-Engineering-Team (Backlog)
mmodell closed T118760: create a scap3 command to bootstrap a new deployment repo, a subtask of T151470: Define a stable API for scap plugins, as Declined.
Mon, Mar 4, 8:08 PM · Release-Engineering-Team (Backlog), Documentation, Scap
mmodell closed T118760: create a scap3 command to bootstrap a new deployment repo as Declined.

I don't really think this is useful after all.

Mon, Mar 4, 8:08 PM · Scap, WorkType-NewFunctionality
mmodell committed rMSCA2bcea908a42c: Targets: strip spaces from service_groups (authored by thcipriani).
Targets: strip spaces from service_groups
Mon, Mar 4, 8:07 PM
mmodell added a comment to T217587: Class 'Memcached' not found for php7 in beta.

I am not quite sure what is happening here. It comes from AbstractSync._check_fatals which unfortunately shallows the mwscript exception and return its own exception. Maybe giving more details as to why it fails would be informative.

I have no idea why the exit code would be None. I looked a bit at scap/sh.py but it is rather complicated :-\

Mon, Mar 4, 6:19 PM · Release-Engineering-Team (Kanban), Beta-Cluster-Infrastructure, Scap

Sat, Mar 2

mmodell closed T208976: Add user description for phabricator import bots as Resolved.
Sat, Mar 2, 9:07 AM · importbots, Phabricator
mmodell updated rtimport.
Sat, Mar 2, 9:05 AM
mmodell updated bzimport.
Sat, Mar 2, 9:03 AM
mmodell updated flimport.
Sat, Mar 2, 9:02 AM

Fri, Mar 1

mmodell renamed Phabricating Phabricator blog from Phabricator Phork to Phabricating Phabricator.
Fri, Mar 1, 7:42 AM · Phabricator
mmodell added a comment to Blog Post: Projects, Forms and Subtypes oh my!.

Is the fact that the “Task” type is now visible on all task also a part of this update?

Fri, Mar 1, 7:33 AM · Phabricator (2019-02-27)
mmodell updated the post content for Blog Post: Projects, Forms and Subtypes oh my!.
Fri, Mar 1, 7:30 AM · Phabricator (2019-02-27)

Thu, Feb 28

mmodell added a comment to T93499: Add support for task types.

@jmatazzoni: interesting, I hadn't thought about that. That does sound like a fairly compelling use-case. I'll add the types with default forms so that they behave otherwise the same as regular tasks.

Thu, Feb 28, 9:16 PM · User-MModell, Phabricator
mmodell added a comment to T93499: Add support for task types.

One more thing to consider: we now have project types as well. We already differentiated between project types by using different icon and color combinations. Now this is just a bit more formalized and I can create custom forms that correspond to e.g. team projects (Release-Engineering-Team), tags (Regression), sprint and personal work-boards (User-MModell)

Thu, Feb 28, 9:08 PM · User-MModell, Phabricator
mmodell added a comment to T93499: Add support for task types.

I'm already a bit unsure about "feature request" as I don't know if that type will really be useful.

Thu, Feb 28, 9:04 PM · User-MModell, Phabricator
mmodell added a comment to T93499: Add support for task types.

@jmatazzoni: To be clear, I don't want to discourage teams from having what is needed in order to have the most optimized workflow, I just want to be sure we think it through before I go and add more types. This is mainly due to two reasons:

Thu, Feb 28, 9:03 PM · User-MModell, Phabricator
mmodell added a comment to T93499: Add support for task types.

@Aklapper: do you think we should use types instead of tags? The advantage of tags is that a task can have more than one tag and they are light-weight / low maintenance.

Thu, Feb 28, 8:55 PM · User-MModell, Phabricator
mmodell added a comment to T93499: Add support for task types.

@jmatazzoni ok, what special features do these task types need? Are you just wanting them for categorization? In that case it might make more sense to use 'tag' type projects to group them. Task types enable custom forms but it takes some setup to configure them and I'd rather not create a bunch of types that are all identical.

Thu, Feb 28, 8:53 PM · User-MModell, Phabricator
MusikAnimal awarded Blog Post: Phab Phebruary a Orange Medal token.
Thu, Feb 28, 6:45 PM · Phabricator (2019-02-20)
MBinder_WMF awarded T217082: Unable to edit Herald rules:: "Argument 1 passed to HeraldTokenizerFieldValue::setValueMap() must be of the type array, object given" a Party Time token.
Thu, Feb 28, 4:40 PM · Phabricator (2019-02-27)
mmodell created Blog Post: Projects, Forms and Subtypes oh my!.
Thu, Feb 28, 2:24 AM · Phabricator (2019-02-27)
mmodell edited projects for T217082: Unable to edit Herald rules:: "Argument 1 passed to HeraldTokenizerFieldValue::setValueMap() must be of the type array, object given", added: Phabricator (2019-02-27); removed Release-Engineering-Team (Kanban), Phabricator.
Thu, Feb 28, 2:08 AM · Phabricator (2019-02-27)
mmodell closed T216849: Search broken in phabricator project tag/subscribed fields as Resolved.
Thu, Feb 28, 1:45 AM · Phabricator (2019-02-27), Upstream, Regression, Release-Engineering-Team (Kanban)
mmodell edited projects for T216849: Search broken in phabricator project tag/subscribed fields, added: Phabricator (2019-02-27); removed Phabricator (Upstream).
Thu, Feb 28, 1:07 AM · Phabricator (2019-02-27), Upstream, Regression, Release-Engineering-Team (Kanban)
mmodell renamed Phabricating Phabricator blog from The Wikimedia Phork to Phabricator Phork.
Thu, Feb 28, 1:02 AM · Phabricator
mmodell changed the profile image for blog Phabricating Phabricator.
Thu, Feb 28, 1:01 AM · Phabricator
mmodell archived Phabricator (2019-02-20).
Thu, Feb 28, 12:58 AM
mmodell committed rPHUTIL98182db0fe49: Merge remote-tracking branch 'upstream/stable' into wmf/stable (authored by mmodell).
Merge remote-tracking branch 'upstream/stable' into wmf/stable
Thu, Feb 28, 12:02 AM

Wed, Feb 27

mmodell committed rPHDEP11fccde1d213: Updated submodules (authored by mmodell).
Updated submodules
Wed, Feb 27, 11:41 PM
mmodell created Phabricator (2019-02-27).
Wed, Feb 27, 11:39 PM
Krinkle awarded Blog Post: Phab Phebruary a Orange Medal token.
Wed, Feb 27, 10:59 PM · Phabricator (2019-02-20)
mmodell closed T217082: Unable to edit Herald rules:: "Argument 1 passed to HeraldTokenizerFieldValue::setValueMap() must be of the type array, object given" as Resolved by committing rPHEX0399339684ef: Comply with upstream type hint by returning an array.
Wed, Feb 27, 9:21 PM · Phabricator (2019-02-27)
mmodell closed T217082: Unable to edit Herald rules:: "Argument 1 passed to HeraldTokenizerFieldValue::setValueMap() must be of the type array, object given", a subtask of T217078: Update Herald (H228) to include project 2960, as Resolved.
Wed, Feb 27, 9:21 PM · Proton, Phabricator, Reading-Infrastructure-Team-Backlog
mmodell committed rPHEX0399339684ef: Comply with upstream type hint by returning an array (authored by mmodell).
Comply with upstream type hint by returning an array
Wed, Feb 27, 9:21 PM
mmodell committed rPHTR4367830007e2: Added default translations for phabricator-extensions (rPHEX) (authored by mmodell).
Added default translations for phabricator-extensions (rPHEX)
Wed, Feb 27, 8:49 PM
mmodell committed rPHTR4517d0a6dfc9: updated phutil library cache and removed duplicate locale (authored by mmodell).
updated phutil library cache and removed duplicate locale
Wed, Feb 27, 8:49 PM
mmodell committed rPHTR83737054c6a9: Readme for the projects subdirectory (authored by mmodell).
Readme for the projects subdirectory
Wed, Feb 27, 8:49 PM
mmodell committed rPHTRf568063623c8: Exported strings from phabricator-extensions rPHEX (authored by mmodell).
Exported strings from phabricator-extensions rPHEX
Wed, Feb 27, 8:49 PM
mmodell committed rPHTR10a1cf53c0f8: Updated phabricator source/raw strings in projects/phabricator/* (authored by mmodell).
Updated phabricator source/raw strings in projects/phabricator/*
Wed, Feb 27, 8:49 PM
mmodell committed rPHTR60b9a2b9f784: Updated translations and added two new locales (ko and pt-br) (authored by mmodell).
Updated translations and added two new locales (ko and pt-br)
Wed, Feb 27, 8:49 PM
mmodell updated the task description for T196517: Automate weekly branch cut.
Wed, Feb 27, 6:37 PM · MediaWiki-Release-Tools, Release-Engineering-Team (Kanban)
mmodell edited projects for T196517: Automate weekly branch cut, added: MediaWiki-Release-Tools; removed Scap.
Wed, Feb 27, 6:35 PM · MediaWiki-Release-Tools, Release-Engineering-Team (Kanban)
mmodell added a comment to T196517: Automate weekly branch cut.

The rest api is PUT /projects/{project-name}/branches/{branch-id} ( api docs )

Wed, Feb 27, 6:33 PM · MediaWiki-Release-Tools, Release-Engineering-Team (Kanban)
mmodell edited P8140 config-new.json.
Wed, Feb 27, 6:23 PM
mmodell added a comment to T196517: Automate weekly branch cut.

itshappening

Wed, Feb 27, 6:22 PM · MediaWiki-Release-Tools, Release-Engineering-Team (Kanban)
mmodell added a comment to T196517: Automate weekly branch cut.

We can do almost the entire branch cut via API: either by calling https://gerrit-review.googlesource.com/Documentation/cmd-create-branch.html or the equivalent https rest api in gerrit.

Wed, Feb 27, 6:22 PM · MediaWiki-Release-Tools, Release-Engineering-Team (Kanban)
mmodell added a comment to T196517: Automate weekly branch cut.

@thcipriani and I went over all of the issues that were previously blockers to getting this done and we've decided that this is now officially unblocked. We should be able to do this in the near future.

Wed, Feb 27, 6:13 PM · MediaWiki-Release-Tools, Release-Engineering-Team (Kanban)
mmodell added a comment to T216067: Recover from corrupted beta MySQL slave (deployment-db04).

The page title looks wrong: 'Nove Resource:' instead of Nova Resource.

Wed, Feb 27, 12:01 AM · User-Ryasmeen, Patch-For-Review, Release-Engineering-Team (Kanban), Beta-Cluster-Infrastructure

Tue, Feb 26

mmodell added a comment to T216067: Recover from corrupted beta MySQL slave (deployment-db04).

@Krenair: does that look better now?

Tue, Feb 26, 10:24 PM · User-Ryasmeen, Patch-For-Review, Release-Engineering-Team (Kanban), Beta-Cluster-Infrastructure
mmodell added a comment to T216067: Recover from corrupted beta MySQL slave (deployment-db04).

I'm not sure, I got to this kinda late, I'm still tweaking the docs by re-reading the history on this task.

Tue, Feb 26, 10:18 PM · User-Ryasmeen, Patch-For-Review, Release-Engineering-Team (Kanban), Beta-Cluster-Infrastructure
mmodell added a comment to T216067: Recover from corrupted beta MySQL slave (deployment-db04).

https://wikitech.wikimedia.org/wiki/Nove_Resource:Deployment-prep/MariaDB_Slave_instance_setup

Tue, Feb 26, 10:16 PM · User-Ryasmeen, Patch-For-Review, Release-Engineering-Team (Kanban), Beta-Cluster-Infrastructure
mmodell placed T216067: Recover from corrupted beta MySQL slave (deployment-db04) up for grabs.

@ArielGlenn it's mostly covered on https://wikitech.wikimedia.org/wiki/Setting_up_a_MySQL_replica#Transferring_Data but not very clearly... I'll make an attempt to organize it better on a new wikitech page. It definitely needs to be easier to understand.

Tue, Feb 26, 9:31 PM · User-Ryasmeen, Patch-For-Review, Release-Engineering-Team (Kanban), Beta-Cluster-Infrastructure
Tgr awarded Blog Post: Phab Phebruary a Yellow Medal token.
Tue, Feb 26, 6:59 PM · Phabricator (2019-02-20)
mmodell added a comment to T177867: Pipeline image build cleanup.

maybe possibly helpful? docker tag tool: https://github.com/gofunky/tuplip

Tue, Feb 26, 5:34 PM · Patch-For-Review, Release-Engineering-Team (Kanban), Release Pipeline
kostajh awarded T216067: Recover from corrupted beta MySQL slave (deployment-db04) a Orange Medal token.
Tue, Feb 26, 5:04 PM · User-Ryasmeen, Patch-For-Review, Release-Engineering-Team (Kanban), Beta-Cluster-Infrastructure
mmodell added a comment to T213595: Release 1.32.1 as a maintenance release.

I still haven't announced this release. I've been overwhelmed with phabricator work and that doesn't seem to be easing up much this week.

Tue, Feb 26, 3:51 PM · Release-Engineering-Team (Kanban), MW-1.32-notes, MW-1.32-release
RhinosF1 awarded T216067: Recover from corrupted beta MySQL slave (deployment-db04) a Like token.
Tue, Feb 26, 3:11 PM · User-Ryasmeen, Patch-For-Review, Release-Engineering-Team (Kanban), Beta-Cluster-Infrastructure
debt awarded T216067: Recover from corrupted beta MySQL slave (deployment-db04) a Stroopwafel token.
Tue, Feb 26, 1:48 PM · User-Ryasmeen, Patch-For-Review, Release-Engineering-Team (Kanban), Beta-Cluster-Infrastructure