Skip to content
This repository was archived by the owner on Sep 17, 2026. It is now read-only.
This repository was archived by the owner on Sep 17, 2026. It is now read-only.

verifyNoStaleFiles misses Docker containerd task path in moby namespace #62

Description

@mdryaaan

Description

`verifyNoStaleFiles` in `tests/e2e/utils.go` checks containerd task paths under `default` and `k8s.io` but does not check the `moby` namespace. Docker
uses the `moby` namespace for containerd tasks. As a result, post-test stale file verification never catches leftover containerd task directories when tests
run with Docker, which defeats the purpose of the check.

Fixes #64

Current Behavior

Only two task paths are checked:

  • `/run/containerd/io.containerd.runtime.v2.task/default/`
  • `/run/containerd/io.containerd.runtime.v2.task/k8s.io/`

The `moby` path used by Docker is not checked. A stale directory under `/run/containerd/io.containerd.runtime.v2.task/moby/` will not be
detected and the function returns nil.

Expected Behavior

`verifyNoStaleFiles` should verify all three containerd namespaces that urunc can place task directories under: `default`, `k8s.io`, and `moby`.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions