HomePhabricator

Cache generation of the SSH authentication keyfile for sshd

Description

Cache generation of the SSH authentication keyfile for sshd

Summary:
Ref T11469. This isn't directly related, but has been on my radar for a while: building SSH keyfiles (particular for installs with a lot of keys, like ours) can be fairly slow.

At least one cluster instance is making multiple clone requests per second. While that should probably be rate limited separately, caching this should mitigate the impact of these requests.

This is pretty straightforward to cache since it's exactly the same every time, and only changes when users modify SSH keys (which is rare).

Test Plan:

  • Ran bin/auth-ssh, saw authfile generate.
  • Ran it again, saw it read from cache.
  • Changed an SSH key.
  • Ran it again, saw it regenerate.

Reviewers: chad

Reviewed By: chad

Maniphest Tasks: T11469

Differential Revision: https://secure.phabricator.com/D16744

Details

Provenance
epriestleyAuthored on Oct 21 2016, 2:23 PM
epriestleyPushed on Oct 21 2016, 2:29 PM
Reviewer
chad
Differential Revision
D16744: Cache generation of the SSH authentication keyfile for sshd
Parents
rPeb80f3fcd529: Fix policy capitalization
Branches
Unknown
Tags
Unknown
Tasks
T11469: Checking out from Phacility sometimes fails
Build Status
Buildable 14216
Build 18478: Run Core Tests