Skip to content

allow k8s-nameserver to read dnsrecords directly#12

Merged
thirdeyenick merged 1 commit intomainfrom
allow-direct-configmap-watch
May 14, 2025
Merged

allow k8s-nameserver to read dnsrecords directly#12
thirdeyenick merged 1 commit intomainfrom
allow-direct-configmap-watch

Conversation

@thirdeyenick
Copy link

This adds an argument for the k8s-nameserver to watch and read the dnsrecods configMap directly. As the default way of mount propagating the dnsrecords might take some time to sync when the configMap updates, reading and watching the configMap directly should reduce this time. This allows the DNS server to react quicker on tailscale proxy restarts.

@coveralls
Copy link

coveralls commented May 12, 2025

Pull Request Test Coverage Report for Build 15001012253

Details

  • 59 of 106 (55.66%) changed or added relevant lines in 2 files are covered.
  • 86 unchanged lines in 9 files lost coverage.
  • Overall coverage decreased (-0.02%) to 51.345%

Changes Missing Coverage Covered Lines Changed/Added Lines %
cmd/k8s-nameserver/main.go 38 85 44.71%
Files with Coverage Reduction New Missed Lines %
cmd/sniproxy/server.go 2 44.02%
net/dns/resolver/forwarder.go 3 74.79%
util/syspolicy/rsop/resultant_policy.go 3 94.35%
tstest/integration/testcontrol/testcontrol.go 4 83.07%
tstime/tstime.go 6 56.0%
derp/derphttp/derphttp_client.go 7 62.44%
ipn/policy/policy.go 11 0.0%
ipn/ipnlocal/local.go 19 63.77%
logtail/logtail.go 31 62.26%
Totals Coverage Status
Change from base Build 14038508949: -0.02%
Covered Lines: 56004
Relevant Lines: 109073

💛 - Coveralls

@thirdeyenick thirdeyenick force-pushed the allow-direct-configmap-watch branch from 2ef3ab3 to 938ebfa Compare May 12, 2025 11:04
@thirdeyenick thirdeyenick force-pushed the allow-direct-configmap-watch branch 3 times, most recently from e915b70 to 419f6b7 Compare May 13, 2025 15:40
This adds an argument for the k8s-nameserver to watch and read the
dnsrecods configMap directly. As the default way of mount propagating
the dnsrecords might take some time to sync when the configMap updates,
reading and watching the configMap directly should reduce this time.
@thirdeyenick thirdeyenick force-pushed the allow-direct-configmap-watch branch from 419f6b7 to d7df2bf Compare May 13, 2025 15:46
@thirdeyenick thirdeyenick merged commit 0ce0a26 into main May 14, 2025
45 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants