Conversation
Reduce APIS pollInterval values from 30s to 1s in the kube overlays to reduce preprocessing workflow run times in development. Because the APIS mock server requires 3 polls to complete analysis this reduces the API poll activity time from ~1m30s to ~3s.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #190 +/- ##
=======================================
Coverage 61.12% 61.12%
=======================================
Files 47 47
Lines 2796 2796
=======================================
Hits 1709 1709
Misses 940 940
Partials 147 147 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
@jraddaoui 1 second worked fine when I tested it. Where do you think that timing might be tight? The mock server is certainly capable of returning a response much faster than 1 second, and I don't think we'll be making a lot of simultaneous requests. |
|
Feels safer to me, and it doesn't add a big amount of time as before. We may add extra processing needs to the mock server, add k8s proxies like linkerd, or other changes that could increase response time. Up to you, we can always change it again ;) |
|
Well, I'm impatient so I'll take the risk in exchange for the preprocessing workflow running 12 seconds faster when I'm doing testing. 😈 I think the worst thing that can happen on a slow response from the mock server is that the activity will poll an extra time or two, I'd only really be concerned if we were piling a lot of requests too fast and DOS'd the mock server. |
Reduce APIS pollInterval values from 30s to 1s in the kube overlays to reduce preprocessing workflow run times in development. Because the APIS mock server requires 3 polls to complete analysis this reduces the API poll activity time from ~1m30s to ~3s.