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

Prometheus tls path #3615

Merged
merged 8 commits into from
Mar 8, 2023
Merged

Prometheus tls path #3615

merged 8 commits into from
Mar 8, 2023

Conversation

shaun-nx
Copy link
Contributor

@shaun-nx shaun-nx commented Mar 2, 2023

Proposed changes

Update the path that prometheus TLS secrets are written to.

Checklist

Before creating a PR, run through this checklist and mark each as complete.

  • I have read the CONTRIBUTING doc
  • I have added tests that prove my fix is effective or that my feature works
  • I have checked that all unit tests pass after adding my changes
  • I have updated necessary documentation
  • I have rebased my branch onto main
  • I will ensure my PR is targeting the main branch and pulling from my branch from my own fork

@codecov-commenter
Copy link

codecov-commenter commented Mar 2, 2023

Codecov Report

Merging #3615 (0868508) into main (c26677c) will not change coverage.
The diff coverage is n/a.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@           Coverage Diff           @@
##             main    #3615   +/-   ##
=======================================
  Coverage   52.24%   52.24%           
=======================================
  Files          59       59           
  Lines       16873    16873           
=======================================
  Hits         8816     8816           
  Misses       7762     7762           
  Partials      295      295           
Impacted Files Coverage Δ
internal/configs/configurator.go 38.67% <ø> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@shaun-nx shaun-nx marked this pull request as ready for review March 7, 2023 09:15
@shaun-nx shaun-nx requested a review from a team as a code owner March 7, 2023 09:15
Copy link
Contributor

@jjngx jjngx left a comment

Choose a reason for hiding this comment

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

left comment

internal/metrics/listener.go Show resolved Hide resolved
@shaun-nx shaun-nx requested a review from jjngx March 7, 2023 11:15
internal/metrics/listener.go Outdated Show resolved Hide resolved
@shaun-nx shaun-nx requested a review from jjngx March 8, 2023 11:00
@shaun-nx shaun-nx merged commit 7e7c824 into main Mar 8, 2023
@shaun-nx shaun-nx deleted the prometheusTLSPath branch March 8, 2023 15:36
jjngx pushed a commit that referenced this pull request Mar 14, 2023
* Update path to store prometheus secrets

* Move DefaultSecretPath const to configurator and add nosec G101

* Fix lint error

* Update error check

* Update error message

* Change function name
@lucacome lucacome added the enhancement Pull requests for new features/feature enhancements label Mar 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Pull requests for new features/feature enhancements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants