Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1594)

Unified Diff: infra_libs/ts_mon/targets.py

Issue 1260293009: make version of ts_mon compatible with appengine (Closed) Base URL: https://chromium.googlesource.com/infra/infra.git@master
Patch Set: clean up code Created 5 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: infra_libs/ts_mon/targets.py
diff --git a/infra_libs/ts_mon/targets.py b/infra_libs/ts_mon/targets.py
deleted file mode 100644
index 1a50044f43008e4ca995cf6a944c3c53ee2ea789..0000000000000000000000000000000000000000
--- a/infra_libs/ts_mon/targets.py
+++ /dev/null
@@ -1,88 +0,0 @@
-# Copyright 2015 The Chromium Authors. All rights reserved.
-# Use of this source code is governed by a BSD-style license that can be
-# found in the LICENSE file.
-
-"""Classes representing the monitoring interface for tasks or devices."""
-
-
-class Target(object):
- """Abstract base class for a monitoring target.
-
- A Target is a "thing" that should be monitored, for example, a device or a
- process. The majority of the time, a single process will have only a single
- Target.
-
- Do not directly instantiate an object of this class.
- Use the concrete child classes instead:
- * TaskTarget to monitor a job or tasks running in (potentially) many places;
- * DeviceTarget to monitor a host machine that may be running a task.
- """
- def _populate_target_pb(self, metric):
- """Populate the 'target' embedded message field of a metric protobuf."""
- raise NotImplementedError()
-
-
-class DeviceTarget(Target):
- """Monitoring interface class for monitoring specific hosts or devices."""
-
- def __init__(self, region, network, hostname):
- """Create a Target object exporting info about a specific device.
-
- Args:
- region (str): physical region in which the device is located.
- network (str): virtual network on which the device is located.
- hostname (str): name by which the device self-identifies.
- """
- super(DeviceTarget, self).__init__()
- self._region = region
- self._network = network
- self._hostname = hostname
- self._realm = 'ACQ_CHROME'
- self._alertable = True
-
- def _populate_target_pb(self, metric):
- """Populate the 'network_device' embedded message of a metric protobuf.
-
- Args:
- metric (metrics_pb2.MetricsData): the metric proto to be populated.
- """
- # Note that this disregards the pop, asn, role, and vendor fields.
- metric.network_device.metro = self._region
- metric.network_device.hostgroup = self._network
- metric.network_device.hostname = self._hostname
- metric.network_device.realm = self._realm
- metric.network_device.alertable = self._alertable
-
-
-class TaskTarget(Target):
- """Monitoring interface class for monitoring active jobs or processes."""
-
- def __init__(self, service_name, job_name,
- region, hostname, task_num=0):
- """Create a Target object exporting info about a specific task.
-
- Args:
- service_name (str): service of which this task is a part.
- job_name (str): specific name of this task.
- region (str): general region in which this task is running.
- hostname (str): specific machine on which this task is running.
- task_num (int): replication id of this task.
- """
- super(TaskTarget, self).__init__()
- self._service_name = service_name
- self._job_name = job_name
- self._region = region
- self._hostname = hostname
- self._task_num = task_num
-
- def _populate_target_pb(self, metric):
- """Populate the 'task' embedded message field of a metric protobuf.
-
- Args:
- metric (metrics_pb2.MetricsData): the metric proto to be populated.
- """
- metric.task.service_name = self._service_name
- metric.task.job_name = self._job_name
- metric.task.data_center = self._region
- metric.task.host_name = self._hostname
- metric.task.task_num = self._task_num

Powered by Google App Engine
This is Rietveld 408576698