Mercurial > hg > release > thermostat-0.6
view system-backend/src/main/java/com/redhat/thermostat/backend/system/SystemBackend.java @ 997:0d9534310a17
Use bundle version in SystemBackend
I have noticed that SystemBackend is reporting 0.5.0 as its version
number. It turns out this is still using a hard-coded version string,
while the other Backends use the version of their containing bundle.
This commit changes SystemBackend to use the bundle version and includes
a test for this as well.
Reviewed-by: omajid
Review-thread: http://icedtea.classpath.org/pipermail/thermostat/2013-February/005850.html
author | Elliott Baron <ebaron@redhat.com> |
---|---|
date | Mon, 25 Feb 2013 17:49:36 -0500 |
parents | cf7a445c359b |
children |
line wrap: on
line source
/* * Copyright 2012, 2013 Red Hat, Inc. * * This file is part of Thermostat. * * Thermostat is free software; you can redistribute it and/or modify * it under the terms of the GNU General Public License as published * by the Free Software Foundation; either version 2, or (at your * option) any later version. * * Thermostat is distributed in the hope that it will be useful, but * WITHOUT ANY WARRANTY; without even the implied warranty of * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU * General Public License for more details. * * You should have received a copy of the GNU General Public License * along with Thermostat; see the file COPYING. If not see * <http://www.gnu.org/licenses/>. * * Linking this code with other modules is making a combined work * based on this code. Thus, the terms and conditions of the GNU * General Public License cover the whole combination. * * As a special exception, the copyright holders of this code give * you permission to link this code with independent modules to * produce an executable, regardless of the license terms of these * independent modules, and to copy and distribute the resulting * executable under terms of your choice, provided that you also * meet, for each linked independent module, the terms and conditions * of the license of that module. An independent module is a module * which is not derived from or based on this code. If you modify * this code, you may extend this exception to your version of the * library, but you are not obligated to do so. If you do not wish * to do so, delete this exception statement from your version. */ package com.redhat.thermostat.backend.system; import java.net.URISyntaxException; import java.util.Timer; import java.util.TimerTask; import java.util.logging.Level; import java.util.logging.Logger; import sun.jvmstat.monitor.HostIdentifier; import sun.jvmstat.monitor.MonitorException; import sun.jvmstat.monitor.MonitoredHost; import com.redhat.thermostat.backend.BaseBackend; import com.redhat.thermostat.common.Version; import com.redhat.thermostat.common.utils.LoggingUtils; import com.redhat.thermostat.storage.dao.HostInfoDAO; import com.redhat.thermostat.storage.dao.NetworkInterfaceInfoDAO; import com.redhat.thermostat.storage.dao.VmInfoDAO; import com.redhat.thermostat.storage.model.NetworkInterfaceInfo; import com.redhat.thermostat.utils.ProcDataSource; public class SystemBackend extends BaseBackend { private static final Logger logger = LoggingUtils.getLogger(SystemBackend.class); private HostInfoDAO hostInfos; private NetworkInterfaceInfoDAO networkInterfaces; private long procCheckInterval = 1000; // TODO make this configurable. private Timer timer = null; private HostIdentifier hostId = null; private MonitoredHost host = null; private JvmStatHostListener hostListener = null; private final HostInfoBuilder hostInfoBuilder; public SystemBackend(HostInfoDAO hostInfoDAO, NetworkInterfaceInfoDAO netInfoDAO, VmInfoDAO vmInfoDAO, Version version, VmStatusChangeNotifier notifier) { super("System Backend", "Gathers basic information from the system", "Red Hat, Inc.", version.getVersionNumber(), true); this.hostInfos = hostInfoDAO; this.networkInterfaces = netInfoDAO; ProcDataSource source = new ProcDataSource(); hostInfoBuilder = new HostInfoBuilder(source); hostListener = new JvmStatHostListener(vmInfoDAO, notifier); } @Override public synchronized boolean activate() { if (timer != null) { return true; } if (!getObserveNewJvm()) { logger.fine("not monitoring new vms"); } hostInfos.putHostInfo(hostInfoBuilder.build()); timer = new Timer(); timer.scheduleAtFixedRate(new TimerTask() { @Override public void run() { for (NetworkInterfaceInfo info: NetworkInfoBuilder.build()) { networkInterfaces.putNetworkInterfaceInfo(info); } } }, 0, procCheckInterval); try { hostId = new HostIdentifier((String) null); host = MonitoredHost.getMonitoredHost(hostId); host.addHostListener(hostListener); } catch (MonitorException me) { logger.log(Level.WARNING, "problems with connecting jvmstat to local machine", me); } catch (URISyntaxException use) { logger.log(Level.WARNING, "problems with connecting jvmstat to local machine", use); } return true; } @Override public synchronized boolean deactivate() { if (timer == null) { return true; } timer.cancel(); timer = null; try { host.removeHostListener(hostListener); } catch (MonitorException me) { logger.log(Level.INFO, "something went wrong in jvmstat's listening to this host"); } host = null; hostId = null; return true; } @Override public synchronized boolean isActive() { return (timer != null); } @Override public int getOrderValue() { return ORDER_DEFAULT_GROUP; } }