Page MenuHomePhabricator

Give AlmanacServices a service type
ClosedPublic

Authored by epriestley on Dec 15 2014, 9:05 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Dec 3, 5:41 PM
Unknown Object (File)
Wed, Nov 27, 9:11 PM
Unknown Object (File)
Wed, Nov 20, 11:13 AM
Unknown Object (File)
Oct 25 2024, 11:46 PM
Unknown Object (File)
Oct 24 2024, 9:21 AM
Unknown Object (File)
Oct 18 2024, 6:52 AM
Unknown Object (File)
Oct 13 2024, 11:59 AM
Unknown Object (File)
Oct 6 2024, 6:28 AM
Subscribers

Details

Summary

Ref T5833. This allows services to be typed, to distinguish between different kinds of services. This makes a few things easier:

  • It's easier for clients to select the services they're interested in (see note in T5873 about Phacility). This isn't a full-power solution, but gets is some of the way there.
  • It's easier to set appropriate permissions around when modifications to the Phabricator cluster are allowed. These service nodes need to be demarcated as special in some way no matter what (see T6741). This also defines a new policy for users who are permitted to create services.
  • It's easier to browse/review/understand services.
  • Future diffs will allow ServiceTypes to specify more service structure (for example, default properties) to make it easier to configure services correctly. Instead of a free-for-all, you'll get a useful list of things that consumers of the service expect to read.

The "custom" service type allows unstructured/freeform services to be created.

Test Plan
  • Created a new service (and hit error cases).
  • Edited an existing service.
  • Saw service types on list and detail views.
  • Poked around new permission stuff.
  • Ran almanac.queryservices with service class specification.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Give AlmanacServices a service type.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
  • Only accept ClusterRepository services as valid cluster nodes in Diffusion.
btrahan edited edge metadata.
This revision is now accepted and ready to land.Dec 16 2014, 12:57 AM
This revision was automatically updated to reflect the committed changes.