-
Notifications
You must be signed in to change notification settings - Fork 23.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
route53_info: add pagination support to list #60433
Conversation
@bkmeneguello, just so you are aware we have a dedicated Working Group for aws. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks very much for this patch @bkmeneguello, would like a small change in the implementation please.
7a554c1
to
6e2557f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the quick update @bkmeneguello! Tests pass locally for me and this change looks good, but I should note I don't have a test env with a large amount of items readily available.
@IPvSean - would you be able to test with this patch and see if it resolves #49457 for you please?
Hi, I'm using this feature in my domain with more than 2k records and no problems at the moment. Could someone merge this PR? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the ping @bkmeneguello, apologies for the delay.
SUMMARY
Add support for paginated results when lists returns more then 100 results
Fixes #49457
ISSUE TYPE
COMPONENT NAME
route53_info
ADDITIONAL INFORMATION