Page MenuHomePhabricator
Differential D19174

Add a "lock log" for debugging where locks are being held
ClosedPublic
Actions

Authored by epriestley on Mar 5 2018, 11:27 PM.
Tags
None
Referenced Files
F13212471: D19174.diff
Fri, May 17, 6:45 AM
F13202045: D19174.id45928.diff
Tue, May 14, 8:54 PM
F13196065: D19174.diff
Sun, May 12, 10:54 PM
F13179782: D19174.diff
Wed, May 8, 9:26 PM
Unknown Object (File)
Sat, May 4, 7:06 PM
Unknown Object (File)
Wed, May 1, 12:01 AM
Unknown Object (File)
Sun, Apr 21, 9:27 PM
Unknown Object (File)
Mar 31 2024, 2:20 AM
View All 77 Files
Subscribers
None

Details

Reviewers
None
Maniphest Tasks
T13096: Implement a global lock log to make it easier to debug global locks
Commits
rP44f0664d2c6f: Add a "lock log" for debugging where locks are being held
Summary

Depends on D19173. Ref T13096. Adds an optional, disabled-by-default lock log to make it easier to figure out what is acquiring and holding locks.

Test Plan

Ran bin/lock log --enable, --disable, --name, etc. Saw sensible-looking output with log enabled and daemons restarted. Saw no additional output with log disabled and daemons restarted.

Diff Detail

Repository
rP Phabricator
Branch
lock2
Lint
Lint Warnings
SeverityLocationCodeMessage
Warningsrc/infrastructure/util/PhabricatorGlobalLock.php:155XHP79Global Variables
Unit
Tests Passed
Build Status
Buildable 19739
Build 26736: Run Core Tests
Build 26735: arc lint + arc unit

Event Timeline

epriestley created this revision.Mar 5 2018, 11:27 PM
Harbormaster failed remote builds in B19739: Diff 45928!Mar 5 2018, 11:27 PM
This revision was not accepted when it landed; it landed in state Draft.Mar 6 2018, 1:55 AM
Closed by commit rP44f0664d2c6f: Add a "lock log" for debugging where locks are being held (authored by epriestley). · Explain Why
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
bin/
1 line
resources/
sql/
autopatches/
9 lines
scripts/
setup/
21 lines
src/
8 lines
applications/
daemon/
garbagecollector/
29 lines
management/
222 lines
4 lines
storage/
32 lines
infrastructure/
daemon/
garbagecollector/
6 lines
storage/
management/
workflow/
12 lines
util/
51 lines

Diff 45928

View Options

bin/lock

Loading...
View Options

resources/sql/autopatches/20180305.lock.01.locklog.sql

Loading...
View Options

scripts/setup/manage_lock.php

Loading...
View Options

src/__phutil_library_map__.php

Loading...
View Options

src/applications/daemon/garbagecollector/PhabricatorDaemonLockLogGarbageCollector.php

Loading...
View Options

src/applications/daemon/management/PhabricatorLockLogManagementWorkflow.php

Loading...
View Options

src/applications/daemon/management/PhabricatorLockManagementWorkflow.php

Loading...
View Options

src/applications/daemon/storage/PhabricatorDaemonLockLog.php

Loading...
View Options

src/infrastructure/daemon/garbagecollector/PhabricatorGarbageCollector.php

Loading...
View Options

src/infrastructure/storage/management/workflow/PhabricatorStorageManagementWorkflow.php

Loading...
View Options

src/infrastructure/util/PhabricatorGlobalLock.php

Loading...
Phabricator · Built by Phacility