containerd-shim-lcow-v2: Fix shimManager.Stop not correctly getting the context

[Tingmao Wang (micromaomao)]

0f25a01 ("Replace OpenCensus with OpenTelemetry (#2828)") turned this

    ctx, span := oc.StartSpan(ctx, "delete")

into a

    ctx, span := ot.StartSpan(context.Background(), "delete")

because we later get a value from the context:

if opts, ok := ctx.Value(shim.OptsKey{}).(shim.Opts); ok {
	bundlePath = opts.BundlePath
}

if bundlePath == "" {
	return resp, fmt.Errorf("bundle path not found in context")
}

this change probably broke it

Assisted-by: GitHub-Copilot

[Copilot]

🟢 Approval recommended

The context propagation fix is focused, regression-tested, and has no unresolved issues.

Pull request overview

Restores context propagation in shimManager.Stop, ensuring shim options remain available after tracing begins.

Changes:

• Passes the caller context to ot.StartSpan.
• Adds regression coverage for bundle-path preservation and stop status.

File summaries

File	Description
cmd/containerd-shim-lcow-v2/manager.go	Preserves the incoming context when starting the trace span.
cmd/containerd-shim-lcow-v2/manager_test.go	Tests context propagation through Stop.

Review details

• Files reviewed: 2/2 changed files
• Comments generated: 0
• Review effort level: Balanced

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.