[WIP][DO NOT MERGE] test vendoring for https://github.com/containers/container-libs/pull/734#6832
[WIP][DO NOT MERGE] test vendoring for https://github.com/containers/container-libs/pull/734#6832giuseppe wants to merge 2 commits into
Conversation
|
the failure seems related to the change, need to investigate it |
and bump container-libs/common too Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
| done | ||
| } | ||
|
|
||
| @test "idmapping-non-contiguous-maps" { |
There was a problem hiding this comment.
unless I a missing something that test passes on main so this is not a regression test of the bug reported?!
There was a problem hiding this comment.
are you using overlay as storage driver?
There was a problem hiding this comment.
or at least I think I do not sure if the test overwrite anything, I just do sudo bats -f "idmapping-non-contiguous-maps" tests/namespaces.bats with a buildah build on main not of this branch
There was a problem hiding this comment.
you need to set STORAGE_DRIVER=overlay, can you try the following command?
sudo STORAGE_DRIVER=overlay bats -f "idmapping-non-contiguous-maps" tests/namespaces.bats
There was a problem hiding this comment.
ah yes that fails, sorry I was not aware that vfs is the test default locally.
Error: building at STEP "RUN useradd -u 579841 -g 67283 -s /bin/bash -m crunbuild": ensuring mount targets for container "28c00debdd6d34ea47a9652df9342ea94aee72f7a786d936fe558c145d7a843c": copier: ensure: error checking item "dev/mqueue": mkdir /dev/mqueue: permission denied
I assume that is the expected error before your patch?
There was a problem hiding this comment.
yes, that is the error I am seeing too
What type of PR is this?
What this PR does / why we need it:
How to verify it
Which issue(s) this PR fixes:
Special notes for your reviewer:
Does this PR introduce a user-facing change?