Skip to content

Replace DNS suffix when snapshot is restored in different cluster#551

Merged
dudizimber merged 11 commits intomainfrom
copilot/replace-dns-suffix-snapshot
Feb 15, 2026
Merged

Replace DNS suffix when snapshot is restored in different cluster#551
dudizimber merged 11 commits intomainfrom
copilot/replace-dns-suffix-snapshot

Conversation

Copy link
Contributor

Copilot AI commented Feb 12, 2026

When snapshots are restored across clusters with different DNS suffixes, configuration files persist the old DNS suffix, causing resolution failures.

Changes

Enhanced fix_namespace_in_config_files in all entrypoint scripts to automatically replace DNS suffixes:

  • node-entrypoint.sh: Handles node.conf
  • sentinel-entrypoint.sh: Handles sentinel.conf
  • cluster-entrypoint.sh: Handles node.conf and nodes.conf

Implementation

Regex pattern matches hostnames with multi-segment DNS suffixes and replaces the suffix while preserving the hostname:

# Pattern: ([hostname with ≥1 letter])\.(domain.segments)
sed -i -E "s/([a-zA-Z0-9_-]*[a-zA-Z][a-zA-Z0-9_-]*)\.(([a-zA-Z0-9-]+\.)+[a-zA-Z0-9-]+)/\1.${escaped_dns_suffix}/g"

Example transformation:

node-1.old-cluster.svc.cluster.local → node-1.new-cluster.svc.cluster.local

Key properties:

  • Excludes IP addresses (hostname must contain ≥1 letter)
  • RFC 1035 compliant (no underscores in domain segments, allowed in hostnames)
  • Escapes sed metacharacters (&, \, /)
  • Idempotent (safe on already-correct suffixes)

Triggered when LOCAL_DNS_SUFFIX environment variable is set, runs before server startup.

Original prompt

This section details on the original issue you should resolve

<issue_title>replace DNS suffix when snapshot is restored in different cluster</issue_title>
<issue_description></issue_description>

Comments on the Issue (you are @copilot in this section)


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI and others added 8 commits February 12, 2026 17:51
Co-authored-by: dudizimber <16744955+dudizimber@users.noreply.github.com>
Co-authored-by: dudizimber <16744955+dudizimber@users.noreply.github.com>
Co-authored-by: dudizimber <16744955+dudizimber@users.noreply.github.com>
Co-authored-by: dudizimber <16744955+dudizimber@users.noreply.github.com>
Co-authored-by: dudizimber <16744955+dudizimber@users.noreply.github.com>
Co-authored-by: dudizimber <16744955+dudizimber@users.noreply.github.com>
Co-authored-by: dudizimber <16744955+dudizimber@users.noreply.github.com>
Co-authored-by: dudizimber <16744955+dudizimber@users.noreply.github.com>
Copilot AI changed the title [WIP] Replace DNS suffix when snapshot is restored in different cluster Replace DNS suffix when snapshot is restored in different cluster Feb 12, 2026
Copilot AI requested a review from dudizimber February 12, 2026 18:05
@dudizimber dudizimber marked this pull request as ready for review February 15, 2026 09:25
@dudizimber dudizimber merged commit ca0ed9e into main Feb 15, 2026
80 of 81 checks passed
@dudizimber dudizimber deleted the copilot/replace-dns-suffix-snapshot branch February 15, 2026 11:49
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.

replace DNS suffix when snapshot is restored in different cluster

3 participants