Page MenuHomePhabricator

Fix two straggling pagination issues in Drydock
ClosedPublic

Authored by epriestley on May 23 2019, 6:55 PM.
Tags
None
Referenced Files
F13181304: D20553.diff
Thu, May 9, 11:23 AM
F13174658: D20553.id49034.diff
Wed, May 8, 12:29 AM
Unknown Object (File)
Fri, May 3, 5:58 AM
Unknown Object (File)
Wed, May 1, 1:05 PM
Unknown Object (File)
Sat, Apr 27, 9:34 AM
Unknown Object (File)
Thu, Apr 25, 12:59 AM
Unknown Object (File)
Fri, Apr 19, 8:39 AM
Unknown Object (File)
Wed, Apr 17, 1:52 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