Bug 598794

Summary: Enable parametrization of sadc arguments
Product: Red Hat Enterprise Linux 5 Reporter: Kevin Graham <kgraham>
Component: sysstatAssignee: Ivana Varekova <varekova>
Status: CLOSED ERRATA QA Contact: BaseOS QE - Apps <qe-baseos-apps>
Severity: low Docs Contact:
Priority: low    
Version: 5.5CC: bnater, mpoole, pat.kartas, rvokal, tao
Target Milestone: rc   
Target Release: ---   
Hardware: All   
OS: All   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Previously, there was no easy way to enable collection of statistics about disks and interrupts. Now, the SADC_OPTIONS variable can be used to set parameters for the sads utility, fixing this bug.
Story Points: ---
Clone Of:
: 661724 690400 (view as bug list) Environment:
Last Closed: 2011-07-21 10:42:51 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 661724, 690400    
Attachments:
Description Flags
patch to warn disk stats will not be gathered
none
patch to source /etc/sysconfig/sysstat in sa1 & init script none

Description Kevin Graham 2010-06-02 03:45:15 UTC
Using the stock sysstat, the only apparent way to enable collection of disk and interrupt statistics is to either squeeze it into the command line in cron.d/sysstat and/or not using the stock scripts at all.

There's plenty of ways to solve this, but the quickest and lowest impact presumably would be for sa1 to pick up an 'SADC_OPTIONS' value from /etc/sysconfig/sysstat (which sa2 is already using), ie:

shell1:/ $ diff /usr/lib64/sa/sa1.orig /usr/lib64/sa/sa1
4a5
> [ -r /etc/sysconfig/sysstat ] && . /etc/sysconfig/sysstat
13c14
<       exec ${ENDIR}/sadc -F -L 1 1 -
---
>       exec ${ENDIR}/sadc -F -L ${SADC_OPTIONS} 1 1 -
15c16
<       exec ${ENDIR}/sadc -F -L $* -
---
>       exec ${ENDIR}/sadc -F -L ${SADC_OPTIONS} $* -
shell1:/ $

Comment 1 Martin Poole 2010-08-12 16:32:48 UTC
The sourcing /etc/sysconfig/sysstat should also be performed in /etc/rc.d/init.d/sysstat

Comment 2 Martin Poole 2010-08-12 16:43:15 UTC
sadc, which is the underlying data gather command, only pays attention to the -d flag if creating a new file, and not when updating an existing file or rotating an old file to start the next day's file (see comments in /usr/lib/sa/sa1)

since a system administrator might not realise this is the case and expected simply changing the config file to alter behaviour it would be sensible to emit a warning if the disk stats flag is set when the data files are not going to honour it.

Comment 3 Martin Poole 2010-08-12 16:44:29 UTC
Created attachment 438481 [details]
patch to warn disk stats will not be gathered

Comment 4 Martin Poole 2010-08-12 16:58:05 UTC
Created attachment 438485 [details]
patch to source /etc/sysconfig/sysstat in sa1 & init script

Comment 7 RHEL Program Management 2011-01-11 20:00:20 UTC
This request was evaluated by Red Hat Product Management for
inclusion in the current release of Red Hat Enterprise Linux.
Because the affected component is not scheduled to be updated in the
current release, Red Hat is unfortunately unable to address this
request at this time. Red Hat invites you to ask your support
representative to propose this request, if appropriate and relevant,
in the next release of Red Hat Enterprise Linux.

Comment 8 RHEL Program Management 2011-01-12 15:12:53 UTC
This request was erroneously denied for the current release of
Red Hat Enterprise Linux.  The error has been fixed and this
request has been re-proposed for the current release.

Comment 9 Ivana Varekova 2011-01-24 07:02:50 UTC
*** Bug 586217 has been marked as a duplicate of this bug. ***

Comment 11 Branislav NĂ¡ter 2011-05-27 11:48:39 UTC
Bugfix was verified on sysstat-7.0.2-8.el5 package on all supported architectures.

It's now possible to set sadc parameters using SADC_OPTIONS variable in /etc/sysconfig/sysstat file.

Comment 12 Tomas Capek 2011-06-29 08:30:13 UTC
    Technical note added. If any revisions are required, please edit the "Technical Notes" field
    accordingly. All revisions will be proofread by the Engineering Content Services team.
    
    New Contents:
Previously, there was no easy way to enable collection of statistics about disks and interrupts. Now, the SADC_OPTIONS variable can be used to set parameters for the sads utility, fixing this bug.

Comment 13 errata-xmlrpc 2011-07-21 10:42:51 UTC
An advisory has been issued which should help the problem
described in this bug report. This report is therefore being
closed with a resolution of ERRATA. For more information
on therefore solution and/or where to find the updated files,
please follow the link below. You may reopen this bug report
if the solution does not work for you.

http://rhn.redhat.com/errata/RHSA-2011-1005.html