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

Virtual kubelet: working queue-based reflection manager #925

Merged
merged 2 commits into from Oct 12, 2021

Conversation

giorio94
Copy link
Member

@giorio94 giorio94 commented Oct 5, 2021

Description

This PR introduces in the virtual kubelet a reflection manager leveraging working queues, for better resiliency and improved performance. Currently, only the management/generic logic is introduced, while future PRs will implement the different reflectors.

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Please also note any relevant details for your test configuration.

  • New unit tests
  • Existing

@giorio94
Copy link
Member Author

giorio94 commented Oct 5, 2021

/test

1 similar comment
@giorio94
Copy link
Member Author

giorio94 commented Oct 5, 2021

/test

@giorio94 giorio94 marked this pull request as ready for review October 5, 2021 16:46
@giorio94
Copy link
Member Author

/hold Giving precedence to #933 to limit conflicts

@giorio94 giorio94 added the hold Prevent bot merging label Oct 11, 2021
@giorio94
Copy link
Member Author

/unhold #933 has been merged

@giorio94
Copy link
Member Author

/test

@giorio94 giorio94 removed the hold Prevent bot merging label Oct 11, 2021
@giorio94
Copy link
Member Author

/rebase test=true

@giorio94
Copy link
Member Author

/test

1 similar comment
@giorio94
Copy link
Member Author

/test

@giorio94 giorio94 added the kind/cleanup Refactoring code label Oct 12, 2021
@giorio94
Copy link
Member Author

/merge

@adamjensenbot adamjensenbot added the merge-requested Request bot merging (automatically managed) label Oct 12, 2021
@adamjensenbot adamjensenbot merged commit 7bc4895 into master Oct 12, 2021
@adamjensenbot adamjensenbot deleted the mio/reflection-manager branch October 12, 2021 17:39
@adamjensenbot adamjensenbot removed the merge-requested Request bot merging (automatically managed) label Oct 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/cleanup Refactoring code size/XXL
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants