Page MenuHomePhabricator

Cleanup some clustering rough edges
ClosedPublic

Authored by amckinley on Dec 20 2018, 12:26 AM.
Tags
None
Referenced Files
F19099522: D19917.id.diff
Dec 4 2025, 8:12 PM
F19089428: D19917.diff
Dec 3 2025, 7:15 AM
F19075624: D19917.diff
Dec 1 2025, 10:35 AM
F19075592: D19917.diff
Dec 1 2025, 10:25 AM
F19075560: D19917.diff
Dec 1 2025, 10:15 AM
F19075382: D19917.diff
Dec 1 2025, 9:40 AM
F18852260: D19917.diff
Oct 31 2025, 7:44 AM
F18840651: D19917.id.diff
Oct 28 2025, 3:03 AM
Subscribers

Details

Summary

Suppress an unhelpful Almanac transaction and document the location of the secret clustering management capability. I thought maybe implementing shouldHide and checking for isCreate would work, but the binding apparently gets created before an interface is bound to it.

Test Plan

Looked at a fresh binding and didn't see "Unknown Object(??)", ran bin/diviner and saw expected output.

Diff Detail

Repository
rP Phabricator
Branch
master
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 21393
Build 29131: Run Core Tests
Build 29130: arc lint + arc unit

Event Timeline

epriestley added inline comments.
src/applications/almanac/xaction/AlmanacBindingInterfaceTransaction.php
59–62

Some day, this transaction will be marked as TYPE_CREATE and hidden, but adding a better rendering to it is quite reasonable for now (and maybe we'll go back and add more of these if we let you render hidden transactions with transaction.search at some point).

63–66

I think removing an interface is impossible so this code is unreachable.

This revision is now accepted and ready to land.Dec 20 2018, 2:53 PM
This revision was automatically updated to reflect the committed changes.