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

runtime: agent: use PCI segments 1+ for blk devices #183

Merged
merged 5 commits into from
Apr 30, 2024

Conversation

danmihai1
Copy link

@danmihai1 danmihai1 commented Apr 25, 2024

Hotplug block devices on PCI segments >= 1. PCI segment 0 is used for the network interface, any disks present at Guest boot time, etc.

Just bus 0 of each segment is used, and up to 31 devices can be hotplugged to each bus.

This reverts commit 627be9b, that was
insufficient. Waiting for blk devices used just the PCI device/slot
index, but not the PCI segment/domain index.

Signed-off-by: Dan Mihai <dmihai@microsoft.com>
Initialize the CLH Platform a single time.

Signed-off-by: Dan Mihai <dmihai@microsoft.com>
@danmihai1 danmihai1 changed the title Danmihai1/hotplug7 runtime: agent: use PCI segments 1+ for blk devices Apr 25, 2024
Hotplug block devices on PCI segments >= 1. PCI segment 0 is used for
the network interface, any disks present at Guest boot time, etc.

Just bus 0 of each segment is used, and up to 31 devices can be
hotplugged to each bus.

Signed-off-by: Dan Mihai <dmihai@microsoft.com>
This pod starts successfully when using default AKS-CC settings,
and a permissive policy.

When the Kata debug options are enabled, this pod fails to start while
trying to hotplug image layer index 41. This bug is being investigated.

The genpolicy tool should also try to create a smaller policy for
this pod, because otherwise "kubectl apply" rejects the policy
annotation as being too large.

Signed-off-by: Dan Mihai <dmihai@microsoft.com>
@danmihai1 danmihai1 marked this pull request as ready for review April 30, 2024 01:05
@danmihai1 danmihai1 requested review from a team as code owners April 30, 2024 01:05
@Redent0r
Copy link

Redent0r commented Apr 30, 2024

Test run https://dev.azure.com/mariner-org/mariner/_build/results?buildId=560047&view=ms.vss-test-web.build-test-results-tab (2 failures (1 flake and one addressed by updating test sample))

Move pod-many-layers.yaml to needs_containerd_pull category

Signed-off-by: Saul Paredes <saulparedes@microsoft.com>
@Redent0r Redent0r merged commit 99f1e83 into msft-main Apr 30, 2024
42 of 54 checks passed
@Redent0r Redent0r deleted the danmihai1/hotplug7 branch April 30, 2024 17:51
@Redent0r Redent0r restored the danmihai1/hotplug7 branch April 30, 2024 17:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants