Skip to content

Fix DNS migration to handle UseDNS=no in network files#1314

Merged
dhh merged 3 commits into
basecamp:devfrom
jardahrazdera:dns-fix-clean
Aug 30, 2025
Merged

Fix DNS migration to handle UseDNS=no in network files#1314
dhh merged 3 commits into
basecamp:devfrom
jardahrazdera:dns-fix-clean

Conversation

@jardahrazdera
Copy link
Copy Markdown
Contributor

@jardahrazdera jardahrazdera commented Aug 30, 2025

Fixes #1246

Reopening the DNS fix since the previous PR #1271 got closed accidentally.

Migration 1755109182.sh only fixed resolved.conf but didn't remove UseDNS=no from network files, causing DNS to revert to Cloudflare after reboot.

This adds:

  • Fix to migration 1755109182.sh to remove UseDNS=no from network files
  • New migration 1756491748.sh for users who already ran the old version

Both now match what omarchy-setup-dns DHCP does.

Tested and confirmed it fixes the issue.

Issue basecamp#1246: Migration 1755109182.sh only fixes resolved.conf but doesn't
remove UseDNS=no from network files. When UseDNS=no persists, it blocks
DHCP DNS and forces fallback to Cloudflare after reboot.

Fix: Remove UseDNS=no from all network files, matching omarchy-setup-dns DHCP behavior.
Fixes DNS issue for users who already ran migration 1755109182.sh.
Removes UseDNS=no from network files to enable DHCP DNS.
@dhh
Copy link
Copy Markdown
Member

dhh commented Aug 30, 2025

Let's just combine the entire cleanup in the new migration. Just wanted to catch that some folks will have run the old one. But once you add a new name, you'll get everyone.

Comment thread migrations/1756491748.sh Outdated

# Migration to fix DNS issue for existing users who already ran 1755109182.sh
# Remove UseDNS=no from all network files to allow DHCP DNS
# This matches what omarchy-setup-dns DHCP does
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Remove both bash and comments. Style is just a single echo at the top. No need for other echos during progress either unless things fail.

- Revert 1755109182.sh to original state
- Consolidate all fixes in new migration 1756491748.sh
- Follow migration style guidelines
@dhh dhh merged commit 59c70fd into basecamp:dev Aug 30, 2025
hamidriaz1998 pushed a commit to hamidriaz1998/omarchy that referenced this pull request Sep 8, 2025
* Fix DNS migration to handle UseDNS=no in network files

Issue basecamp#1246: Migration 1755109182.sh only fixes resolved.conf but doesn't
remove UseDNS=no from network files. When UseDNS=no persists, it blocks
DHCP DNS and forces fallback to Cloudflare after reboot.

Fix: Remove UseDNS=no from all network files, matching omarchy-setup-dns DHCP behavior.

* Add new migration 1756491748.sh for existing users

Fixes DNS issue for users who already ran migration 1755109182.sh.
Removes UseDNS=no from network files to enable DHCP DNS.

* Apply requested changes

- Revert 1755109182.sh to original state
- Consolidate all fixes in new migration 1756491748.sh
- Follow migration style guidelines
@jardahrazdera jardahrazdera deleted the dns-fix-clean branch September 9, 2025 19:21
chriopter pushed a commit to chriopter/omarchy that referenced this pull request Nov 18, 2025
* Fix DNS migration to handle UseDNS=no in network files

Issue basecamp#1246: Migration 1755109182.sh only fixes resolved.conf but doesn't
remove UseDNS=no from network files. When UseDNS=no persists, it blocks
DHCP DNS and forces fallback to Cloudflare after reboot.

Fix: Remove UseDNS=no from all network files, matching omarchy-setup-dns DHCP behavior.

* Add new migration 1756491748.sh for existing users

Fixes DNS issue for users who already ran migration 1755109182.sh.
Removes UseDNS=no from network files to enable DHCP DNS.

* Apply requested changes

- Revert 1755109182.sh to original state
- Consolidate all fixes in new migration 1756491748.sh
- Follow migration style guidelines
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.

2 participants