Bug #3768
closed
perl is required for logrotate, we need to include Perl as a dependency
Added by Anonymous over 11 years ago.
Updated about 11 years ago.
Description
logrotate for ceph (/etc/logrotate.d/ceph) uses perl commands
if perl is not installed, logrotate fails
if logrotate fails, the root filesystem fills up and the system fails.
So, we need to make sure that perl is set as a dependency upon ceph installation.
seeing this on a centos box, but logrotate is across all OS's, so we need to ensure that perl is available for all installs.
- Assignee set to Anonymous
- Priority changed from Normal to High
- Target version set to v0.57a
I hate to see a dependency like perl get added for a oneliner perl regex. Is this the only place perl is used? Can we swap out that line for some read/sed scriptfoo instead?
Can we check to ensure perl is not used elsewhere?
Are there guidelines that are provided to the developers that specify that perl can not be used in the packaged code?
If those two are done, changing that line to something else would be fabulous. I hate to rely on the current perl package to be auto installed since it could change to a version that is not compatible with the line of code. not that it would be likely, but it would be possible.
The same issue exists with the debian packages. We have an explicit dependency on python, but not on perl. I don't see any other use of perl in system scripts, but we do have a bunch of ceph tools and utilities written in perl. It does not appear that they are packaged for installation.
- Status changed from New to In Progress
- Project changed from CephFS to Ceph
- Target version deleted (
v0.57a)
Whoops, this was never an FS bug. :)
- Status changed from In Progress to Resolved
It had sounded to me like the trend was towards eliminating the Perl usage rather than adding it as a dependency. Did that turn out to be infeasible?
- Status changed from Resolved to In Progress
Putting back to in-progress. The preferred solution is to replace the perl filter line with sed or python and remove the dependency on perl.
- Status changed from In Progress to Fix Under Review
- Status changed from Fix Under Review to Resolved
Also available in: Atom
PDF