Skip to content

Conversation

jaininshah9
Copy link

@jaininshah9 jaininshah9 commented Sep 17, 2025

Adding an option for podAnnotation within the prometheusExporter deployment helm template.

Since it useful to add since it helps with managing ingress/servicemesh etc

What this PR does / why we need it?

Appends the functionality for making deployment usage in environments where istio or any other service mesh is running.

PR Checklist

  • This PR adds K8s exceptions (false positives)
  • This PR adds new code
  • This PR includes tests for new/existing code
  • This PR adds docs

@codecov-commenter
Copy link

codecov-commenter commented Sep 17, 2025

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 44.02%. Comparing base (42443ec) to head (8fcccca).
⚠️ Report is 8 commits behind head on main.
❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #502   +/-   ##
=======================================
  Coverage   44.02%   44.02%           
=======================================
  Files          67       67           
  Lines        4502     4502           
=======================================
  Hits         1982     1982           
  Misses       2248     2248           
  Partials      272      272           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@jaininshah9 jaininshah9 force-pushed the add_pod_annotation_helm branch from 90ff63b to faea5c2 Compare September 18, 2025 05:24
@jaininshah9 jaininshah9 force-pushed the add_pod_annotation_helm branch from faea5c2 to 8fcccca Compare September 18, 2025 05:42
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