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.

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

add support for Prometheus AlertManager #515

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

varuzam
Copy link

@varuzam varuzam commented Dec 10, 2020

Summary

support for Prometheus AlertManager #474

awslimitchecker \
    --alert-provider=AlertManager \
    --alert-config=endpoints=http://alertmanager1:9093,http://alertmanger2:9093 \
    --alert-config=basic_auth_username=user \
    --alert-config=basic_auth_password=pass \
    --alert-config=alert_duration=1800

Contributor License Agreement

By submitting this work for inclusion in awslimitchecker, I agree to the following terms:

  • The contribution included in this request (and any subsequent revisions or versions of it)
    is being made under the same license as the awslimitchecker project (the Affero GPL v3,
    or any subsequent version of that license if adopted by awslimitchecker).
  • My contribution may perpetually be included in and distributed with awslimitchecker; submitting
    this pull request grants a perpetual, global, unlimited license for it to be used and distributed
    under the terms of awslimitchecker's license.
  • I have the legal power and rights to agree to these terms.

@jantman
Copy link
Owner

jantman commented Apr 20, 2021

Apologies for letting this sit for so long. As is right now, this can't be merged until it has full test coverage.

@jantman jantman added the Needs-Tests PR has no or incomplete test coverage label Apr 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Needs-Tests PR has no or incomplete test coverage
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants