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

cnf-network: Adjusted day1day2 sriov and exmanaged tests #132

Merged
merged 1 commit into from
Sep 23, 2024

Conversation

evgenLevin
Copy link
Collaborator

@evgenLevin evgenLevin commented Aug 6, 2024

Updated the existing Day 1/Day 2 and exmanaged tests due to changes in the Day 1 cluster configuration—VLAN will now be configured directly on the VF instead of through the interface VLAN.

@evgenLevin evgenLevin changed the title DRAFT: cnf-network: Adjusted day1day2 sriov and exmanaged tests cnf-network: Adjusted day1day2 sriov and exmanaged tests Sep 15, 2024
@evgenLevin evgenLevin force-pushed the fix_day1 branch 2 times, most recently from 54ac9a4 to f013c66 Compare September 16, 2024 13:21
@gkopels
Copy link
Collaborator

gkopels commented Sep 18, 2024

Hi Evgeny can you add a description to the PR? Thanks

@gkopels
Copy link
Collaborator

gkopels commented Sep 18, 2024

Just to make sure I understand the bonded interface will be sriov interfaces connected to our switch with a specific VLAN for external connectivity?

@evgenLevin
Copy link
Collaborator Author

@gkopels
I've added the description.
Yes, this is a request to deploy a cluster using a bond interface with enslaved VFs, and to use other created VFs in Day 2 operations.

Copy link
Collaborator

@ajaggapa ajaggapa left a comment

Choose a reason for hiding this comment

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

lgtm

@gkopels
Copy link
Collaborator

gkopels commented Sep 23, 2024

lgtm

@gkopels gkopels merged commit 6c29e4c into openshift-kni:main Sep 23, 2024
1 check passed
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.

3 participants