Skip to content

Suppress redundant --server-name warning when names match - #7187

Open
Shasha-17 wants to merge 1 commit into
rethinkdb:mainfrom
Shasha-17:fix-server-name-warning
Open

Suppress redundant --server-name warning when names match#7187
Shasha-17 wants to merge 1 commit into
rethinkdb:mainfrom
Shasha-17:fix-server-name-warning

Conversation

@Shasha-17

@Shasha-17 Shasha-17 commented Jan 3, 2026

Copy link
Copy Markdown

Description

This change suppresses a redundant startup warning when the configured
--server-name matches the name already stored in server metadata.

The warning is now only emitted when the names differ.

Fixes #6922

Checklist

@clovergaze clovergaze left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Looks good to me, as in functionally the same with more checks 👍

@Shasha-17

Copy link
Copy Markdown
Author

TFTR @clovergaze
Ready to merge

@gabor-boros

Copy link
Copy Markdown
Member

@Shasha-17 in order to avoid signing Contributor Agreements, we have to make sure the commits are signed off. I restored the PR description's checklist part. Could you please make sure to sign-off the commit (git commit --amend --sign)?

Otherwise, this looks good to merge once done.

Signed-off-by: Shasha-17 <73274457+Shasha-17@users.noreply.github.com>
@Shasha-17
Shasha-17 force-pushed the fix-server-name-warning branch from b461bce to e20dabd Compare March 28, 2026 18:41
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.

Redundant warning message

3 participants