Page MenuHomePhabricator

Add support for a "containerPHID" in the worker queue
ClosedPublic

Authored by epriestley on Jan 29 2021, 12:33 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Nov 24, 9:48 PM
Unknown Object (File)
Thu, Oct 31, 10:27 AM
Unknown Object (File)
Oct 24 2024, 3:08 PM
Unknown Object (File)
Oct 15 2024, 10:22 AM
Unknown Object (File)
Oct 15 2024, 9:22 AM
Unknown Object (File)
Oct 14 2024, 1:05 AM
Unknown Object (File)
Oct 13 2024, 9:36 PM
Unknown Object (File)
Oct 13 2024, 7:11 PM
Subscribers
Restricted Owners Package

Details

Summary

Ref T13591. Worker queue tasks which affect commits currently (mostly) store the commit as an "objectPHID", but do not directly reference the repository the commit belongs to.

This can make certain operations (like "change the priority of all tasks affecting repository Y") more difficult than it needs to be.

Support a "containerPHID", similar to the field of the same name on builds, that can store a parent object like a repository and better support operations against subsets of tasks.

See also D11044 for the genesis of "objectPHID".

This depends on the introduction of storage patch phases (in D21529) so that earlier migrations which queue worker tasks don't try to insert this column before it actually exists.

Test Plan
  • Ran bin/storage upgrade.
  • No callers yet, see further changes for usage.

Diff Detail

Repository
rP Phabricator
Branch
phase2
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 25063
Build 34581: Run Core Tests
Build 34580: arc lint + arc unit

Event Timeline

Owners added a subscriber: Restricted Owners Package.Jan 29 2021, 12:33 AM
This revision was not accepted when it landed; it landed in state Needs Review.Feb 2 2021, 9:40 PM
This revision was automatically updated to reflect the committed changes.