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

[SecondaryNetwork] Require resourceName annotations for SRIOV #6999

Conversation

antoninbas
Copy link
Contributor

The "k8s.v1.cni.cncf.io/resourceName" annotation should be provided for all SRIOV NetworkAttachmentDefinitions. Otherwise, it is impossible to guarantee that the right device type is allocated for a given interface.

The "k8s.v1.cni.cncf.io/resourceName" annotation should be provided for
all SRIOV NetworkAttachmentDefinitions. Otherwise, it is impossible to
guarantee that the right device type is allocated for a given interface.

Signed-off-by: Antonin Bas <[email protected]>
@antoninbas
Copy link
Contributor Author

antoninbas commented Feb 15, 2025

References:

@antoninbas
Copy link
Contributor Author

Depends on #6922 for testing

@antoninbas antoninbas added the area/secondary-network Issues or PRs related to support for secondary networks in Antrea label Feb 15, 2025
@antoninbas antoninbas added this to the Antrea v2.3 release milestone Feb 15, 2025
Copy link
Member

@tnqn tnqn left a comment

Choose a reason for hiding this comment

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

LGTM

@wenqiq
Copy link
Contributor

wenqiq commented Feb 17, 2025

Depends on #6922 for testing
The test process record:
https://jenkins.antrea.io/job/cloud-antrea-test-aws-sriov-secondary-network/33

@antoninbas
Copy link
Contributor Author

Thanks @wenqiq, I appreciate you running the tests for this PR!

@antoninbas
Copy link
Contributor Author

/skip-all

@antoninbas antoninbas merged commit cb63c72 into antrea-io:main Feb 19, 2025
59 of 61 checks passed
@antoninbas antoninbas deleted the secondary-network-require-resourceName-for-sriov-interfaces branch February 19, 2025 20:39
@antoninbas antoninbas added the action/release-note Indicates a PR that should be included in release notes. label Feb 19, 2025
@antoninbas antoninbas mentioned this pull request Feb 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
action/release-note Indicates a PR that should be included in release notes. area/secondary-network Issues or PRs related to support for secondary networks in Antrea
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants