mmodell (Mukunda Modell)Administrator
Release Engineer, Phabricator Admin

Projects (30)

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

Wednesday

  • Clear sailing ahead.

User Details

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

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


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


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

Yesterday

greg awarded Blog Post: Phabricator Updates for February 2018 a Love token.
Sun, Feb 18, 10:26 PM · Release, Phabricator-Bot-Requests

Sat, Feb 17

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

@jcrespo: just let me know when you are available, I'll be glad to get to work with you on any phabricator tasks.

Sat, Feb 17, 6:51 PM · Release-Engineering-Team (Kanban), DBA, Phabricator, Security
mmodell claimed T146055: Improve privilege separation for phabricator's config files and mysql credentials.
Sat, Feb 17, 6:49 PM · Release-Engineering-Team (Kanban), DBA, Phabricator, Security
mmodell merged T186620: Phabricator down due to "Failed to `proc_open()`: proc_open() expects parameter 2 to be array" into T182832: Apache on phab1001 is gradually leaking worker processes which are stuck in "Gracefully finishing" state.
Sat, Feb 17, 5:37 PM · Patch-For-Review, User-Elukey, Release-Engineering-Team (Kanban), Operations, Phabricator
mmodell merged task T186620: Phabricator down due to "Failed to `proc_open()`: proc_open() expects parameter 2 to be array" into T182832: Apache on phab1001 is gradually leaking worker processes which are stuck in "Gracefully finishing" state.
Sat, Feb 17, 5:37 PM · User-Elukey, Release-Engineering-Team, Operations, Phabricator
mmodell added a comment to T182832: Apache on phab1001 is gradually leaking worker processes which are stuck in "Gracefully finishing" state.

@elukey, @Dzahn: do you think that setting MaxConnectionsPerChild would be worth a try? I somehow doubt it will help but it seems like it might improve the situation.

Sat, Feb 17, 5:37 PM · Patch-For-Review, User-Elukey, Release-Engineering-Team (Kanban), Operations, Phabricator
mmodell awarded T187378: BadMethodCallException from line 66 of extensions/TwoColConflict/includes/TwoColConflictHooks.php: Call to a member function getId() on a non-object (null) a Cookie token.
Sat, Feb 17, 7:55 AM · MW-1.31-release-notes (WMF-deploy-2018-02-13 (1.31.0-wmf.21)), Patch-For-Review, User-Addshore, WMDE-QWERTY-Sprint-2018-02-06, Two-Column-Edit-Conflict-Merge, Wikimedia-log-errors, TCB-Team
mmodell added a comment to D977: Less shit version of `git submodule foreach`.

Definitely needs testing, however, it's a good idea :)

Sat, Feb 17, 7:52 AM · Release-Engineering-Team
mmodell accepted D978: Remove python2 shebang stuff from tests.
Sat, Feb 17, 7:51 AM · Release-Engineering-Team

Fri, Feb 16

mmodell added a comment to T187587: Improve default Maniphest search settings.

The default query in maniphest doesn't use a search keyword so relevance doesn't come into play at all, that's why it's grouped and ordered.

Fri, Feb 16, 11:04 PM · Phabricator
mmodell added a comment to T187587: Improve default Maniphest search settings.

@Tgr: You can set a default search settings, it's a per-user preference. Documentation

Fri, Feb 16, 11:02 PM · Phabricator
mmodell added a parent task for T187153: BadMethodCallException (mRecord->getContent() when mRecord is null) when viewing details or examine of Abuselog of Abusefilter 131 on zh.wikipedia: T183961: 1.31.0-wmf.22 deployment blockers.
Fri, Feb 16, 9:19 PM · Anti-Harassment, Regression, Multi-Content-Revisions, User-Addshore, Wikimedia-log-errors, Chinese-Sites, AbuseFilter
mmodell added a subtask for T183961: 1.31.0-wmf.22 deployment blockers: T187153: BadMethodCallException (mRecord->getContent() when mRecord is null) when viewing details or examine of Abuselog of Abusefilter 131 on zh.wikipedia.
Fri, Feb 16, 9:19 PM · Release-Engineering-Team (Kanban), Release, Train Deployments
mmodell added a comment to T187153: BadMethodCallException (mRecord->getContent() when mRecord is null) when viewing details or examine of Abuselog of Abusefilter 131 on zh.wikipedia.

This is now the most frequent error in fatal monitor by a full order of magnitude. It probably should have blocked the train, however, it slipped through the cracks yesterday during the train rollout. I'm going to make this block wmf.21 for now, for the sake of visibility at least.

Fri, Feb 16, 9:18 PM · Anti-Harassment, Regression, Multi-Content-Revisions, User-Addshore, Wikimedia-log-errors, Chinese-Sites, AbuseFilter
mmodell added a comment to T182832: Apache on phab1001 is gradually leaking worker processes which are stuck in "Gracefully finishing" state.

If it's really as simple as importing the packages from buster and deploying them, then that seems like a fairly ideal solution to me. We would need to do some testing beyond simply installing them on a vps instance and declaring it works!!, however, I don't think there will be any huge issues with compatibility, phabricator has supported 7.1 for a full year now.

Fri, Feb 16, 9:12 PM · Patch-For-Review, User-Elukey, Release-Engineering-Team (Kanban), Operations, Phabricator

Thu, Feb 15

mmodell removed a parent task for T187378: BadMethodCallException from line 66 of extensions/TwoColConflict/includes/TwoColConflictHooks.php: Call to a member function getId() on a non-object (null): T183960: 1.31.0-wmf.21 deployment blockers.
Thu, Feb 15, 8:24 PM · MW-1.31-release-notes (WMF-deploy-2018-02-13 (1.31.0-wmf.21)), Patch-For-Review, User-Addshore, WMDE-QWERTY-Sprint-2018-02-06, Two-Column-Edit-Conflict-Merge, Wikimedia-log-errors, TCB-Team
mmodell removed a subtask for T183960: 1.31.0-wmf.21 deployment blockers: T187378: BadMethodCallException from line 66 of extensions/TwoColConflict/includes/TwoColConflictHooks.php: Call to a member function getId() on a non-object (null).
Thu, Feb 15, 8:24 PM · Release-Engineering-Team (Kanban), Release, Train Deployments
mmodell added a commit to T183960: 1.31.0-wmf.21 deployment blockers: rOMWC09ae320cdbbd: all wikis to 1.31.0-wmf.21.
Thu, Feb 15, 8:14 PM · Release-Engineering-Team (Kanban), Release, Train Deployments
mmodell edited parent tasks for T187492: RemexHtml\Tokenizer\Tokenizer: pcre.backtrack_limit exhausted, added: T183961: 1.31.0-wmf.22 deployment blockers; removed: T183960: 1.31.0-wmf.21 deployment blockers.
Thu, Feb 15, 8:05 PM · RemexHtml, Wikimedia-log-errors
mmodell removed a subtask for T183960: 1.31.0-wmf.21 deployment blockers: T187492: RemexHtml\Tokenizer\Tokenizer: pcre.backtrack_limit exhausted.
Thu, Feb 15, 8:05 PM · Release-Engineering-Team (Kanban), Release, Train Deployments
mmodell added a subtask for T183961: 1.31.0-wmf.22 deployment blockers: T187492: RemexHtml\Tokenizer\Tokenizer: pcre.backtrack_limit exhausted.
Thu, Feb 15, 8:05 PM · Release-Engineering-Team (Kanban), Release, Train Deployments
mmodell added a parent task for T187492: RemexHtml\Tokenizer\Tokenizer: pcre.backtrack_limit exhausted: T183960: 1.31.0-wmf.21 deployment blockers.
Thu, Feb 15, 8:00 PM · RemexHtml, Wikimedia-log-errors
mmodell added a subtask for T183960: 1.31.0-wmf.21 deployment blockers: T187492: RemexHtml\Tokenizer\Tokenizer: pcre.backtrack_limit exhausted.
Thu, Feb 15, 8:00 PM · Release-Engineering-Team (Kanban), Release, Train Deployments
mmodell created T187492: RemexHtml\Tokenizer\Tokenizer: pcre.backtrack_limit exhausted.
Thu, Feb 15, 8:00 PM · RemexHtml, Wikimedia-log-errors
mmodell created T187491: No specifications provided to ArchivedFile constructor.
Thu, Feb 15, 7:41 PM · Multimedia, MediaWiki-Uploading, Wikimedia-log-errors
mmodell updated subscribers of T182832: Apache on phab1001 is gradually leaking worker processes which are stuck in "Gracefully finishing" state.

Interesting: I think I may have found a relevant (and long standing) bug report for the deadlock: https://bugs.php.net/bug.php?id=31749, it's even got comments from our own @tstarling who points to https://www.mediawiki.org/wiki/Special:Code/MediaWiki/103433#140

Thu, Feb 15, 7:23 PM · Patch-For-Review, User-Elukey, Release-Engineering-Team (Kanban), Operations, Phabricator
mmodell added a comment to T182832: Apache on phab1001 is gradually leaking worker processes which are stuck in "Gracefully finishing" state.

Yeah it's especially odd that it's happening on the exact same json string every time. I'll see if upstream has heard of anything like it

Thu, Feb 15, 6:49 PM · Patch-For-Review, User-Elukey, Release-Engineering-Team (Kanban), Operations, Phabricator
mmodell added a comment to T182832: Apache on phab1001 is gradually leaking worker processes which are stuck in "Gracefully finishing" state.

Looks like we still have workers in the "G" state:

Thu, Feb 15, 5:22 PM · Patch-For-Review, User-Elukey, Release-Engineering-Team (Kanban), Operations, Phabricator
mmodell added a comment to T182832: Apache on phab1001 is gradually leaking worker processes which are stuck in "Gracefully finishing" state.

Paladin: how many packages from buster were required to get it working?

Thu, Feb 15, 4:02 PM · Patch-For-Review, User-Elukey, Release-Engineering-Team (Kanban), Operations, Phabricator
Elitre awarded T150766: Distinguish "mentions" from "subscribers" a Party Time token.
Thu, Feb 15, 2:34 PM · Phabricator (2018-02-15), Upstream
Dzahn awarded Blog Post: Phabricator Updates for February 2018 a Party Time token.
Thu, Feb 15, 1:36 PM · Release, Phabricator-Bot-Requests
mmodell closed T150766: Distinguish "mentions" from "subscribers" as Resolved.

Mail stamps are available now.

Thu, Feb 15, 11:19 AM · Phabricator (2018-02-15), Upstream
mmodell added a comment to T182832: Apache on phab1001 is gradually leaking worker processes which are stuck in "Gracefully finishing" state.

I'm now convinced that the problem is a bug in php internals and our best bet is probably to switch to just about any newer version of php. That said, it is probably triggered by something sortof unusual that phabricator is doing in recent versions. Fixing it from the Phabricator side would really just be working around a php bug, however, I really don't know where to look for the trigger.

Thu, Feb 15, 10:03 AM · Patch-For-Review, User-Elukey, Release-Engineering-Team (Kanban), Operations, Phabricator
mmodell added a comment to T182832: Apache on phab1001 is gradually leaking worker processes which are stuck in "Gracefully finishing" state.

@elukey: last night, before the upgrade, I had @Dzahn merge a few opcache related config changes: rOPUP9ddf42a8267a: Phabricator: Increase zend opcache limits in php.ini

Thu, Feb 15, 9:40 AM · Patch-For-Review, User-Elukey, Release-Engineering-Team (Kanban), Operations, Phabricator
hashar awarded Blog Post: Phabricator Updates for February 2018 a Like token.
Thu, Feb 15, 9:02 AM · Release, Phabricator-Bot-Requests
mmodell published Blog Post: Phabricator Updates for February 2018.
Thu, Feb 15, 7:55 AM · Release, Phabricator-Bot-Requests
mmodell awarded T76942: Allow to search tasks about MediaWiki core and core only (create MediaWiki umbrella project?) a Like token.
Thu, Feb 15, 7:53 AM · Phabricator (Search), Developer-Wishlist (2017), MediaWiki-General-or-Unknown, Project-Admins
mmodell abandoned D251: Add ToxTestEngine for arcanist.
Thu, Feb 15, 7:52 AM · Release-Engineering-Team
mmodell abandoned D240: Support github style code block language specification.
Thu, Feb 15, 7:52 AM · Phabricator
mmodell added a comment to T184882: Scap: TypeError: error: (not all arguments converted during string formatting); format string: (Passed unrecognized git_binary_manager {}); arguments: ((u'fat',)).

can this be closed, then?

Thu, Feb 15, 7:49 AM · Patch-For-Review, Release-Engineering-Team (Kanban), Scap
mmodell added a comment to T183792: Commits merged in Gerrit should appear near-instantly in Phabricator.

@Legoktm: So although I think we should still do some things to make phabricator update faster, given that the original need is now met, should we close this?

Thu, Feb 15, 7:47 AM · Patch-For-Review, Gerrit, Diffusion
mmodell added a comment to T182832: Apache on phab1001 is gradually leaking worker processes which are stuck in "Gracefully finishing" state.

Ok so the upstream code is deployed, now we just wait and see if the situation improves.

Thu, Feb 15, 7:34 AM · Patch-For-Review, User-Elukey, Release-Engineering-Team (Kanban), Operations, Phabricator
mmodell closed T187143: Upcoming phabricator upgrade requires unusually long database migrations, a subtask of T182832: Apache on phab1001 is gradually leaking worker processes which are stuck in "Gracefully finishing" state, as Resolved.
Thu, Feb 15, 7:30 AM · Patch-For-Review, User-Elukey, Release-Engineering-Team (Kanban), Operations, Phabricator
mmodell closed T187143: Upcoming phabricator upgrade requires unusually long database migrations as Resolved.

Everything looks good, thanks @jcrespo and @Marostegui

Thu, Feb 15, 7:30 AM · Patch-For-Review, Phabricator (2018-02-15), Release, DBA
mmodell added a comment to T187425: test.

upgrade: success!

Thu, Feb 15, 7:30 AM · DBA
mmodell awarded T187425: test a Barnstar token.
Thu, Feb 15, 6:47 AM · DBA
mmodell committed rPHDEP63525601b5bc: submodule update (authored by mmodell).
submodule update
Thu, Feb 15, 6:28 AM
mmodell added a commit to T187143: Upcoming phabricator upgrade requires unusually long database migrations: rPHABf177f92217c2: Remove 20180208.maniphest.01.close.sql, schema change already applied.
Thu, Feb 15, 6:28 AM · Patch-For-Review, Phabricator (2018-02-15), Release, DBA
mmodell edited Description on Phabricator (2018-02-15).
Thu, Feb 15, 3:08 AM
mmodell edited Description on Phabricator (2018-02-15).
Thu, Feb 15, 3:07 AM
mmodell edited Description on Phabricator (2018-02-15).
Thu, Feb 15, 3:06 AM
mmodell edited Description on Phabricator (2018-02-15).
Thu, Feb 15, 3:06 AM
mmodell added a commit to T187143: Upcoming phabricator upgrade requires unusually long database migrations: rPHAB4ca0e44bf025: disable the 20180208.maniphest.02.populate.php migration.
Thu, Feb 15, 3:01 AM · Patch-For-Review, Phabricator (2018-02-15), Release, DBA
mmodell committed rPHDEP6a0d37beac08: updated tag-release script and disabled a migration in phabricator/ (authored by mmodell).
updated tag-release script and disabled a migration in phabricator/
Thu, Feb 15, 3:00 AM
mmodell added a comment to T182832: Apache on phab1001 is gradually leaking worker processes which are stuck in "Gracefully finishing" state.

This looks much better now:

Thu, Feb 15, 2:45 AM · Patch-For-Review, User-Elukey, Release-Engineering-Team (Kanban), Operations, Phabricator
mmodell added a comment to T187149: Delete all Phabricator git repos that haven't been referenced / aren't used..

Now... I'd be fine with nuking them and recreating them in a sensible manner, dropping the non-branch/tag refs and having a bridge to handle creation/deletion automatically.

Thu, Feb 15, 2:41 AM · Diffusion, Gerrit, Phabricator
mmodell edited projects for T150766: Distinguish "mentions" from "subscribers", added: Phabricator (2018-02-15); removed Phabricator (Upstream).
Thu, Feb 15, 2:28 AM · Phabricator (2018-02-15), Upstream
mmodell edited Description on Phabricator (2018-02-15).
Thu, Feb 15, 1:57 AM
mmodell archived Phabricator (2018-02-xx).
Thu, Feb 15, 1:56 AM
mmodell edited projects for T126711: When another user removes you as a subscriber from a task, you don't receive an email notification, added: Phabricator (2018-02-15); removed Phabricator (2018-02-xx).
Thu, Feb 15, 1:56 AM · Phabricator (2018-02-15), Upstream
mmodell edited projects for T103009: allow users to export maniphest advanced search to csv, added: Phabricator (2018-02-15); removed Phabricator (2018-02-xx).
Thu, Feb 15, 1:56 AM · Phabricator (2018-02-15), Upstream
mmodell edited projects for T187143: Upcoming phabricator upgrade requires unusually long database migrations, added: Phabricator (2018-02-15); removed Phabricator (2018-02-xx).
Thu, Feb 15, 1:56 AM · Patch-For-Review, Phabricator (2018-02-15), Release, DBA
mmodell added a subtask for T182832: Apache on phab1001 is gradually leaking worker processes which are stuck in "Gracefully finishing" state: T187143: Upcoming phabricator upgrade requires unusually long database migrations.
Thu, Feb 15, 1:54 AM · Patch-For-Review, User-Elukey, Release-Engineering-Team (Kanban), Operations, Phabricator
mmodell added a parent task for T187143: Upcoming phabricator upgrade requires unusually long database migrations: T182832: Apache on phab1001 is gradually leaking worker processes which are stuck in "Gracefully finishing" state.
Thu, Feb 15, 1:54 AM · Patch-For-Review, Phabricator (2018-02-15), Release, DBA
mmodell committed rPHDEPccf2748f66cc: Merged upstream changes into arcanist, libphutil and phabricator (authored by mmodell).
Merged upstream changes into arcanist, libphutil and phabricator
Thu, Feb 15, 1:45 AM
mmodell created Phabricator (2018-02-15).
Thu, Feb 15, 1:42 AM

Wed, Feb 14

mmodell committed rPHUTILa43053f52687: Merge remote-tracking branch 'upstream/stable' into wmf/stable (authored by mmodell).
Merge remote-tracking branch 'upstream/stable' into wmf/stable
Wed, Feb 14, 11:20 PM
mmodell added a comment to T187149: Delete all Phabricator git repos that haven't been referenced / aren't used..

Phabricator doesn't only exist as an accessory for gerrit. Having repositories in Phabricator, for one thing, provides a mirror we control. Phabricator currently provides us with a multi-datacenter redundant repository hosting infrastructure. Soon we will be able to have deployments fetch from a git repo hosted in the same datacenter as the target server. If phabricator is a sunk cost then we can close a lot of open tasks and I might as well start looking for a new job.

Wed, Feb 14, 9:12 PM · Diffusion, Gerrit, Phabricator
mmodell updated the task description for T187378: BadMethodCallException from line 66 of extensions/TwoColConflict/includes/TwoColConflictHooks.php: Call to a member function getId() on a non-object (null).
Wed, Feb 14, 9:07 PM · MW-1.31-release-notes (WMF-deploy-2018-02-13 (1.31.0-wmf.21)), Patch-For-Review, User-Addshore, WMDE-QWERTY-Sprint-2018-02-06, Two-Column-Edit-Conflict-Merge, Wikimedia-log-errors, TCB-Team
mmodell added a comment to T187149: Delete all Phabricator git repos that haven't been referenced / aren't used..

I disagree. I still think all repos should be sync'd to phabricator.

Wed, Feb 14, 9:01 PM · Diffusion, Gerrit, Phabricator
mmodell triaged T187378: BadMethodCallException from line 66 of extensions/TwoColConflict/includes/TwoColConflictHooks.php: Call to a member function getId() on a non-object (null) as High priority.
Wed, Feb 14, 8:54 PM · MW-1.31-release-notes (WMF-deploy-2018-02-13 (1.31.0-wmf.21)), Patch-For-Review, User-Addshore, WMDE-QWERTY-Sprint-2018-02-06, Two-Column-Edit-Conflict-Merge, Wikimedia-log-errors, TCB-Team
mmodell added a parent task for T187378: BadMethodCallException from line 66 of extensions/TwoColConflict/includes/TwoColConflictHooks.php: Call to a member function getId() on a non-object (null): T183960: 1.31.0-wmf.21 deployment blockers.
Wed, Feb 14, 8:54 PM · MW-1.31-release-notes (WMF-deploy-2018-02-13 (1.31.0-wmf.21)), Patch-For-Review, User-Addshore, WMDE-QWERTY-Sprint-2018-02-06, Two-Column-Edit-Conflict-Merge, Wikimedia-log-errors, TCB-Team
mmodell added a subtask for T183960: 1.31.0-wmf.21 deployment blockers: T187378: BadMethodCallException from line 66 of extensions/TwoColConflict/includes/TwoColConflictHooks.php: Call to a member function getId() on a non-object (null).
Wed, Feb 14, 8:54 PM · Release-Engineering-Team (Kanban), Release, Train Deployments
mmodell created T187378: BadMethodCallException from line 66 of extensions/TwoColConflict/includes/TwoColConflictHooks.php: Call to a member function getId() on a non-object (null).
Wed, Feb 14, 8:52 PM · MW-1.31-release-notes (WMF-deploy-2018-02-13 (1.31.0-wmf.21)), Patch-For-Review, User-Addshore, WMDE-QWERTY-Sprint-2018-02-06, Two-Column-Edit-Conflict-Merge, Wikimedia-log-errors, TCB-Team
mmodell awarded D976: Support Python projects a Cookie token.
Wed, Feb 14, 12:54 PM · Release-Engineering-Team
mmodell added a comment to T187143: Upcoming phabricator upgrade requires unusually long database migrations.

