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

take statefulset pod annotations from Helm values #66

Merged
merged 3 commits into from
Jul 27, 2020

Conversation

novakov-alexey-zz
Copy link
Contributor

What this PR does / why we need it:

User can setup Statefulset Pod template annotations. One of the use case is special Prometheus annotation to let it to scrape the NiFi metrics from the Statefulset Pods.

Which issue this PR fixes

Special notes for your reviewer:

@alexnuttinck

Checklist

[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

  • DCO signed
  • Chart Version bumped
  • Variables are documented in the README.md

Signed-off-by: Alexey Novakov <novakov.alex@gmail.com>
Signed-off-by: Alexey Novakov <novakov.alex@gmail.com>
Signed-off-by: Alexey Novakov <novakov.alex@gmail.com>
@novakov-alexey-zz
Copy link
Contributor Author

@alexnuttinck could you please code review this PR?

@alexnuttinck alexnuttinck self-requested a review July 27, 2020 09:01
@alexnuttinck alexnuttinck changed the base branch from master to develop July 27, 2020 09:07
@alexnuttinck
Copy link
Contributor

Thanks @novakov-alexey, it will be present in the next release.

@alexnuttinck alexnuttinck merged commit 751f020 into cetic:develop Jul 27, 2020
@novakov-alexey-zz
Copy link
Contributor Author

Cool, thank you @alexnuttinck

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[cetic/nifi] Add annotation support for processors port, to make prometheus scrape autodiscovery possible
3 participants