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.

Sign up for GitHub

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

Jump to bottom

DNM: Verify policy is only applied to partions' VFs #541

Closed

mlguerrero12 wants to merge 1 commit into k8snetworkplumbingwg:master from mlguerrero12:verifypfpartition

Contributor

mlguerrero12 commented Nov 8, 2023

No description provided.

github-actions bot commented Nov 8, 2023

Thanks for your PR,
To run vendors CIs use one of:

/test-all: To run all tests for all vendors.
/test-e2e-all: To run all E2E tests for all vendors.
/test-e2e-nvidia-all: To run all E2E tests for NVIDIA vendor.

To skip the vendors CIs use one of:

/skip-all: To skip all tests for all vendors.
/skip-e2e-all: To skip all E2E tests for all vendors.
/skip-e2e-nvidia-all: To skip all E2E tests for NVIDIA vendor.
Best regards.

coveralls commented Nov 8, 2023 •

edited

Loading

Pull Request Test Coverage Report for Build 6863441514

0 of 0 changed or added relevant lines in 0 files are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.09%) to 25.464%

Files with Coverage Reduction	New Missed Lines	%
controllers/sriovibnetwork_controller.go	2	70.45%

Totals
Change from base Build 6848351544:	0.09%
Covered Lines:	2334
Relevant Lines:	9166

💛 - Coveralls

mlguerrero12 force-pushed the verifypfpartition branch from 58af181 to 91250f0 Compare

November 8, 2023 12:56

github-actions bot commented Nov 8, 2023

Thanks for your PR,
To run vendors CIs use one of:

/test-all: To run all tests for all vendors.
/test-e2e-all: To run all E2E tests for all vendors.
/test-e2e-nvidia-all: To run all E2E tests for NVIDIA vendor.

To skip the vendors CIs use one of:

/skip-all: To skip all tests for all vendors.
/skip-e2e-all: To skip all E2E tests for all vendors.
/skip-e2e-nvidia-all: To skip all E2E tests for NVIDIA vendor.
Best regards.


          Verify policy is only applied to partions' VFs

7ff06d8

mlguerrero12 force-pushed the verifypfpartition branch from 91250f0 to 7ff06d8 Compare

November 14, 2023 12:15

github-actions bot commented Nov 14, 2023

Thanks for your PR,
To run vendors CIs use one of:

/test-all: To run all tests for all vendors.
/test-e2e-all: To run all E2E tests for all vendors.
/test-e2e-nvidia-all: To run all E2E tests for NVIDIA vendor.

To skip the vendors CIs use one of:

/skip-all: To skip all tests for all vendors.
/skip-e2e-all: To skip all E2E tests for all vendors.
/skip-e2e-nvidia-all: To skip all E2E tests for NVIDIA vendor.
Best regards.

Contributor Author

mlguerrero12 commented Nov 14, 2023

DNM - failed OCP job shows that there is a bug. It will be fixed in #539. Test will also be included there.

mlguerrero12 mentioned this pull request

Skip vf configuration when not found in VF group #539

Merged

mlguerrero12 changed the title ~~Verify policy is only applied to partions' VFs~~ DNM: Verify policy is only applied to partions' VFs

mlguerrero12 closed this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet