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

test failure: mkdir.exp #3690

Closed
reinerh opened this issue Oct 24, 2020 · 3 comments
Closed

test failure: mkdir.exp #3690

reinerh opened this issue Oct 24, 2020 · 3 comments
Labels
bug Something isn't working
Milestone

Comments

@reinerh
Copy link
Collaborator

reinerh commented Oct 24, 2020

$ cd test/fs/
$ ./mkdir.exp
spawn /bin/bash
firejail --profile=mkdir.profile find ~/.firejail_test
  $ firejail --profile=mkdir.profile find ~/.firejail_test
Reading profile mkdir.profile
Warning: cannot create user directory
TESTING ERROR 1.2

The test fails because of the warning "cannot create user directory".
I'm not sure if this is expected and can be ignored (so the test needs to be adapted) or an actual issue.

@reinerh reinerh added the bug Something isn't working label Oct 24, 2020
@netblue30 netblue30 added this to the 0.9.65 milestone Oct 25, 2020
netblue30 pushed a commit that referenced this issue Nov 1, 2020
@netblue30
Copy link
Owner

I think we can ignore them, if something goes wrong the test will catch it anyway. Try it out.

@netblue30 netblue30 added the in testing A bugfix that is being tested label Nov 1, 2020
@reinerh
Copy link
Collaborator Author

reinerh commented Nov 1, 2020

It's still failing, but at a later point. See log in #3719.

TESTING: recursive mkdir (test/fs/mkdir.exp)
spawn /bin/bash
rm -fr ~/.firejail_test
runner@fv-az108-477:~/work/firejail/firejail/test/fs$ rm -fr ~/.firejail_test
runner@fv-az108-477:~/work/firejail/firejail/test/fs$ 
<ejail --profile=mkdir.profile find ~/.firejail_test  ��
Reading profile mkdir.profile
Warning: cannot create 1001 directory
Parent pid 5169, child pid 5176
Warning: cannot create 1001 directory
�]0;firejail find /home/runner/.firejail_test �Child process initialized in 13.30 ms
/home/runner/.firejail_test
/home/runner/.firejail_test/a
/home/runner/.firejail_test/a/b
/home/runner/.firejail_test/a/b/c
/home/runner/.firejail_test/a/b/c/d.txt
rm -rf ~/.firejail_test
Warning: removing 24 bytes from stdin

Parent is shutting down, bye...
runner@fv-az108-477:~/work/firejail/firejail/test/fs$ 
<il --profile=mkdir.profile find /tmp/.firejail_test  ��
Reading profile mkdir.profile
Warning: cannot create 1001 directory
Parent pid 5181, child pid 5184
Warning: cannot create 1001 directory
�]0;firejail find /tmp/.firejail_test �Child process initialized in 10.48 ms
/tmp/.firejail_test
/tmp/.firejail_test/a
/tmp/.firejail_test/a/b
/tmp/.firejail_test/a/b/c
/tmp/.firejail_test/a/b/c/d.txt
rm -rf /tmp/.firejail_test
Warning: removing 27 bytes from stdin

Parent is shutting down, bye...
runner@fv-az108-477:~/work/firejail/firejail/test/fs$ 
<le=mkdir.profile find /run/user/1001/.firejail_test  ��
Reading profile mkdir.profile
Warning: cannot create 1001 directory
Parent pid 5191, child pid 5194
Warning: cannot create 1001 directory
�]0;firejail find /run/user/1001/.firejail_test �Child process initialized in 11.46 ms
find: '/run/user/1001/.firejail_test': No such file or directory

Parent is shutting down, bye...
runner@fv-az108-477:~/work/firejail/firejail/test/fs$ TESTING ERROR 3.1

@reinerh reinerh removed the in testing A bugfix that is being tested label Nov 1, 2020
@netblue30
Copy link
Owner

All set. I also added some of the profile tests.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants