[#XX-2111] define MRTG as sipx service (sipXmrtg) - SIPfoundry Issue Tracker

sipXecs

define MRTG as sipx service (sipXmrtg)

Created: 2008-09-11 15:04 Updated: 2009-04-24 20:50 Resolved: 2008-12-05 14:01

Component/s:

None

Affects Version/s:

None

Fix Version/s:

3.11.9

Time Tracking:

Not Specified

File Attachments:

0001-XCF-2810-Add-support-for-MRTG-as-a-sipx-service-in-s.patch (38 kB) 2008-11-20 11:34 - Kevin Thorley
2.

XCF-2810-Incremental.patch (0.8 kB) 2008-11-27 05:27 - Paul McDaid

Issue Links:

Dependancy

This issue required for:

~~XX-2092~~ precommit creates a file in the source tree

XX-5235 improve configuration of MRTG service

Related

This issue related to:

XX-5286 sipx-mrtg subsystem should not use hardcoded path

Description

« Hide

MRTG service installation should not be in any way related to sipXconfig.

New service (sipXmrtg) would be better. It could install all the configuration files and scripts that are needed to make MRTG
manageable and configuratble form sipx. It should also provide a real process descriptor so that sipxsupervisor can start/stop and report on MRTG.

Sort Order:

[ Permalink | « Hide ]

Paul McDaid added a comment - 2008-11-13 09:10

This patch will fix the following JIRAs:

XCF-2810, XCF-2960 and XCF-3012

[ Permalink | « Hide ]

Kevin Thorley added a comment - 2008-11-20 11:34

I have attached a rebased and slightly modified patch that handles a couple test issues as well as some rpm packaging issues. However, there is a problem with a file (neoconf/etc/mrtg.cfg) being left after the tests run. This file does not exist in the src dir, but after running ant precommit (actually, neoconf/ant test-integation) it is left over. I was able to eliminate the file by changing the path for the mrtg.cfg file in TestUtil.java to point to outputDirectory rather than etc directory, but this breaks StatisticsPageTestUi

[ Permalink | « Hide ]

Kevin Thorley added a comment - 2008-11-26 13:00

Closed wrong issue - sorry!

[ Permalink | « Hide ]

Paul McDaid added a comment - 2008-11-27 05:27

I've found the issue that was causing the unit tests to fail when the TestUtil.java sysProps Properties were changed. Only the mrtgConfig.filename needs to be changed from the etcDirectory -> ouputDirectory as the Template mrtg-t.cfg file is read and not written.

I've added an incremental patch which has the TestUtil.java line change included.

[ Permalink | « Hide ]

Damian Krzeminski added a comment - 2008-12-05 14:01

http://sipxecs.sipfoundry.org/ViewVC/sipXecs?view=rev&rev=14187

XCF-2810 Add support for MRTG as a sipx service in sipxconfig.

Default values are set and config file is replicated

[ Permalink | « Hide ]

Rashmi Ratkal added a comment - 2008-12-17 01:53

The task has been completed verified on 3.11.9-014285 build, the sipXmrtg service is added to the servers page with status as running.
The sipxproc also show the sipXmrtg service as running.
[root@jupiter ~]# sipxproc

"sipXmrtg"=>"Running",

Therefore the issue can be closed.