Skip to content

Support StatefulSet patching#4637

Closed
heylongdacoder wants to merge 3 commits intoprometheus-operator:mainfrom
heylongdacoder:feature/patch-statefulset
Closed

Support StatefulSet patching#4637
heylongdacoder wants to merge 3 commits intoprometheus-operator:mainfrom
heylongdacoder:feature/patch-statefulset

Conversation

@heylongdacoder
Copy link
Contributor

Description

Fixes #3488, Fixes #4298

Type of change

What type of changes does your code introduce to the Prometheus operator? Put an x in the box that apply.

  • CHANGE (fix or feature that would cause existing functionality to not work as expected)
  • FEATURE (non-breaking change which adds functionality)
  • BUGFIX (non-breaking change which fixes an issue)
  • ENHANCEMENT (non-breaking change which improves existing functionality)
  • NONE (if none of the other choices apply. Example, tooling, build system, CI, docs, etc.)

Changelog entry

Please put a one-line changelog entry below. This will be copied to the changelog file during the release process.

Allow user to inject arbitrary fields into Alertmanager, ThanosRuler and Prometheus StatefulSet

Prometheus StatefulSet

Fixes prometheus-operator#3488, Fixes prometheus-operator#4298

Signed-off-by: heylongdacoder <heylongdacoder@gmail.com>
Signed-off-by: heylongdacoder <heylongdacoder@gmail.com>
@heylongdacoder heylongdacoder requested a review from a team as a code owner March 15, 2022 04:36
Signed-off-by: heylongdacoder <heylongdacoder@gmail.com>
@Grounz
Copy link

Grounz commented Apr 26, 2022

Hi, what's the status for this feature ?

@heylongdacoder
Copy link
Contributor Author

heylongdacoder commented May 1, 2022

hi @Grounz, I think this PR wont be merge probably. You may check the whole discussion from this youtube video :)

@simonpasquier
Copy link
Contributor

Hey! Sorry for the very late feedback but we've discussed #4691 at the last contributors office hours and we agreed that instead of exposing a way to patch the generated statefulset, we were ok adding support for hostAlias in the Prometheus, Alertmanager and ThanosRuler CRDs.
Do you want to tackle this? If yes, it's probably best to close this PR and open a new one.

@heylongdacoder
Copy link
Contributor Author

alright, I will do it :)

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.

Add hostAlias support in Prometheus CRD Add an option to add hostAlias for alert manager

3 participants