Skip to content
This repository has been archived by the owner on Dec 4, 2024. It is now read-only.

chore: bump alert-manager resource limits #298

Merged
merged 3 commits into from
Jun 23, 2020
Merged

chore: bump alert-manager resource limits #298

merged 3 commits into from
Jun 23, 2020

Conversation

hectorj2f
Copy link

What type of PR is this?

Bug
What this PR does/ why we need it:

During the last MWT we found how the alert manager was killed with OOM errors. We should bump the memory limit. When the cluster is overloaded, the alertmanager is not able to handle it with the curremt 50Mi memory limit.
Which issue(s) this PR fixes:

Special notes for your reviewer:

Does this PR introduce a user-facing change?:

prometheus-alert-manager: increase memory and cpu limits due to OOM errors

Checklist

  • The commit message explains the changes and why are needed.
  • The code builds and passes lint/style checks locally.
  • The relevant subset of integration tests pass locally.
  • The core changes are covered by tests.
  • The documentation is updated where needed.

@hectorj2f hectorj2f requested a review from a team as a code owner June 17, 2020 13:34
@hectorj2f hectorj2f self-assigned this Jun 17, 2020
@hectorj2f hectorj2f merged commit 3b547b2 into master Jun 23, 2020
@hectorj2f hectorj2f deleted the hectorj2f/bump_alertmanager_limits branch June 23, 2020 10:44
gracedo added a commit that referenced this pull request Jun 23, 2020
@gracedo
Revert "chore: bump alert-manager resource limits (#298)"
d7ffc9d 
This reverts commit 3b547b2.
@gracedo gracedo mentioned this pull request Jun 23, 2020
Merged
5 tasks
gracedo added a commit that referenced this pull request Jun 23, 2020
@gracedo
Revert "chore: bump alert-manager resource limits (#298)"
fd1578b 
This reverts commit 3b547b2.
joejulian pushed a commit that referenced this pull request Jun 24, 2020
@gracedo
prometheus: Bump chart to latest version (#308)
076e777 
* Revert "chore: bump alert-manager resource limits (#298)"

This reverts commit 3b547b2.

* chore: Copy previous prometheus revision

* chore: Bump alert-manager resource limits

* chore: Copy previous prometheus revision

* chore: Bump prometheus-operator to latest chart version
joejulian pushed a commit that referenced this pull request Jul 8, 2020
@joejulian
chore: bump alert-manager resource limits (#298)
d8dbec6
joejulian pushed a commit that referenced this pull request Jul 8, 2020
@gracedo @joejulian
prometheus: Bump chart to latest version (#308)
c69187d 
* Revert "chore: bump alert-manager resource limits (#298)"

This reverts commit 3b547b2.

* chore: Copy previous prometheus revision

* chore: Bump alert-manager resource limits

* chore: Copy previous prometheus revision

* chore: Bump prometheus-operator to latest chart version
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@gracedo gracedo gracedo approved these changes

@makkes makkes makkes approved these changes

@branden branden Awaiting requested review from branden

@joejulian joejulian Awaiting requested review from joejulian

Assignees

@hectorj2f hectorj2f

Labels
ready
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@hectorj2f @makkes @gracedo