Differential D21054

Update some Phabricator behaviors for changes to Futures
ClosedPublic
Actions

Authored by epriestley on Apr 3 2020, 7:03 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Tue, Nov 19, 10:15 PM

	Unknown Object (File)
	Sat, Nov 16, 2:23 PM

	Unknown Object (File)
	Mon, Nov 11, 7:19 PM

	Unknown Object (File)
	Fri, Nov 8, 7:53 AM

	Unknown Object (File)
	Mon, Nov 4, 5:16 PM

	Unknown Object (File)
	Sun, Nov 3, 11:48 PM

	Unknown Object (File)
	Fri, Nov 1, 8:45 PM

	Unknown Object (File)
	Sun, Oct 27, 8:40 AM

Subscribers

Restricted Owners Package

Details

Reviewers: None
Maniphest Tasks: T11968: Decide the fate of FutureGraph
Commits: rP1a59cae743c2: Update some Phabricator behaviors for changes to Futures

Summary

Depends on D21053. Ref T11968. Three things have changed:

Overseers can no longer use FutureIterator to continue execution of an arbitrary list of futures from any state. Use FuturePool instead.
Same with repository daemons.
Probably (?) fix an API change in the Harbormaster exec future.

Test Plan

Ran "bin/phd debug task" and "bin/phd debug pull", no longer saw Future-management related errors.
The Harbormaster future is easiest to test by just seeing if production works once this change is deployed there.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Apr 3 2020, 7:03 PM

Owners added a subscriber: Restricted Owners Package.Apr 3 2020, 7:03 PM

Harbormaster returned this revision to the author for changes because remote builds failed.Apr 3 2020, 7:04 PM

Harbormaster failed remote builds in B23977: Diff 50158!

Rebase on "ini_set" unit test fix.

Harbormaster completed remote builds in B23981: Diff 50162.Apr 3 2020, 7:27 PM

This revision was not accepted when it landed; it landed in state Needs Review.Apr 3 2020, 7:27 PM

epriestley requested review of this revision.

Closed by commit rP1a59cae743c2: Update some Phabricator behaviors for changes to Futures (authored by epriestley). · Explain Why

This revision was automatically updated to reflect the committed changes.

epriestley added a commit: rP1a59cae743c2: Update some Phabricator behaviors for changes to Futures.

Revision Contents
Changeset List

Path

Size

src/

applications/

harbormaster/

future/

HarbormasterExecFuture.php

17 lines

repository/

daemon/

PhabricatorRepositoryPullLocalDaemon.php

34 lines

infrastructure/

daemon/

PhutilDaemonOverseer.php

41 lines

Diff 50163

src/applications/harbormaster/future/HarbormasterExecFuture.php

Loading...

src/applications/repository/daemon/PhabricatorRepositoryPullLocalDaemon.php

Loading...

src/infrastructure/daemon/PhutilDaemonOverseer.php

Loading...