search

krkn-chaos / cerberus

Guardian of Kubernetes clusters. Tool to monitor clusters health and signal/alert on failures.
Apache License 2.0
92 stars 41 forks source link

Looping through continue #167

Closed paigerube14 closed 2 years ago

paigerube14 commented 2 years ago

Start of fixing uses of limits and using continue to get list of all namespaces and nodes using a common helper function

Need to continue with other client functions that use limit

Fixes

https://github.com/chaos-kubox/cerberus/issues/165

comet-perf-ci commented 2 years ago

Can one of the admins verify this patch?

ShellyXueHan commented 2 years ago

Can one of the admins verify this patch?

+1, experiencing same issue and we are also waiting for fix to merge in! thx!

paigerube14 commented 2 years ago

@janosdebugs @chaitanyaenr @sanjacodes can you please take a look at these changes when you get a chance. Easiest way to make sure it's working is set kube_api_request_chunk_size to 2 in your config and make sure it runs properly

I wasn't able to use this new function for _get_nodeinfo and _get_podstatus because they don't have a "_continue" attribute