@Marostegui: that's one of the largest maniphest_task tables in teh world, despite it's size in raw bytes ;)

Wed, Feb 14, 12:53 PM · Patch-For-Review, Phabricator (2018-02-15), Release, DBA
mmodell added a comment to T187143: Upcoming phabricator upgrade requires unusually long database migrations.

@jcrespo: it's possible but I'd have to hack around some detection scheme, phabricator normally refuses to even serve pages when the migrations are out of date. I could probably just modify the migration script to make it a noop, that way it'll get recorded as if it was applied. Then it's simple enough to run the exact same code manually after the upgrade.

Wed, Feb 14, 12:51 PM · Patch-For-Review, Phabricator (2018-02-15), Release, DBA
mmodell added a comment to T187143: Upcoming phabricator upgrade requires unusually long database migrations.

The slow one is backfilling the data, the alter table is in a separate migration which should be much quicker than the population script. There is a similar pair of migrations in there for some differential tables, however, we don't have as much data in differential so I don't expect that one to be slow.

Wed, Feb 14, 12:48 PM · Patch-For-Review, Phabricator (2018-02-15), Release, DBA
mmodell added a comment to T187143: Upcoming phabricator upgrade requires unusually long database migrations.

https://secure.phabricator.com/source/phabricator/browse/master/resources/sql/autopatches/20180208.maniphest.01.close.sql

Wed, Feb 14, 12:47 PM · Patch-For-Review, Phabricator (2018-02-15), Release, DBA
mmodell added a comment to T187143: Upcoming phabricator upgrade requires unusually long database migrations.

Here is the one that I expect to be slow: https://secure.phabricator.com/source/phabricator/browse/master/resources/sql/autopatches/20180208.maniphest.02.populate.php;215b8b4727aa96dc5cbd2a53120f97d5f4d4ce3b

Wed, Feb 14, 12:39 PM · Patch-For-Review, Phabricator (2018-02-15), Release, DBA
mmodell added a comment to T150766: Distinguish "mentions" from "subscribers".

Mail Stamps will soon be exposed in the body of messages, rather than only in the X-Phabricator-Mail-Stamps header. This should make it possible for gmail users to create a filter which matches any notification which @mentions their username. See https://secure.phabricator.com/w/changelog/2018.06/

Wed, Feb 14, 12:31 PM · Phabricator (2018-02-15), Upstream
mmodell awarded T183116: Find best way to allow CoC committee to temporarily disable Phab user accounts a Cookie token.
Wed, Feb 14, 11:50 AM · Developer-Relations (Jan-Mar-2018), Phabricator

Tue, Feb 13

mmodell accepted D965: Add quick environment variable to disable scap plugins.

goat-for-it

Tue, Feb 13, 9:29 PM · Release-Engineering-Team
mmodell updated the task description for T187253: Reset Niharika's 2factor auth.
Tue, Feb 13, 9:27 PM · Security, Phabricator
mmodell closed T187253: Reset Niharika's 2factor auth as Resolved.
Tue, Feb 13, 9:26 PM · Security, Phabricator
mmodell created T187253: Reset Niharika's 2factor auth.
Tue, Feb 13, 9:26 PM · Security, Phabricator
mmodell added a comment to T182832: Apache on phab1001 is gradually leaking worker processes which are stuck in "Gracefully finishing" state.

Also, remember we need to coordinate timing with the DBAs. :)

Tue, Feb 13, 7:56 PM · Patch-For-Review, User-Elukey, Release-Engineering-Team (Kanban), Operations, Phabricator
mmodell added a comment to T187143: Upcoming phabricator upgrade requires unusually long database migrations.

Midnight UTC would be impossible for me
Thursday UTC morning I could do it (I could be around 6AM UTC or so). Would that work?

Tue, Feb 13, 7:55 PM · Patch-For-Review, Phabricator (2018-02-15), Release, DBA
mmodell added a comment to T182832: Apache on phab1001 is gradually leaking worker processes which are stuck in "Gracefully finishing" state.

Yeah, I could try to rush the update, there isn't necessarily a reason to wait until Wednesday, other than that's the scheduled deployment window

Tue, Feb 13, 6:41 PM · Patch-For-Review, User-Elukey, Release-Engineering-Team (Kanban), Operations, Phabricator
mmodell added a comment to T182832: Apache on phab1001 is gradually leaking worker processes which are stuck in "Gracefully finishing" state.

Still, PHP 7.1 should be considered independently, has this specific bug been reported upstream (or is that blocked by Wednesday's upgrade as upstream doesn't accepted bug report for outdated versions?)

Tue, Feb 13, 3:37 PM · Patch-For-Review, User-Elukey, Release-Engineering-Team (Kanban), Operations, Phabricator
mmodell accepted D970: Add `script` check type and environment variables.

Looks good. Tests pass. Good work @thcipriani!

Tue, Feb 13, 3:31 PM · Release-Engineering-Team
mmodell added a comment to T182832: Apache on phab1001 is gradually leaking worker processes which are stuck in "Gracefully finishing" state.

Yes definitely something similar, from the stacktraces I can always see len=8739) at json_tokener.c:434, that is definitely a big buffered string. Do we have all commit/fixes in our Phabricator version? This is why I suggested to follow up with upstream :)

Tue, Feb 13, 10:03 AM · Patch-For-Review, User-Elukey, Release-Engineering-Team (Kanban), Operations, Phabricator
mmodell moved T187143: Upcoming phabricator upgrade requires unusually long database migrations from To Triage to Backlog on the Phabricator board.
Tue, Feb 13, 12:37 AM · Patch-For-Review, Phabricator (2018-02-15), Release, DBA
mmodell triaged T187143: Upcoming phabricator upgrade requires unusually long database migrations as Normal priority.
Tue, Feb 13, 12:36 AM · Patch-For-Review, Phabricator (2018-02-15), Release, DBA
mmodell updated subscribers of T187143: Upcoming phabricator upgrade requires unusually long database migrations.
Tue, Feb 13, 12:36 AM · Patch-For-Review, Phabricator (2018-02-15), Release, DBA
mmodell created T187143: Upcoming phabricator upgrade requires unusually long database migrations.
Tue, Feb 13, 12:35 AM · Patch-For-Review, Phabricator (2018-02-15), Release, DBA
mmodell added a comment to T182832: Apache on phab1001 is gradually leaking worker processes which are stuck in "Gracefully finishing" state.

Although PHP 7.1 was declined in T160714, I believe we should reconsider. There is a backport of PHP 7.1 to jessie and stretch, maintained by the same package maintainer as official debian packaging for PHP. It should be fairly trivial to upload that package to our apt repo and keep it in sync from upstream, or alternatively, to run the build ourself using the same source package. If there is any budget for it, we could even consider tipping the maintainer to help encourage the continued support for this package.

Tue, Feb 13, 12:17 AM · Patch-For-Review, User-Elukey, Release-Engineering-Team (Kanban), Operations, Phabricator
mmodell added a comment to T160714: Upload php7.1 to apt.wm.org.

Confirmed: all of the news entries on https://tracker.debian.org/pkg/php7.1 are by Ondřej Surý.

Tue, Feb 13, 12:09 AM · Phabricator, Operations
mmodell added a comment to T160714: Upload php7.1 to apt.wm.org.

I think those packages are the same - sury.org is owned by the debian maintainer for php - so those are semi-official backports, as far as I can tell.

Tue, Feb 13, 12:03 AM · Phabricator, Operations

Mon, Feb 12

mmodell added a comment to T160714: Upload php7.1 to apt.wm.org.

@MoritzMuehlenhoff: There is at least one 3rd party PHP 7.1 package available [[[ https://packages.sury.org/php/ | 1 ]]]. Could we not use their source packages to build our own binaries? I realize it is indeed a large amount of maintenance, however, it seems preferable to being stuck on a buggy and obsolete version of PHP for the next couple of years.

Mon, Feb 12, 11:56 PM · Phabricator, Operations