Bug 1400480 - Install tmpfiles.d snippet that prevents removal of /tmp/beakerlib-* and /var/tmp/beakerlib-* files
Summary: Install tmpfiles.d snippet that prevents removal of /tmp/beakerlib-* and /var...
Keywords:
Status: CLOSED NOTABUG
Alias: None
Product: Fedora
Classification: Fedora
Component: beakerlib
Version: rawhide
Hardware: Unspecified
OS: Unspecified
medium
unspecified
Target Milestone: ---
Assignee: Dalibor Pospíšil
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2016-12-01 10:42 UTC by Michal Sekletar
Modified: 2017-01-03 14:27 UTC (History)
5 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2017-01-03 14:27:58 UTC
Type: Bug


Attachments (Terms of Use)

Description Michal Sekletar 2016-12-01 10:42:37 UTC
Description of problem:
If some test sets system time to the future date (time has to jump for more than 30 days to the future) then it may happen that systemd-tmpfiles service will delete file backups or other important beakerlib files from /var/tmp/ or /tmp.

Version-Release number of selected component (if applicable):
beakerlib-1.11-2.fc24

How reproducible:
always

Steps to Reproduce:
1. Run beakerlib test
2. Change time in test to the future
3. Run systemctl restart systemd-tmpfiles-clean.service

Actual results:
Observe that some files needed by beakerlib stored in /tmp or /var/tmp got removed

Expected results:
No files used by beakerlib are removed

Additional info:
beakerlib rpm package should ship tmpfiles.d snippet like,

/usr/lib/tmpfiles.d/beakerlib.conf

x /tmp/beakerlib-*
x /var/tmp/beakerlib-*

Comment 1 Dalibor Pospíšil 2016-12-02 08:14:31 UTC
Good point, we can definitely implement this.

Comment 4 Dalibor Pospíšil 2017-01-03 14:27:58 UTC
As I'm looking at the code I can see that we actually ship that file.

# cat /usr/lib/tmpfiles.d/beakerlib.conf 
x /var/tmp/beakerlib-*

I think it should be enough regarding to backed up files. And the /tmp directory should not be critical if it is removed in the middle of the test.

I'm considering this ans not a bug. If you're sure the problem is really in beakerlib, feel free to reopen this bug.


Note You need to log in before you can comment on or make changes to this bug.