Skip to content

fix: configure ServiceMonitor TLS - #286

Merged
as51340 merged 1 commit into
mainfrom
fix/monitoring
Sep 7, 2026
Merged

fix: configure ServiceMonitor TLS#286
as51340 merged 1 commit into
mainfrom
fix/monitoring

Conversation

@as51340

@as51340 as51340 commented Sep 7, 2026

Copy link
Copy Markdown
Collaborator

Adds https option to service monitor tls.

@as51340 as51340 added this to the September 06th milestone Sep 7, 2026
@as51340 as51340 self-assigned this Sep 7, 2026
@as51340 as51340 added bug Something isn't working memgraph-ha Memgraph HA chart issue docs-needed Set if you think documentation needs to be updated because of this change labels Sep 7, 2026
@as51340

as51340 commented Sep 7, 2026 •
edited
Loading

Copy link
Copy Markdown
Collaborator Author

Tracking

  • [Link to Epic/Issue]

Standard development

  • Update / add chart templates, values, and NOTES.txt as needed
  • Run helm lint and helm template locally against the changed chart(s)
  • Verify the chart installs and upgrades cleanly on a test cluster

Labels checklist

  • Add a docs label (exactly one): docs-changelog-only, docs-needed, or docs-not-needed
  • Add at least one component label: memgraph, memgraph-ha, memgraph-lab, or infrastructure
  • Add at least one type label: bug, feature, or infrastructure
  • Assign the PR to a milestone
    • If not known, set for a later milestone

Documentation checklist

  • Write a release note, including added/changed clauses
    • Fixes the bug with ServiceMonitor that should use https for connecting to instances when bolt tls is enabled. #286
    • What has changed? What does it mean for a user? What should a user do with it? [#{{PR_number}}]({{link to the PR}})
  • Documentation PR link memgraph/documentation
    • Is back linked to this development PR

@github-actions

github-actions Bot commented Sep 7, 2026

Copy link
Copy Markdown

Release note advisory

Fixes the bug with ServiceMonitor that should use https for connecting to instances when bolt tls is enabled. #286

Format and link are correct, but minor wording/capitalization issues: use 'HTTPS' and 'Bolt TLS' and consider rephrasing 'that should use https' to 'uses HTTPS to connect to instances when Bolt TLS is enabled.'

@as51340 as51340 mentioned this pull request Sep 7, 2026
Merged
11 tasks
@as51340
as51340 merged commit b059282 into main Sep 7, 2026
5 of 10 checks passed
@as51340
as51340 deleted the fix/monitoring branch September 7, 2026 07:27
as51340 added a commit to memgraph/documentation that referenced this pull request Sep 7, 2026
@as51340
docs(ha-k8s): document ServiceMonitor scheme and tlsConfig for direct…
edb91a5 
… scraping

Covers memgraph/helm-charts#286, which adds prometheus.serviceMonitor.scheme
and prometheus.serviceMonitor.tlsConfig so Prometheus can scrape Memgraph's
OpenMetrics endpoint over HTTPS when Bolt TLS is enabled.
@as51340 as51340 mentioned this pull request Sep 7, 2026
Open
11 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@antejavor antejavor Awaiting requested review from antejavor antejavor is a code owner
@katarinasupe katarinasupe Awaiting requested review from katarinasupe katarinasupe is a code owner

Assignees

@as51340 as51340

Labels

bug Something isn't working docs-needed Set if you think documentation needs to be updated because of this change memgraph-ha Memgraph HA chart issue

Projects

None yet

Milestone

September 09th

Development

Successfully merging this pull request may close these issues.

1 participant

@as51340