Skip to content
  • epriestley's avatar
    Merge the "Herald" and "Owners" daemon workers into a single "Publish" worker · c0a4d1de
    epriestley authored
    Summary:
    Depends on D20466. Ref T13277. Currently:
    
      - The "Owners" worker writes ownership relationships (e.g., commit X affects package Y, because it touches a path in package Y) -- these are just edges.
      - It also triggers audits.
      - Then it queues a "Herald" worker.
      - This formally publishes the commit and triggers Herald.
    
    These aren't really separate steps and can happen more easily in one shot. Merge them.
    
    Test Plan:
      - Ran `bin/repository reparse --publish` to republish various commits, got sensible behavior.
      - Grepped for "IMPORTED_OWNERS", "IMPORTED_HERALD", "--herald", "--owners", and "--force-local" flags.
    
    Reviewers: amckinley
    
    Reviewed By: amckinley
    
    Maniphest Tasks: T13277
    
    Differential Revision: https://secure.phabricator.com/D20467
    c0a4d1de