Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix bug: too many cloudwatch metrics #1885

Merged
merged 5 commits into from
Dec 13, 2016
Merged

Fix bug: too many cloudwatch metrics #1885

merged 5 commits into from
Dec 13, 2016

Commits on Dec 5, 2016

  1. Fix bug: too many cloudwatch metrics 

    Cloudwatch metrics were being added incorrectly. The most obvious
symptom of this was that too many metrics were being added. A simple
check against the name of the metric proved to be a sufficient fix. In
order to test the fix, a metric selection function was factored out.
    @leon-barrett
    leon-barrett committed Dec 5, 2016
    Configuration menu
    Copy the full SHA
    dc0e919 View commit details
    Browse the repository at this point in the history

  2. Go fmt cloudwatch

    @leon-barrett
    leon-barrett committed Dec 5, 2016
    Configuration menu
    Copy the full SHA
    ddeeb15 View commit details
    Browse the repository at this point in the history

  3. Cloudwatch isSelected checks metric name

    @leon-barrett
    leon-barrett committed Dec 5, 2016
    Configuration menu
    Copy the full SHA
    1800f9d View commit details
    Browse the repository at this point in the history

  4. Move cloudwatch line in changelog to 1.2 features

    @leon-barrett
    leon-barrett committed Dec 5, 2016
    Configuration menu
    Copy the full SHA
    83b8e80 View commit details
    Browse the repository at this point in the history

Commits on Dec 13, 2016

  1. Merge branch 'master' into too-many-metrics-fix

    @sparrc
    sparrc authored Dec 13, 2016
    Configuration menu
    Copy the full SHA
    6900e10 View commit details
    Browse the repository at this point in the history