Page MenuHomePhabricator

Generate a random unique "Request ID" for SSH requests so processes can coordinate better
ClosedPublic

Authored by epriestley on Mar 22 2018, 7:56 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Jan 12, 12:46 PM
Unknown Object (File)
Dec 10 2024, 6:50 PM
Unknown Object (File)
Dec 10 2024, 5:26 AM
Unknown Object (File)
Dec 7 2024, 12:24 PM
Unknown Object (File)
Dec 3 2024, 1:41 PM
Unknown Object (File)
Dec 2 2024, 5:03 PM
Unknown Object (File)
Nov 27 2024, 2:20 PM
Unknown Object (File)
Nov 27 2024, 1:05 AM
Subscribers
None

Details

Summary

Depends on D19247. Ref T13109. When we receive an SSH request, generate a random unique ID for the request. Then thread it down through the process tree.

The immediate goal is to let the ssh-exec process coordinate with commit-hook process and log information about read and write lock wait times. Today, there's no way for ssh-exec to interact with the PushEvent, but this is the most helpful place to store this data for users.

Test Plan

Made pushes, saw the PushEvent table populate with a random request ID. Exported data and saw the ID preserved in the export.

Diff Detail

Repository
rP Phabricator
Branch
lock2
Lint
Lint Passed
SeverityLocationCodeMessage
Advicesrc/applications/config/option/PhabricatorAccessLogConfigOptions.php:51XHP16TODO Comment
Unit
Tests Passed
Build Status
Buildable 19902
Build 26976: Run Core Tests
Build 26975: arc lint + arc unit

Event Timeline

This revision was not accepted when it landed; it landed in state Needs Review.Mar 22 2018, 8:44 PM
This revision was automatically updated to reflect the committed changes.