-
Notifications
You must be signed in to change notification settings - Fork 41.6k
Add a caching minion registry. #483
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
Conversation
LGTM modulo comments. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
do you want to do a force refresh here?
this will not refresh the cache if it is not expired.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done.
Comments addressed. ptal. Thanks! |
lgtm |
Tests added. ptal. Thanks! |
LGTM |
1 similar comment
LGTM |
Add a caching minion registry.
Remove reports that are no longer valid
Remove reports that are no longer valid
Remove reports that are no longer valid
Proposed schedule for 1.9 release
adds a unit test for checking if graceful shutdown of HTTP2 server works
add a subnet manager backed by the kubernetes apiserver
No description provided.