-
Notifications
You must be signed in to change notification settings - Fork 1
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
Configures alertmanger for heartbeat with standalone only #63
Conversation
@John2020-cyber Image is available for testing. |
@John2020-cyber Image is available for testing. |
@John2020-cyber Image is available for testing. |
@John2020-cyber Image is available for testing. |
@John2020-cyber Image is available for testing. |
@John2020-cyber Image is available for testing. |
@John2020-cyber Image is available for testing. |
@John2020-cyber Image is available for testing. |
@John2020-cyber Image is available for testing. |
@John2020-cyber Image is available for testing. |
@John2020-cyber Image is available for testing. |
@John2020-cyber Image is available for testing. |
394362d
to
b452eaf
Compare
@John2020-cyber Image is available for testing. |
@John2020-cyber Yikes! You better fix it before anyone else finds out! Build has Failed! |
@John2020-cyber Image is available for testing. |
roles/grafana_cloud_operator/tasks/grafana_oncall_standalone.yml
Outdated
Show resolved
Hide resolved
roles/grafana_cloud_operator/tasks/grafana_oncall_standalone.yml
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There is a bit of confusion in the Readme's mermaid diagram.
According to the ansible tasks, the order of execution is Patch alertmanager-main secret
and then Adds prometheus rule
. And then finally Update CR status
. But in the diagram, there is another flow between patch to update cr status. Kindly remove that.
b0f7003
to
8d8e4d2
Compare
@John2020-cyber Image is available for testing. |
README.md
Outdated
@@ -129,7 +129,8 @@ The operator's workflow can be described in two different architectural models: | |||
GO -->|Return: Endpoint| ConfigureSlack | |||
ConfigureSlack --> ModSecret | |||
ModSecret --> PatchSecret | |||
PatchSecret --> UpdateCR | |||
PatchSecret --> AddPrometheusRule | |||
AddPrometheusRule --> CheckIntegration |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[LanguageTool] reported by reviewdog 🐶
Possible typo: you repeated a word (ENGLISH_WORD_REPEAT_RULE)
Suggestions: CheckIntegration
Rule: https://community.languagetool.org/rule/show/ENGLISH_WORD_REPEAT_RULE?lang=en-US
Category: MISC
@John2020-cyber Image is available for testing. |
@John2020-cyber Image is available for testing. |
@bnallapeta Image is available for testing. |
Adds a heartbeat for Alertmanager and Grafana OnCall. This enables us to move away from DMS