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

Update modules #31

Closed
wants to merge 4 commits into from
Closed

Update modules #31

wants to merge 4 commits into from

Conversation

cgiraldo
Copy link

Use k8s.io/kubelet/pkg/apis/deviceplugin/v1beta1 instead of k8s.io/kubernetes/pkg/kubelet/apis/deviceplugin/v1beta1.

Use sirupsen/logrus instead of Sirupsen/logrus sirupsen/logrus#570 (comment)

@yuzhang66
Copy link
Contributor

Hi Cgiraldo, Thank you for your contribution! Considering there's a lot library updates, I need to testing this pull request in our internal testing repo and going through the open-source legal scanning process, so currently I'm not able to directly merge your pull request, I will add your name & github account in an contribution list in our next release to prove your contribution, is this ok for you?

@cgiraldo
Copy link
Author

It is Ok.

@cgiraldo cgiraldo closed this May 25, 2022
@cgiraldo
Copy link
Author

i have also include a commit to support zcu102 board. Thanks to @Vae1997 and the repo: https://github.com/Vae1997/zcu102-fpga-device-plugin-demo

@cgiraldo
Copy link
Author

cgiraldo commented Jun 2, 2022

Github repo was transfered to Gradiant Organization:
https://github.com/Gradiant/SI-PR-01370-FPGA_as_a_Service

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.

2 participants