Page MenuHomePhabricator

Remove names from Drydock resources
ClosedPublic

Authored by epriestley on Oct 1 2015, 1:20 PM.
Tags
None
Referenced Files
F14051598: D14213.diff
Fri, Nov 15, 2:12 AM
F14037632: D14213.diff
Sun, Nov 10, 5:40 PM
F14035252: D14213.id.diff
Sun, Nov 10, 5:11 AM
F14025078: D14213.diff
Thu, Nov 7, 2:17 PM
F13998877: D14213.id34314.diff
Thu, Oct 24, 11:39 AM
F13984345: D14213.diff
Sun, Oct 20, 12:39 PM
F13969791: D14213.id34323.diff
Oct 17 2024, 3:52 AM
F13956909: D14213.id34314.diff
Oct 14 2024, 7:31 AM
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.