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

Only set monitoring_role_arn if monitoring_interval > 0 #32

Merged
merged 4 commits into from
Feb 17, 2019
Merged

Only set monitoring_role_arn if monitoring_interval > 0 #32

merged 4 commits into from
Feb 17, 2019

Commits on Feb 15, 2019

  1. Only set monitoring_role_arn if monitoring_interval > 0 

    * Update main.tf
* Add note about enhanced monitoring support to README.md
    @rbreslow
    rbreslow committed Feb 15, 2019
    Configuration menu
    Copy the full SHA
    be1d339 View commit details
    Browse the repository at this point in the history

Commits on Feb 17, 2019

  1. Incorporate Hector's README comments

    @rbreslow
    rbreslow committed Feb 17, 2019
    Configuration menu
    Copy the full SHA
    f906cc1 View commit details
    Browse the repository at this point in the history

  2. fixup! Incorporate Hector's README comments

    @rbreslow
    rbreslow committed Feb 17, 2019
    Configuration menu
    Copy the full SHA
    0b7062b View commit details
    Browse the repository at this point in the history

  3. Update README.md 

    Co-Authored-By: rbreslow <rbreslow@azavea.com>
    @hectcastro @rbreslow
    hectcastro and rbreslow authored Feb 17, 2019
    Configuration menu
    Copy the full SHA
    dcfc8bd View commit details
    Browse the repository at this point in the history