Skip to content

CLOUDP-137882: Use different mount path for prometheus TLS secret #1155

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

Merged
merged 3 commits into from
Nov 29, 2022

Conversation

mircea-cosbuc
Copy link
Member

@mircea-cosbuc mircea-cosbuc commented Nov 24, 2022

This patch changes the prometheus TLS secret mount path to avoid the conflict with the MongoD server TLS certificate mount path.

closes #1127

All Submissions:

  • Have you opened an Issue before filing this PR?
  • Have you signed our CLA?
  • Have you checked to ensure there aren't other open Pull Requests for the same update/change?
  • Put closes #XXXX in your comment to auto-close the issue that your PR fixes (if such).

@mircea-cosbuc mircea-cosbuc force-pushed the CLOUDP-137882_prometheus-tls-mount-path branch from 8ffa394 to 58e5c70 Compare November 24, 2022 21:07
@mircea-cosbuc mircea-cosbuc marked this pull request as ready for review November 24, 2022 21:25
Copy link
Contributor

@irajdeep irajdeep left a comment

Choose a reason for hiding this comment

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

LGTM

@mircea-cosbuc mircea-cosbuc merged commit 4631a9c into master Nov 29, 2022
@mircea-cosbuc mircea-cosbuc deleted the CLOUDP-137882_prometheus-tls-mount-path branch November 29, 2022 08:29
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.

Can not enable TLS in both security and prometheus
2 participants