[Commotion-admin] [mlab-observatory] Adding sample checking module, Telescope data parser (#29)

[Collin Anderson]

> +        Currently the only supported value for the 'metric' key is
> +        'download_throughput'.
> +
> +      results: (list) A list of (datetime, value) pairs representing Telescope
> +        results for the given metadata.
> +
> +    Raises:
> +      ValueError: If the metadata parameter does not specify
> +        'download_throughput' as the value of the 'metric' key.
> +    """
> +    if metadata['metric_name'] != 'download_throughput':
> +      raise ValueError('Results must be for download_throughput metric.')
> +
> +    counts_key = self._hash_key_from_metadata(metadata)
> +
> +    try:

why use try/except here instead of has_key()?

---
Reply to this email directly or view it on GitHub:
https://github.com/opentechinstitute/mlab-observatory/pull/29/files#r21558244
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.chambana.net/pipermail/commotion-admin/attachments/20141209/86807490/attachment.html>