-
Notifications
You must be signed in to change notification settings - Fork 103
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 Node Controller 2/6 #582
Conversation
Hi @mlavacca. Thanks for your PR! I am @adamjensenbot.
Make sure this PR appears in the liqo changelog, adding one of the following labels:
|
Hi @mlavacca. Thanks for your PR! I'm waiting for a Liqo member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test. Once the patch is verified, the new status will be reflected by the ok-to-test label. |
c4711f6
to
30fff7d
Compare
Hi @mlavacca. Thanks for your PR! I'm waiting for a Liqo member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test. Once the patch is verified, the new status will be reflected by the ok-to-test label. |
/ok-to-test |
30fff7d
to
a08af6d
Compare
/rebase |
68ff576
to
63b1421
Compare
63b1421
to
9dd74fe
Compare
9dd74fe
to
b74ca98
Compare
/merge |
Description
This PR is the second related to the NodeController refactoring.
What has been done:
NodeFromProvider
functions from/cmd
to/pkg/
...What has to be done yet:
NodeErrorHandler
fromcmd
to theNodeController
.address #474