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: expose metrics port in helm charts #1119
Conversation
@microsoft-github-policy-service agree company="Microsoft" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for the PR, @Suraiya-Hameed! Could you also add this change in the helm charts?
Done, PTAL |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For the charts changes, please update the yamls in the manifest_staging/ folder, where we host the staging charts and deployment yamls. All the yaml changes will then be promoted into the released charts folder with the next release.
/azp run pr-e2e |
Azure Pipelines successfully started running 1 pipeline(s). |
Codecov Report
@@ Coverage Diff @@
## master #1119 +/- ##
=======================================
Coverage 56.39% 56.39%
=======================================
Files 10 10
Lines 1071 1071
=======================================
Hits 604 604
Misses 443 443
Partials 24 24 |
...fest_staging/charts/csi-secrets-store-provider-azure/templates/provider-azure-installer.yaml
Outdated
Show resolved
Hide resolved
ea0601a
to
14d7c97
Compare
manifest_staging/charts/csi-secrets-store-provider-azure/values.yaml
Outdated
Show resolved
Hide resolved
manifest_staging/charts/csi-secrets-store-provider-azure/values.yaml
Outdated
Show resolved
Hide resolved
manifest_staging/charts/csi-secrets-store-provider-azure/arc-values.yaml
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add the metricsAddr
configuration to the table here.
/azp run pr-e2e |
Azure Pipelines successfully started running 1 pipeline(s). |
manifest_staging/deployment/provider-azure-installer-windows.yaml
Outdated
Show resolved
Hide resolved
/azp run pr-e2e |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run pr-e2e-azure |
Azure Pipelines successfully started running 1 pipeline(s). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @Suraiya-Hameed, LGTM.
Reason for Change:
Exposing port to scrape metrics in PodMonitor.
Similar to https://github.com/kubernetes-sigs/secrets-store-csi-driver/blob/22b661d0bbaa896127726dbf9fd1c9c1e3d27230/manifest_staging/deploy/secrets-store-csi-driver.yaml#L69-L75
Requirements
Issue Fixed:
Does this change contain code from or inspired by another project?
If "Yes," did you notify that project's maintainers and provide attribution?
Special Notes for Reviewers: