-
Notifications
You must be signed in to change notification settings - Fork 106
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
openstack: Cherry-pick from upstream #641
Conversation
As well known and well documented, in virtual deployments of Kubernetes such as running on top of OpenStack where the underlying virtualization platform is KVM, it does not support a virtualized iommu, the VFIO PCI driver needs to be loaded with a special flag. This patch aims to do it by default in the virtual_plugin, since this plugin is only used by OpenStack for now. Note: it also changes `utils.LoadKernelModule` function to support kernel module arguments. It also adds a note in `load-kmod.sh` about the fact that a module won't be reloaded with potential new options if the module is already loaded on the system.
This commit includes a new dependency that provides a HTTP client interface with automatic retries and exponential backoff.
In most cases, OpenStack metadata service is available in the cloud, which removes the requirement of config-drive. This patch will change the logic to try reading Metadata and Network data from the config-drive directory but if it's not present, it'll try to get it by safely fetching the URL.
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: MaysaMacedo The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
@MaysaMacedo: The following tests failed, say
Full PR test history. Your PR dashboard. 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. I understand the commands that are listed here. |
/hold |
/close |
@MaysaMacedo: Closed this PR. In response to this:
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. |
This PR backports the following PRs:
k8snetworkplumbingwg/sriov-network-operator#257
k8snetworkplumbingwg/sriov-network-operator#258