Page MenuHomePhabricator

Clean up the workflow for some post-push logging code
ClosedPublic

Authored by epriestley on Nov 6 2018, 9:24 PM.
Tags
None
Referenced Files
F19082846: D19778.id47264.diff
Tue, Dec 2, 12:26 PM
F19065278: D19778.id47264.diff
Sun, Nov 30, 1:08 AM
F19054698: D19778.id47244.diff
Fri, Nov 28, 2:24 PM
F19017841: D19778.diff
Nov 23 2025, 9:40 AM
F19005312: D19778.id47244.diff
Nov 21 2025, 3:06 PM
F19004858: D19778.diff
Nov 21 2025, 12:44 PM
F18984948: D19778.id47244.diff
Nov 17 2025, 12:49 PM
F18783618: D19778.diff
Oct 13 2025, 8:27 AM
Subscribers
Restricted Owners Package

Details

Summary

Ref T13216. When a repository is clustered, we run this cleanup code (to tell the repository to update, and log some timing information) on both nodes. Currently, we do slightly too much work, which is unnecessary and can be a bit confusing to human readers.

The double update message doesn't hurt anything, but there's no reason to write it twice.

Likewise, the second timing information update query doesn't do anything: there's no PushEvent object with the right identifier, so it just updates nothing. We don't need to run it, and it's confusing that we do.

Instead, only do these writes if we're actually the final node with the repository on it.

Test Plan

Added some logging, saw double writes/updates before the change and no doubles afterwards, with no other behavioral changes.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable