Skip to content

chore: log cluster setup connection failures - #280

Merged
as51340 merged 1 commit into
mainfrom
chore/error-logging-hook
Aug 21, 2026
Merged

as51340 merged 1 commit into
mainfrom
chore/error-logging-hook

Conversation

@as51340

@as51340 as51340 commented Aug 21, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • capture Bash diagnostics from the cluster setup TCP readiness probe
  • log connection timeouts separately from immediate connection failures
  • pass the host and port as Bash arguments to avoid command interpolation

Validation

  • helm template probe-logging charts/memgraph-high-availability --namespace default
  • git diff --check

@as51340 as51340 self-assigned this Aug 21, 2026
@as51340 as51340 added memgraph-ha Memgraph HA chart issue feature New non-breaking feature docs-not-needed Docs not needed for such a change labels Aug 21, 2026
@as51340 as51340 added this to the September 06th milestone Aug 21, 2026
@as51340

as51340 commented Aug 21, 2026

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
    • 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#XXXX ]
    • Is back linked to this development PR

@as51340
as51340 merged commit 20b4061 into main Aug 21, 2026
6 of 10 checks passed
@as51340
as51340 deleted the chore/error-logging-hook branch August 21, 2026 08:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs-not-needed Docs not needed for such a change feature New non-breaking feature memgraph-ha Memgraph HA chart issue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants