api icon indicating copy to clipboard operation
api copied to clipboard

Add DNSSettings for Tracer Configuration (API part)

Open aatarasoff opened this issue 9 months ago • 7 comments

A new section called dns_settings has been added to the Tracer Configuration. The purpose of this addition is to limit the Istio proxy's DNS queries to 10 minutes instead of the default 30 seconds when Zipkin tracing is enabled. This can be achieved by defining the below structure in MeshConfig using this PR:

dns_settings:
  refresh_rate: "600s"
  respect_ttl: false

Related PR: https://github.com/istio/istio/pull/47049

aatarasoff avatar Sep 20 '23 12:09 aatarasoff

😊 Welcome @aatarasoff! 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-policy-bot avatar Sep 20 '23 12:09 istio-policy-bot

CLA Signed

The committers listed above are authorized under a signed CLA.

  • :white_check_mark: login: aatarasoff / name: Aleksandr Tarasov (ccfc94a620d973045b8d28cd1fff046ca5763540)

Hi @aatarasoff. Thanks for your PR.

I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

istio-testing avatar Sep 20 '23 12:09 istio-testing

Does this config apply to ALL tracers ? I agree with John, it's something that otel collector should handle.

costinm avatar Sep 25 '23 20:09 costinm

I think dns refresh rate is not only to tracers, but to all DNS services. We have a DnsRefreshRate in MeshConfig now, but actually we do not allow setting respect_ttl, which is hardcoded to true.

We can add a respect_ttl to meshConfig, and allow agent to get both values

hzxuzhonghu avatar Nov 16 '23 09:11 hzxuzhonghu

Find that we have also supported dynamic bootstrap discovery, which can be used to customize the whole bootstrap config

hzxuzhonghu avatar Nov 16 '23 09:11 hzxuzhonghu

PR needs rebase.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

istio-testing avatar Jan 16 '24 22:01 istio-testing