-
Notifications
You must be signed in to change notification settings - Fork 238
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
Discover node features as annotations #1417
Discover node features as annotations #1417
Conversation
✅ Deploy Preview for kubernetes-sigs-nfd ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
/assign @marquiz |
2b7b3f7
to
3aaad73
Compare
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.
Thanks @ArangoGutierrez. Great to move this featre forward. A few comments below.
Also, we need to update docs/usage/customization-guide.md
, at least describe the new annotations
field of the CR
3aaad73
to
d92a448
Compare
@marquiz ready for a second pass |
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.
Thx @ArangoGutierrez for the update. There's still something small in nfd-master.go
Plus docs/usage/customization-guide.md
needs to be updated.
d92a448
to
e449f08
Compare
/retest |
e449f08
to
a87f7dd
Compare
a87f7dd
to
459bd1f
Compare
459bd1f
to
87a17ab
Compare
Signed-off-by: Carlos Eduardo Arango Gutierrez <eduardoa@nvidia.com> Co-authored-by: bebc <mchf1990212@gmail.com> Co-authored-by: Markus Lehtonen <markus.lehtonen@intel.com>
420424e
to
c0063be
Compare
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.
Good job @ArangoGutierrez (and big thanks @bebc for doing a lot of groundwork in #1093!). This is a great new feature.
/lgtm
LGTM label has been added. Git tree hash: 589f463d961e2beaaa718998984d68c445afe71e
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: ArangoGutierrez, marquiz 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 |
/retest |
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## master #1417 +/- ##
==========================================
+ Coverage 30.42% 30.55% +0.13%
==========================================
Files 58 58
Lines 7488 7534 +46
==========================================
+ Hits 2278 2302 +24
- Misses 4962 4982 +20
- Partials 248 250 +2
|
Fixes #863