-
Notifications
You must be signed in to change notification settings - Fork 83
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
Optimize cache warmup on startup #276
Optimize cache warmup on startup #276
Conversation
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: markusthoemmes The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Codecov Report
@@ Coverage Diff @@
## master #276 +/- ##
==========================================
- Coverage 65.99% 65.06% -0.93%
==========================================
Files 17 17
Lines 641 624 -17
==========================================
- Hits 423 406 -17
Misses 191 191
Partials 27 27
Continue to review full report at Codecov.
|
1b96b1e
to
abacbe6
Compare
abacbe6
to
54d81c5
Compare
It'd be good to rebase this PR to pick #282 and run the e2e tests. |
54d81c5
to
3fa42a0
Compare
Done! |
02b8d23
to
956d964
Compare
0c92bf2
to
6a89c78
Compare
6a89c78
to
6e2d69d
Compare
6e2d69d
to
a4f328d
Compare
a4f328d
to
9ac1af7
Compare
The following is the coverage report on the affected files.
|
/lgtm |
This makes cache warmup at startup simpler (and most likely faster too):
Instead of relying on the global resync to do its thing (which can be very slow, especially because of our two-lane queues), this runs all the ingresses through the reconciler without queueing them and before launching the config server. Having this seeded config is necessary to start it anyway, so we might as well block startup on it completely.