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

feat: async calls #311

Merged
merged 6 commits into from
Apr 24, 2023
Merged

feat: async calls #311

merged 6 commits into from
Apr 24, 2023

Conversation

AlexsJones
Copy link
Member

@AlexsJones AlexsJones commented Apr 21, 2023

Closes #

πŸ“‘ Description

βœ… Checks

  • My pull request adheres to the code style of this project
  • My code requires changes to the documentation
  • I have updated the documentation as required
  • All the tests have passed

β„Ή Additional Information

Signed-off-by: Alex Jones <alexsimonjones@gmail.com>
@AlexsJones AlexsJones requested review from a team as code owners April 21, 2023 04:07
@matthisholleville
Copy link
Contributor

Great job, Alex! It looks like the client is experiencing a bit of throttling on one of my clusters with 100 namespaces and 500 deployments:

I0421 09:04:25.405611 32626 request.go:690] Waited for 1.128472333s due to client-side throttling, not priority and fairness, request: GET:...

To avoid this warning and prevent overloading the Kubernetes server, should we consider using channels to synchronize our Go routines? This will help us wait for the completion of each request before sending another.

@AlexsJones
Copy link
Member Author

Ill take a look using a buffered channel

Signed-off-by: Alex Jones <alexsimonjones@gmail.com>
@AlexsJones AlexsJones enabled auto-merge (squash) April 24, 2023 14:04
@AlexsJones AlexsJones requested a review from a team as a code owner April 24, 2023 14:04
@AlexsJones AlexsJones merged commit c3cc413 into main Apr 24, 2023
@AlexsJones AlexsJones deleted the feat/async branch April 24, 2023 14:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

None yet

2 participants