Skip to content
This repository has been archived by the owner on May 12, 2021. It is now read-only.

fix ut failure due to incorrect cleanup and make it runnable with non-root #1526

Merged
merged 2 commits into from
Apr 12, 2019
Merged

fix ut failure due to incorrect cleanup and make it runnable with non-root #1526

merged 2 commits into from
Apr 12, 2019

Conversation

bergwolf
Copy link
Member

  1. fix ut failure as cleanup removes the initial test setup, as stated in ut failures due to incorrect test cleanup #1525
  2. make ut runnable with non-root

@bergwolf
ut: fix UT failure due to non-root
a0f49a9 
We should allow UT to run without root.

Signed-off-by: Peng Tao <bergwolf@hyper.sh>
@bergwolf
ut: fix UT failure due to incorrect cleanup
9040f6a 
cleanup() actually removes all things under testdir and
causes UT failures.

=== RUN   TestCreateMockSandbox
ERRO[0000] Create new sandbox failed                     error="QEMU path (/tmp/vc-tmp-007004370/hypervisor) does not exist" sandbox=7f49d00d-1995-4156-8c79-5f5ab24ce138 sandboxid=7f49d00d-1995-4156-8c79-5f5ab24ce138 source=virtcontainers subsystem=sandbox
exit status 1

Fixes: kata-containers#1525

Signed-off-by: Peng Tao <bergwolf@hyper.sh>
@teawater
Copy link
Member

/test

@devimc devimc merged commit d5a759e into kata-containers:master Apr 12, 2019
@egernst egernst mentioned this pull request Apr 16, 2019
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@amshinde amshinde amshinde approved these changes

@devimc devimc devimc approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@bergwolf @teawater @amshinde @devimc