Bug 1337304 - Add verbose option to fstrim systemd service file
Summary: Add verbose option to fstrim systemd service file
Keywords:
Status: CLOSED UPSTREAM
Alias: None
Product: Fedora
Classification: Fedora
Component: util-linux
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Karel Zak
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2016-05-18 18:51 UTC by Richard Shaw
Modified: 2016-10-18 10:13 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2016-10-18 10:13:45 UTC
Type: Bug

Attachments (Terms of Use)
Add an attachment (proposed patch, testcase, etc.)

Description Richard Shaw 2016-05-18 18:51:49 UTC 
Description of problem:
It's nice to see if fstrim is actually doing anything, plus by default you get no output so all you see in the log is that systemd is "started/starting" the service. 

fstrim.service:

- ExecStart=/usr/sbin/fstrim -a
+ ExecStart=/usr/sbin/fstrim -av

After that you get nice and concise output (hostname removed):

May 18 13:44:30 systemd[1]: Starting Discard unused blocks...
May 18 13:44:30 fstrim[21611]: /var: 0 B (0 bytes) trimmed
May 18 13:44:30 fstrim[21611]: /boot: 0 B (0 bytes) trimmed
May 18 13:44:30 fstrim[21611]: /: 0 B (0 bytes) trimmed
May 18 13:44:30 systemd[1]: Started Discard unused blocks.
Comment 1 Jan Kurik 2016-07-26 04:32:22 UTC 
This bug appears to have been reported against 'rawhide' during the Fedora 25 development cycle.
Changing version to '25'.
Comment 2 Richard Shaw 2016-10-17 19:59:40 UTC 
Is there any objection to this change request?
Comment 3 Karel Zak 2016-10-18 10:13:45 UTC 
Good idea, thanks. Fixed by upstream commit 6a9aaf54915a454c21a16da94b2ed971158b416c. 

The change will be available for Fedora after upgrade to v2.29 (probably f26).
Note You need to log in before you can comment on or make changes to this bug.