Skip to content

feat: Allow usage of --coordinator-id=0 - #4483

Merged
as51340 merged 1 commit into
masterfrom
fix/coord-id
Jul 27, 2026
Merged

feat: Allow usage of --coordinator-id=0#4483
as51340 merged 1 commit into
masterfrom
fix/coord-id

Conversation

@as51340

@as51340 as51340 commented Jul 24, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Users will now be able to use 0 as valid coordinator id. Motivation is the future K8s operator.

@as51340 as51340 added this to the mg-v3.13.0 milestone Jul 24, 2026
@as51340
as51340 requested a review from imilinovic July 24, 2026 14:08
@as51340 as51340 self-assigned this Jul 24, 2026
@as51340 as51340 added Capability - high-availability Docs - changelog only Docs - changelog only CI -build=jepsen -test=core Run jepsen build and core tests on push CI -build=release -test=core Run release build and core tests on push CI -build=release -test=e2e Run release build and e2e tests on push labels Jul 24, 2026
@as51340

as51340 commented Jul 24, 2026

Copy link
Copy Markdown
Contributor Author

Tracking

  • [Link to Epic/Issue]

Standard development

CI Testing Labels

  • Select the appropriate CI test labels (CI -build=build-name -test=test-suite)

Documentation checklist

  • Add the documentation label
  • Add the bug / feature label
  • Add the milestone for which this feature is intended
    • If not known, set for a later milestone
  • Write a release note, including added/changed clauses
    • Users will now be able to use 0 as the valid coordinator id. #4483
    • 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

@sonarqubecloud

sonarqubecloud Bot commented Jul 24, 2026

Copy link
Copy Markdown

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@as51340
as51340 added this pull request to the merge queue Jul 27, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Jul 27, 2026
@as51340
as51340 added this pull request to the merge queue Jul 27, 2026
Merged via the queue into master with commit 2724a6c Jul 27, 2026
38 checks passed
@as51340
as51340 deleted the fix/coord-id branch July 27, 2026 09:01
@vpavicic vpavicic mentioned this pull request Jul 28, 2026
Draft
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@imilinovic imilinovic imilinovic approved these changes

Assignees

@as51340 as51340

Labels

Capability - high-availability CI -build=jepsen -test=core Run jepsen build and core tests on push CI -build=release -test=core Run release build and core tests on push CI -build=release -test=e2e Run release build and e2e tests on push Docs - changelog only Docs - changelog only

Projects

None yet

Milestone

mg-v3.13.0

Development

Successfully merging this pull request may close these issues.

2 participants

@as51340 @imilinovic