Skip to content

Expose dns_respect_ttl option in MeshConfig#3678

Open
leosarra wants to merge 2 commits intoistio:masterfrom
leosarra:addDNSRespectTTL
Open

Expose dns_respect_ttl option in MeshConfig#3678
leosarra wants to merge 2 commits intoistio:masterfrom
leosarra:addDNSRespectTTL

Conversation

@leosarra
Copy link
Contributor

See istio/istio#59555 for context

@istio-testing
Copy link
Collaborator

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@istio-testing istio-testing added the do-not-merge/work-in-progress Block merging of a PR because it isn't ready yet. label Mar 20, 2026
@istio-policy-bot
Copy link

😊 Welcome @leosarra! This is either your first contribution to the Istio api repo, or it's been
a while since you've been here.

You can learn more about the Istio working groups, Code of Conduct, and contribution guidelines
by referring to Contributing to Istio.

Thanks for contributing!

Courtesy of your friendly welcome wagon.

@istio-testing istio-testing added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Mar 20, 2026
@istio-testing istio-testing added size/S Denotes a PR that changes 10-29 lines, ignoring generated files. and removed size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Mar 20, 2026
@leosarra leosarra marked this pull request as ready for review March 20, 2026 15:29
@leosarra leosarra requested a review from a team as a code owner March 20, 2026 15:29
@istio-testing istio-testing removed the do-not-merge/work-in-progress Block merging of a PR because it isn't ready yet. label Mar 20, 2026
@leosarra leosarra changed the title Add dns_respect_ttl option in MeshConfig Expose dns_respect_ttl option in MeshConfig Mar 20, 2026
@ramaraochavali
Copy link
Contributor

istio/istio#16173 - context on why we switched to respect_dns_ttl. If you do this as Mesh Config property, we would reverse that issue - May be you need it configurable per SE where you have problem? BTW, why is DNS TTL so low?

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

Labels

size/S Denotes a PR that changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants