Opened 7 years ago
Closed 7 years ago
#28973 closed Cleanup/optimization (fixed)
Move the output of every file that collectstatic processes to verbosity level 2
Reported by: | Tanguy Nodet | Owned by: | nobody |
---|---|---|---|
Component: | contrib.staticfiles | Version: | dev |
Severity: | Normal | Keywords: | collecstatic verbosity log logging management command |
Cc: | Triage Stage: | Accepted | |
Has patch: | yes | Needs documentation: | no |
Needs tests: | no | Patch needs improvement: | no |
Easy pickings: | no | UI/UX: | no |
Description
I have a suggestion concerning the logging of the collectstatic
command.
Presently, with the default verbosity level (1), individual information on each copied or linked file are printed to sdtout
, resulting in a log looking like this:
Copying '/path/to/the/file' # or Linking '/path/to/the/file' # [...dozens if not hundreds of other lines...] 187 static files copied to '/var/www/my_site/static'.
Now providing the command is launched at every startup of the server (and that you restart your server often, for example in development), this can result in pretty heavy and "polluted" server logs.
If you use verbosity level 0, no output whatsoever is printed (obviously).
Plus, in several cases, the output for verbosity levels 1 and 2 are the same.
Looking at the collectstatic source code, only skipped operations (for post-processing, linking, copy, deletion with --clear
) and deletion (of previously copied files that got modified) are logged with a verbosity level of 2.
In my opinion, it would be nice to shift the lines concerning individual files to a verbosity level of 2, and keep the summary at a verbosity level of 1.
I think it would also bring consistency with other commands, such as loaddata
, which only displays its summary (at a verbosity level of 1):
Installed 42 object(s) from 7 fixture(s)
Cheers!
Change History (5)
follow-up: 2 comment:1 by , 7 years ago
Component: | Core (Management commands) → contrib.staticfiles |
---|
comment:2 by , 7 years ago
Replying to Tim Graham:
I don't know. Generally if you want to revisit a design decision like that, it's better to post on the DevelopersMailingList because it reaches a wider audience.
Okay, will do. Thank you for your advice.
comment:3 by , 7 years ago
Summary: | Change verbosity level of collectstatic command's outputs → Move the output of every file that collectstatic processes to verbosity level 2 |
---|---|
Triage Stage: | Unreviewed → Accepted |
Tentatively accepting as per django-developers discussion.
I don't know. Generally if you want to revisit a design decision like that, it's better to post on the DevelopersMailingList because it reaches a wider audience.