HomePhabricator

Use Conduit in PhabricatorRepositoryGitCommitChangeParserWorker

Description

Use Conduit in PhabricatorRepositoryGitCommitChangeParserWorker

Summary:
Ref T2783. This allows this worker to run on a machine different to the one that stores the repository, by routing the execution of Git over Conduit calls.

This API method is super gross, but fixing it isn't straightforward and it runs into other complicated considerations. We can fix it later; for now, just define it as "internal" to limit how much mess this creates.

"Internal" methods do not appear on the console.

Test Plan: Ran bin/repository reparse --change <commit> --trace on several commits, saw daemons make a Conduit call instead of running a git command.

Reviewers: hach-que, chad

Reviewed By: chad

Subscribers: joshuaspence, Korvin, epriestley

Maniphest Tasks: T2783

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

Details

Provenance
hach-queAuthored on Apr 8 2016, 8:08 PM
epriestleyCommitted on Apr 14 2016, 11:53 AM
epriestleyPushed on Apr 14 2016, 11:53 AM
Reviewer
chad
Differential Revision
D11874: Use Conduit in PhabricatorRepositoryGitCommitChangeParserWorker
Parents
rP0379cc10acbd: Fixes T10805. When clustering is not configured, this check should just return.
Branches
Unknown
Tags
Unknown
Tasks
T2783: Make working-copy operations service-oriented
Build Status
Buildable 11710
Build 14667: Run Core Tests