Skip to content

Sakwa/localdns poc #7639

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Draft
saewoni wants to merge 7 commits into
base: main
Choose a base branch
from
Draft

Sakwa/localdns poc #7639

saewoni wants to merge 7 commits into from

Conversation

@saewoni
Copy link

@saewoni saewoni commented Jan 12, 2026

What type of PR is this?

What this PR does / why we need it:

Which issue(s) this PR fixes:

Fixes #

Requirements:

  • uses conventional commit messages
  • includes documentation
  • adds unit tests
  • tested upgrade from previous version
  • commits are GPG signed and Github marks them as verified

Special notes for your reviewer:

Release note:

none

saewoni added 7 commits January 8, 2026 01:12
@saewoni
initial changes
dbd36ec
@saewoni
update vhdbuilder/packer/
d143f06
@saewoni
testdata files edited bc the gzipped content have changed
363864a
@saewoni
file updates
85773ee
@saewoni
feat: add MCR DNS caching to LocalDNS for improved reliability
fccff34 
- Cache MCR IPs in /etc/hosts.testing, read by LocalDNS Corefile
- Run mcr-hosts-setup timer every 5min (was 30s) with 60s randomization
- Only enable when LocalDNS is active
- Improve error handling and regex validation
@saewoni
Merge main into sakwa/localdns_poc
ada44cf
@saewoni
Fix: Restrict MCR DNS hosts caching to root domain only
4b90487 
Wrapped the hosts /etc/hosts.testing section in isRootDomain conditional
so it only applies to .:53 VnetDNS queries, not cluster.local or custom
domains. Updated tests and regenerated testdata.
@github-actions
Copy link
Contributor

github-actions bot commented Jan 12, 2026

PR Title Lint Failed ❌

Current Title: Sakwa/localdns poc

Your PR title doesn't follow the expected format. Please update your PR title to follow one of these patterns:

Conventional Commits Format:

  • feat: add new feature - for new features
  • fix: resolve bug in component - for bug fixes
  • docs: update README - for documentation changes
  • refactor: improve code structure - for refactoring
  • test: add unit tests - for test additions
  • chore: remove dead code - for maintenance tasks
  • chore(deps): update dependencies - for updating dependencies
  • ci: update build pipeline - for CI/CD changes

Guidelines:

  • Use lowercase for the type and description
  • Keep the description concise but descriptive
  • Use imperative mood (e.g., "add" not "adds" or "added")
  • Don't end with a period

Examples:

  • ✅ feat(windows): add secure TLS bootstrapping for Windows nodes
  • ✅ fix: resolve kubelet certificate rotation issue
  • ✅ docs: update installation guide
  • ❌ Added new feature
  • ❌ Fix bug.
  • ❌ Update docs

Please update your PR title and the lint check will run again automatically.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@yewmsft yewmsft Awaiting requested review from yewmsft yewmsft will be requested when the pull request is marked ready for review yewmsft is a code owner

@YaoC YaoC Awaiting requested review from YaoC YaoC will be requested when the pull request is marked ready for review YaoC is a code owner

@juan-lee juan-lee Awaiting requested review from juan-lee juan-lee will be requested when the pull request is marked ready for review juan-lee is a code owner

@cameronmeissner cameronmeissner Awaiting requested review from cameronmeissner cameronmeissner will be requested when the pull request is marked ready for review cameronmeissner is a code owner

@ganeshkumarashok ganeshkumarashok Awaiting requested review from ganeshkumarashok ganeshkumarashok will be requested when the pull request is marked ready for review ganeshkumarashok is a code owner

@Devinwong Devinwong Awaiting requested review from Devinwong Devinwong will be requested when the pull request is marked ready for review Devinwong is a code owner

@lilypan26 lilypan26 Awaiting requested review from lilypan26 lilypan26 will be requested when the pull request is marked ready for review lilypan26 is a code owner

@AbelHu AbelHu Awaiting requested review from AbelHu AbelHu will be requested when the pull request is marked ready for review AbelHu is a code owner

@junjiezhang1997 junjiezhang1997 Awaiting requested review from junjiezhang1997 junjiezhang1997 will be requested when the pull request is marked ready for review junjiezhang1997 is a code owner

@djsly djsly Awaiting requested review from djsly djsly will be requested when the pull request is marked ready for review djsly is a code owner

@phealy phealy Awaiting requested review from phealy phealy will be requested when the pull request is marked ready for review phealy is a code owner

@r2k1 r2k1 Awaiting requested review from r2k1 r2k1 will be requested when the pull request is marked ready for review r2k1 is a code owner

@timmy-wright timmy-wright Awaiting requested review from timmy-wright timmy-wright will be requested when the pull request is marked ready for review timmy-wright is a code owner

@zachary-bailey zachary-bailey Awaiting requested review from zachary-bailey zachary-bailey will be requested when the pull request is marked ready for review zachary-bailey is a code owner

@awesomenix awesomenix Awaiting requested review from awesomenix awesomenix will be requested when the pull request is marked ready for review awesomenix is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@saewoni