Homec4science

Merge the DrydockLease workers into a single worker

Authored by epriestley <git@epriestley.com> on Oct 1 2015, 17:11.

Description

Merge the DrydockLease workers into a single worker

Summary:
Ref T9252. This is the same as D14201, but for lease stuff instead of resource stuff.

This one is a little heavier but still feels pretty reasonable to me at the end of the day (worker is <1K lines and has a ton of comment stuff).

Also fixes a few random bugs I hit in the task queue.

Test Plan:

  • Restarted some Harbormaster builds, saw them go through cleanly.
  • Released pre-activation resources/leases.
  • Probably still kinda buggy but I'll iron the details out over time.

Logs are starting to look somewhat plausible:

{F855747}

Reviewers: chad

Reviewed By: chad

Maniphest Tasks: T9252

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

Details

Committed
epriestley <git@epriestley.com>Oct 1 2015, 17:11
Pushed
aubortJan 31 2017, 17:16
Parents
rPH91e5ca0ee28c: Merge the DrydockResource workers into a single worker
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPH4ac82be5ed22: Merge the DrydockLease workers into a single worker (authored by epriestley <git@epriestley.com>).Oct 1 2015, 17:11