Page MenuHomePhabricator

Move Drydock logs to PHIDs and increased structure
ClosedPublic

Authored by epriestley on Sep 30 2015, 2:43 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Apr 19, 3:40 PM
Unknown Object (File)
Sat, Apr 13, 12:13 PM
Unknown Object (File)
Fri, Apr 12, 6:04 PM
Unknown Object (File)
Thu, Apr 11, 7:50 AM
Unknown Object (File)
Wed, Apr 10, 11:10 AM
Unknown Object (File)
Sun, Apr 7, 11:33 PM
Unknown Object (File)
Mar 4 2024, 4:35 PM
Unknown Object (File)
Feb 10 2024, 1:13 PM
Subscribers
None

Details

Summary

Ref T9252. Several general changes here:

  • Moves logs to use PHIDs instead of IDs. This generally improves flexibility (for example, it's a lot easier to render handles).
  • Adds blueprintPHID to logs. Although you can usually figure this out from the leasePHID or resourcePHID, it lets us query relevant logs on Blueprint views.
  • Instead of making logs a top-level object, make them strictly a sub-object of Blueprints, Resources and Leases. So you go Drydock > Lease > Logs, etc., to get to logs.
    • I might restore the "everything" view eventually, but it doesn't interact well with policies and I'm not sure it's very useful. A policy-violating bin/drydock log might be cleaner.
  • Policy-wise, we always show you that logs exist, we just don't show you log content if it's about something you can't see. This is similar to seeing restricted handles in other applications.
  • Instead of just having a message, give logs "type" + "data". This will let logs be more structured and translatable. This is similar to recent changes to Herald which seem to have worked well.
Test Plan

Added some placeholder log writes, viewed those logs in the UI.

Screen Shot 2015-09-30 at 7.42.42 AM.png (989×1 px, 171 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Move Drydock logs to PHIDs and increased structure.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad edited edge metadata.
This revision is now accepted and ready to land.Sep 30 2015, 5:35 PM
This revision was automatically updated to reflect the committed changes.