-
Notifications
You must be signed in to change notification settings - Fork 247
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
Enable NodeFeature API by default #1187
Comments
Would be nice to get feedback when do you think we're ready to make this switchover 😇 @ArangoGutierrez @zvonkok @mythi @PiotrProkop et al |
I think one thing that is missing to make it happen is to implement |
Good point. Could you create an issue about that? |
|
@ArangoGutierrez @PiotrProkop what do you think about this one? Wait until v0.15 or push it already in v0.14 |
I am fine with both approaches. Leader-election is implemented and if gc PR will be merged we should be good to roll it out. |
Thanks @PiotrProkop. I'm also find with both. Leaving this to @ArangoGutierrez then as he probably has the most experience from the production point-of-view 😊 |
once this is done, we should set this to true #1305 |
the API has passed all recent QA tests, so I think is ready for show time! |
/milestone v0.14 |
WIP |
What would you like to be added:
Set default of
-enable-nodefeature-api
to true. IOW, will disable the gRPC API (by default) and, instead, use NodeFeature objects for communicating features and labels from nfd-workers to nfd-master. This will also automatically enable the usage of NodeFeature objects for 3rd party extensions.Why is this needed:
Deprecation of NFD gRPC API.
The text was updated successfully, but these errors were encountered: