Page MenuHomePhabricator

Fix two straggling pagination issues in Drydock
ClosedPublic

Authored by epriestley on May 23 2019, 6:55 PM.
Tags
None
Referenced Files
F15477439: D20553.id49028.diff
Mon, Apr 7, 2:42 PM
F15463149: D20553.id.diff
Tue, Apr 1, 8:44 PM
F15456552: D20553.diff
Sun, Mar 30, 10:29 AM
F15434433: D20553.id49034.diff
Tue, Mar 25, 3:39 AM
F15432517: D20553.diff
Mon, Mar 24, 6:26 PM
F15424615: D20553.id49028.diff
Sat, Mar 22, 11:19 PM
F15422117: D20553.id49034.diff
Sat, Mar 22, 5:41 AM
F15421055: D20553.id.diff
Fri, Mar 21, 8:39 PM
Subscribers
Restricted Owners Package

Details

Summary

Ref T13289. See https://discourse.phabricator-community.org/t/fatal-error-in-pagination-in-drydock-resources-host-logs-all-logs/2735.

bin/drydock lease and the web UI for reviewing all object logs when there is more than one page of logs didn't get fully updated to the new cursors.

  • Use a cursor pager in bin/drydock lease.
  • Implement withIDs() in LeaseQuery so the default paging works properly.
Test Plan
  • Ran bin/drydock lease, got a lease with log output along the way.
  • Set page size to 2, viewed host logs with multiple pages, paged to page 2.

Diff Detail

Repository
rP Phabricator
Branch
drydock1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 22902
Build 31418: Run Core Tests
Build 31417: arc lint + arc unit