Skip to content
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

fix(v2client): updates limits for list secrets and routes #270

Merged
merged 1 commit into from
Feb 8, 2024

Conversation

pallabpain
Copy link
Contributor

Description

The existing limits for list secrets and static routes were set to 10 which is quite low. This leads to an increased number of calls to the apiserver. This commit increases the limit to 50, reducing the calls required to fetch all the items.

The existing limits for list secrets and static routes was set to 10
which is quite low. This leads to increased number of calls to the
apiserver. This commit increases the limit to 50, reducing the number of
calls required to fetch all the items.
@pallabpain pallabpain requested review from a team as code owners February 8, 2024 06:27
@pallabpain pallabpain self-assigned this Feb 8, 2024
@pallabpain pallabpain added the 🐛 bug Something isn't working label Feb 8, 2024
Copy link

github-actions bot commented Feb 8, 2024

🤖 Pull Request Artifacts (#7825748545) 🎉

@pallabpain pallabpain merged commit 862fdfc into devel Feb 8, 2024
6 checks passed
rr-github-ci-user pushed a commit that referenced this pull request Feb 14, 2024
# [7.1.0](v7.0.3...v7.1.0) (2024-02-14)

### Bug Fixes

* corrects regex for fetching resources ([1d3c75e](1d3c75e))
* sets default confirmation action to 'No' ([b3348af](b3348af))
* **v2client:** updates limits for list secrets and routes ([#270](#270)) ([862fdfc](862fdfc))

### Features

* **explain:** updates examples for region support and env var inconsistency ([#273](#273)) ([62a2828](62a2828))

### Reverts

* **explain:** adds network examples for us region ([#276](#276)) ([3593def](3593def))
@rr-github-ci-user
Copy link
Collaborator

🎉 This PR is included in version 7.1.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

RomilShah pushed a commit that referenced this pull request Apr 23, 2024
The existing limits for list secrets and static routes was set to 10
which is quite low. This leads to increased number of calls to the
apiserver. This commit increases the limit to 50, reducing the number of
calls required to fetch all the items.
RomilShah pushed a commit that referenced this pull request Apr 23, 2024
# [7.1.0](v7.0.3...v7.1.0) (2024-02-14)

### Bug Fixes

* corrects regex for fetching resources ([1d3c75e](1d3c75e))
* sets default confirmation action to 'No' ([b3348af](b3348af))
* **v2client:** updates limits for list secrets and routes ([#270](#270)) ([862fdfc](862fdfc))

### Features

* **explain:** updates examples for region support and env var inconsistency ([#273](#273)) ([62a2828](62a2828))

### Reverts

* **explain:** adds network examples for us region ([#276](#276)) ([3593def](3593def))
@ankitrgadiya ankitrgadiya deleted the fix/staticroute-list branch May 28, 2024 06:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants