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

feat(alertmanager): add PVC for silences persistance #138

Closed
wants to merge 2 commits into from

Conversation

neilamumi
Copy link

@neilamumi neilamumi commented Dec 19, 2024

Description of the changes

Alertmanager silences were lost on pod or node restarts. By adding a PVC with a default size of 10Gi, silences will now persist across restarts. This storage size can be adjusted by overriding the alertmanager_storage_size variable if we want to

Breaking change

No

@neilamumi neilamumi requested a review from a team as a code owner December 19, 2024 10:12
locals.tf Outdated
accessModes = ["ReadWriteOnce"]
resources = {
requests = {
storage = "2Gi"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would increase the size of this PVC to at least 10Gi. I know it's way over the required amount, but sadly, if we want to use Exoscale's block storage in the future, they don't allow volumes smaller than 10Gi.

If that's too big for you, then I would create a variable, set the default at 10Gi and you could then overload it on you side depending on the use cases.

@neilamumi neilamumi force-pushed the BGIS-1593-persistence-alertmanager branch from 0259d34 to 558f1af Compare December 19, 2024 13:34
@neilamumi neilamumi closed this Dec 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants