Skip to content
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.

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

Failed to get the dns service IP list #303

Closed
carryaimp opened this issue Nov 6, 2023 · 1 comment
Closed

Failed to get the dns service IP list #303

carryaimp opened this issue Nov 6, 2023 · 1 comment
Assignees

Comments

@carryaimp
Copy link
Contributor

kdocker agent app v0.2.0

Failed to get the dns service IP list
image

Hwo To Reproduce
In the program, the default dns service label is selected as "k8s-app": "kube-dns". In the official definition of core-dns, this label is not available, resulting in an empty IP list corresponding to the acquisition of DNS services, resulting in a subsequent index error on the list:Panic details: runtime error: index out of range [0] with length 0

The default core-dns service name is :kubernetes.io/name: "CoreDNS", official link: https://github.com/coredns/helm/blob/master/charts/coredns/templates/_helpers.tpl

Expected behavior
Correct DNS service IP address list

@ii2day
Copy link
Collaborator

ii2day commented Nov 6, 2023

@carryaimp Thank you for raising this issue. The kube-dns label does look a bit old. You are welcome to submit a PR to fix this issue.

carryaimp added a commit to carryaimp/kdoctor that referenced this issue Nov 6, 2023
carryaimp added a commit to carryaimp/kdoctor that referenced this issue Nov 6, 2023
carryaimp added a commit to carryaimp/kdoctor that referenced this issue Nov 6, 2023
weizhoublue added a commit that referenced this issue Nov 6, 2023
[#303]fix-Failed_to_get_the_dns_service_IP_list
@ii2day ii2day closed this as completed Nov 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants