Page MenuHomePhabricator

Remove names from Drydock resources
ClosedPublic

Authored by epriestley on Oct 1 2015, 1:20 PM.
Tags
None
Referenced Files
F13081201: D14213.id34323.diff
Wed, Apr 24, 4:02 PM
Unknown Object (File)
Fri, Apr 19, 3:41 PM
Unknown Object (File)
Wed, Apr 17, 6:16 AM
Unknown Object (File)
Tue, Apr 16, 10:32 AM
Unknown Object (File)
Fri, Apr 12, 5:32 PM
Unknown Object (File)
Mar 11 2024, 1:44 AM
Unknown Object (File)
Mar 4 2024, 6:18 PM
Unknown Object (File)
Mar 4 2024, 6:18 PM
Subscribers
None

Details

Reviewers
hach-que
chad
Maniphest Tasks
T9252: Unprototype Drydock (v1)
Commits
Restricted Diffusion Commit
rPd4a0b1c8709b: Remove names from Drydock resources
Summary

Ref T9252. Long ago you sometimes manually created resources, so they had human-enterable names. However, users never make resources manually any more, so this field isn't really useful any more.

In particular, it means we write a lot of untranslatable strings like "Working Copy" to the database in the default locale. Instead, do the call at runtime so resource names are translatable.

Also clean up a few minor things I hit while kicking the tires here.

It's possible we might eventually want to introduce a human-choosable label so you can rename your favorite resources and this would just be a default name. I don't really have much of a use case for that yet, though, and I'm not sure there will ever be one.

Test Plan
  • Restarted a Harbormaster build, got a clean build.
  • Released all leases/resources, restarted build, got a clean build with proper resource names.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Remove names from Drydock resources.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added reviewers: chad, hach-que.
hach-que edited edge metadata.
This revision is now accepted and ready to land.Oct 1 2015, 1:36 PM
chad edited edge metadata.
This revision was automatically updated to reflect the committed changes.