-
Notifications
You must be signed in to change notification settings - Fork 0
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
feat: flux alerts + optional alerting to Slack #169
Merged
Merged
Changes from 55 commits
Commits
Show all changes
63 commits
Select commit
Hold shift + click to select a range
ca60eb3
test metric push
enjenjenje 85e6951
test metric push
enjenjenje bd7ef8c
test metric push
enjenjenje 57a2c5b
test metric push
enjenjenje 167178f
lol kek
enjenjenje 5ad2ff6
lol kek
enjenjenje 71927cd
lol kek
enjenjenje 0bd12a4
lol kek
enjenjenje 7b466fc
lol kek
enjenjenje a7fe80a
lol kek
enjenjenje 3a3a802
lol kek
enjenjenje b744a22
lol kek
enjenjenje 376ce2f
lol kek
enjenjenje 34ee67d
lol kek
enjenjenje 2102036
lol kek
enjenjenje 1882137
lol kek
enjenjenje b830613
lol kek
enjenjenje 6fba225
lol kek
enjenjenje 60d86b7
lolkek
enjenjenje 7dd5a67
lolkek
enjenjenje a160909
lolkek
enjenjenje 8a09600
lolkek
enjenjenje 0ea173c
lolkek
enjenjenje 1139f60
lolkek
enjenjenje 215ad49
lolkek
enjenjenje ed23310
lolkek
enjenjenje 0ae0bd6
lolkek
enjenjenje 8ef7ea1
lolkek
enjenjenje 87b709c
lolkek
enjenjenje ee45619
lolkek
enjenjenje 5f7434b
lolkek
enjenjenje 0555146
lolkek
enjenjenje d25fb0d
lolkek
enjenjenje 8994a44
lolkek
enjenjenje 3c55933
lolkek
enjenjenje 1a6c580
lolkek
enjenjenje 300047b
lolkek
enjenjenje 06246ef
lolkek
enjenjenje a57f533
lolkek
enjenjenje bcf79da
lolkek
enjenjenje edc457a
lolkek
enjenjenje 6a6c84a
lolkek
enjenjenje 9ec8c66
lolkek
enjenjenje 2066de6
lolkek
enjenjenje e1f3ff2
lolkek
enjenjenje 5bfc541
lolkek
enjenjenje 9894bdd
lolkek
enjenjenje c7fd316
lolkek
enjenjenje 8612be4
lolkek
enjenjenje 8891ba6
lolkek
enjenjenje 01f56a2
lolkek
enjenjenje b6fae15
lolkek
enjenjenje 1e65c32
lolkek
enjenjenje bfd7ae3
lolkek
enjenjenje e44607e
lolkek
enjenjenje 5f07714
lolkek
enjenjenje a238d06
moved alerts to fluxcd config dir
enjenjenje b01448e
moved alerts to fluxcd config dir
enjenjenje bfdd7ac
moved alerts to fluxcd config dir
enjenjenje 4a34734
moved alerts to fluxcd config dir
enjenjenje d479c52
moved alerts to fluxcd config dir
enjenjenje 0e38ebc
moved alerts to fluxcd config dir
enjenjenje 357676d
moved alerts to fluxcd config dir
enjenjenje File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
Can you please add them to components/monitoring/configs/fluxcd
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.
@nahsi , done. Please have a look.
Added a file with the alerts
Plus toggled off this flag, so all
PrometheusRule
objects will immediately get parsed by Prom Operator and added to the rule-setOtherwise, we need to introduce some label selector. Which looks like an overkill