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

Adding multiqueue support for Virtio to the stemcell #289

Merged
merged 1 commit into from
Oct 24, 2024

Conversation

andinod
Copy link
Contributor

@andinod andinod commented Oct 14, 2024

Hello,

based on the last pull request from: #280,
I have added the property hw_vif_multiqueue_enabled, to enable the multiqueue feature to the network based on virtio, with this will complement the full virtio support.

For any questions or requirements, please let me know.

Thanks

David

@beyhan beyhan requested review from a team, aramprice and bgandon and removed request for a team October 17, 2024 15:06
@jpalermo
Copy link
Member

FYI, there is a rewrite of the CPI coming in golang in the near future. It won't affect your changes here as that rewrite will be in a separate job you have to opt in to.

But it might be these changes will need to be re-done in the new job once it's merged in.

@andinod
Copy link
Contributor Author

andinod commented Oct 17, 2024

Hi @jpalermo,

Thanks for the info!. I'm going to check the code and verify how it is designed for the future changes.

All the best

@beyhan
Copy link
Member

beyhan commented Oct 18, 2024

Thanks for the info!. I'm going to check the code and verify how it is designed for the future changes.

there will be a PR which will bring that in in the next weeks.

@beyhan
Copy link
Member

beyhan commented Oct 18, 2024

It't there now #290

OBoehrer added a commit to OBoehrer/bosh-openstack-cpi-release that referenced this pull request Oct 24, 2024
@rkoster rkoster merged commit 0d6b3d8 into cloudfoundry:master Oct 24, 2024
2 checks passed
@rkoster
Copy link
Contributor

rkoster commented Oct 24, 2024

Thanks! @andinod

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

5 participants