Enable codecov reporting

[marquiz]

What would you like to be added:

Now that kubernetes/org#3084 has been solved, we could enable codecov reporing in NFD. We should probably separate unit tests from the image build (Dockerfile) into a distinct prow job. An outline of sub-tasks (might be missing some)

1. Enable coverage output in Makefile test: target
2. Add new presubmit-test Prow job
   • A new test script in NFD under scripts/test-infra/
   • Job config in https://github.com/kubernetes/test-infra
   • Possibly remove make test from Dockerfile(?)
3. Get codecov token added as a secret in the k8s test-infra cluster
4. Add codecov reporting in the NFD test-infra script (``scripts/test-infra/`)

Why is this needed:

Improve development process

[fmuyassarov]

+1 for the codecov and I guess it makes sense to decouple the unit tests from the Dockerfile into its own job.

[ArangoGutierrez]

+1

[fmuyassarov]

@marquiz do you plan to work on this ?

[marquiz]

@marquiz do you plan to work on this ?

Feel free to help out here 😉 TBH, I don't know if I have time to take care of this in the near future

[AhmedGrati]

@marquiz I can work on it once I finish my current PR.

[fmuyassarov]

Hi @AhmedGrati. Thanks for offering help. But I had it halfway. If you don't mind let me finish it and if I don't for some reason I will let you know here.
/assign

[marquiz]

I don't think we're quite ready yet
/reopen

[k8s-ci-robot]

@marquiz: Reopened this issue.

In response to this:

I don't think we're quite ready yet
/reopen

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[ArangoGutierrez]

I think we are ready here, right @fmuyassarov

[fmuyassarov]

I believe yes. This is done via #1201, #1200, #1199, #1194, #1095, #1085, #1069.
/close

[k8s-ci-robot]

@fmuyassarov: Closing this issue.

In response to this:

I believe yes. This is done via #1201, #1200, #1199, #1194, #1095, #1085, #1069.
/close

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.