-
Notifications
You must be signed in to change notification settings - Fork 295
Bug/fix cf stack paging v0.12.x #1482
Bug/fix cf stack paging v0.12.x #1482
Conversation
Rather than ListStacks without any paging. Fixes kubernetes-retired#1479
Fix up tests to use the newly used DescribeStacks
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: If they are not already assigned, you can assign the PR to them by writing The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Codecov Report
@@ Coverage Diff @@
## v0.12.x #1482 +/- ##
===========================================
+ Coverage 37.91% 37.92% +0.01%
===========================================
Files 75 75
Lines 4595 4596 +1
===========================================
+ Hits 1742 1743 +1
Misses 2611 2611
Partials 242 242
Continue to review full report at Codecov.
|
/lgtm |
@mumoshu merge and release a patch for this? Were there any other patches so far relevant for 0.12.x? |
@c-knowles I'm considering to merge #1484 as well before cutting v0.12.1. Edit: Fixed wrong verion num in my prev comment |
@c-knowles Thanks for the fix! |
No description provided.