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

Support device id in host device plugin #471

Merged

Conversation

adrianchiris
Copy link
Contributor

No description provided.

Signed-off-by: Adrian Chiris <adrianc@mellanox.com>
Signed-off-by: Adrian Chiris <adrianc@mellanox.com>
@adrianchiris adrianchiris force-pushed the support-device-id-host-device branch from a2dff63 to 2e7e92d Compare April 2, 2020 08:29
@adrianchiris
Copy link
Contributor Author

related to PR : containernetworking/cni#761

@mars1024
Copy link
Member

mars1024 commented Apr 8, 2020

Could you help to add some comments in host-device/README.md ?

@adrianchiris
Copy link
Contributor Author

@mars1024 sure

Copy link
Member

@mars1024 mars1024 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm, but before merging this, you need to update DCO @adrianchiris

@dcbw
Copy link
Member

dcbw commented Apr 8, 2020

/lgtm

@dcbw
Copy link
Member

dcbw commented Apr 8, 2020

@adrianchiris can you add your signed-off-by DCO? Thanks!

@squeed
Copy link
Member

squeed commented Apr 8, 2020

lgtm. Can you amend your commit with a Signed-Off-By so DCO passes, then we can merge this?

Signed-off-by: Adrian Chiris <adrianc@mellanox.com>
@adrianchiris adrianchiris force-pushed the support-device-id-host-device branch from fd3169a to 3e3ffde Compare April 12, 2020 06:36
@adrianchiris
Copy link
Contributor Author

Updated.

@dcbw dcbw merged commit a78853f into containernetworking:master Apr 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants