Ticket #3807 (closed defect: fixed)
SysInfo.pm - partitionsFileSystems not found
| Reported by: | innocenti_jr | Owned by: | jamor@… |
|---|---|---|---|
| Milestone: | 2.2.X | Component: | base |
| Severity: | normal | Keywords: | |
| Cc: |
Description
Zentyal Core v2.25, Cron says:
/etc/cron.daily/zentyal: Undefined subroutine &EBox::Report::DiskUsage::partitionsFileSystems called at /usr/share/perl5/EBox/SysInfo.pm line 404.
Indeed, this new function is not referenced correctly. It's implemented in FileSystem?.pm but not used in SysInfo?.pm.
Please correct this. TIA - Oliver
Attachments
Change History
comment:1 Changed 17 months ago by jamor@…
- Status changed from new to closed
- Resolution set to worksforme
comment:2 Changed 17 months ago by innocenti_jr
- Status changed from closed to reopened
- Resolution worksforme deleted
(Sigh) I mean zentyal-core 2.2.5. Could you please be not that fast in closing bugs?
Please see:
http://git.zentyal.org/zentyal.git/blob_plain/refs/heads/2.2:/main/core/ChangeLog
2.2.5
( ... )
+ Use partitionFileSystems method from EBox::FileSystem?
Commit:
http://git.zentyal.org/zentyal.git/commit/fb514764f94079fc512658e0e5bf88dd131b5278
- Cron runs /etc/cron.daily/zentyal
- This script runs gather-reportinfo and consolidate-reportinfo
- Part of the executed code tries to call EBox::Report::DiskUsage::partitionsFileSystems but this function is not defined there (but it is in FileSystem?.pm)
IMHO Line 404 from SysInfo?.pm should be like this:
my $fileSysS = EBox::FileSystem::partitionsFileSystems();
comment:3 Changed 17 months ago by jamor@…
- Status changed from reopened to closed
- Resolution set to fixed
Hello Olivier,
thanks for the information, I mistook it with a old bug. The call is fixed in [cfb5e573d825eb]
Since it is a source template file it is better you hotfix changing the line 404 directly.
Regards,
Javier
Hello Olivier,
do you mean 2.1.25?. This problem should be fixed in 2.3, please upgrade.
Regards,
Javier