Paths

Table of Contentst

Differential D17540

Allow the PullLocal daemon to hibernate, and wake it when repositories need an update
ClosedPublic
Actions

Authored by epriestley on Mar 23 2017, 2:02 AM.

Tags

None

Referenced Files

	F15400394: D17540.id.diff
	Mon, Mar 17, 11:23 AM

	F15367350: D17540.id42186.diff
	Tue, Mar 11, 5:40 PM

	F15283977: D17540.diff
	Tue, Mar 4, 9:00 AM

	Unknown Object (File)
	Feb 11 2025, 8:28 AM

	Unknown Object (File)
	Feb 9 2025, 3:33 AM

	Unknown Object (File)
	Feb 9 2025, 3:33 AM

	Unknown Object (File)
	Feb 9 2025, 3:33 AM

	Unknown Object (File)
	Feb 9 2025, 3:33 AM

Subscribers

None

Details

Reviewers

Maniphest Tasks

T12298: Allow daemon pools to autoscale down to 0 processes

Commits

rP9099485a7125: Allow the PullLocal daemon to hibernate, and wake it when repositories need an…

Summary

Ref T12298. This allows the PullLocal daemon to hibernate like the Trigger daemon, but automatically wakes it back up when it needs to do something.

Test Plan

Ran bin/phd debug pulllocal --trace.
Saw the daemon hibernate after doing a checkup on repositories.
Saw periodic queries to look for new update messages.
After clicking "Update Now" in the web UI to schedule an update, saw the daemon wake up immediately.

Diff Detail

Repository

Branch

wake3

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 16101
Build 21365: Run Core Tests
Build 21364: arc lint + arc unit

Event Timeline

epriestley created this revision.Mar 23 2017, 2:02 AM

Harbormaster failed remote builds in B16101: Diff 42186!Mar 23 2017, 2:03 AM

That build failure is because this depends on D17539, in libphutil.

chad accepted this revision.Mar 23 2017, 4:06 PM

This revision is now accepted and ready to land.Mar 23 2017, 4:06 PM

Closed by commit rP9099485a7125: Allow the PullLocal daemon to hibernate, and wake it when repositories need an… (authored by epriestley, committed by epriestley). · Explain WhyMar 23 2017, 5:52 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

__phutil_library_map__.php

2 lines

applications/

repository/

daemon/

PhabricatorRepositoryPullLocalDaemon.php

13 lines

PhabricatorRepositoryPullLocalDaemonModule.php

38 lines

infrastructure/

daemon/

overseer/

PhabricatorDaemonOverseerModule.php

29 lines

Diff 42186

src/__phutil_library_map__.php

Loading...

src/applications/repository/daemon/PhabricatorRepositoryPullLocalDaemon.php

Loading...

src/applications/repository/daemon/PhabricatorRepositoryPullLocalDaemonModule.php

Loading...

src/infrastructure/daemon/overseer/PhabricatorDaemonOverseerModule.php

Loading...