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

Multiple PTR records being returned for a Pod backed by multiple Services #124418

Closed
adrianmoisey opened this issue Apr 20, 2024 · 7 comments
Closed
Assignees
Labels
kind/bug Categorizes issue or PR as related to a bug. sig/network Categorizes an issue or PR as relevant to SIG Network. triage/accepted Indicates an issue or PR is ready to be actively worked on.

Comments

@adrianmoisey
Copy link

What happened?

In the scenario where a Pod has multiple Services pointing at it, multiple PTR records are also created.

What did you expect to happen?

A single PTR record to be created.

How can we reproduce it (as minimally and precisely as possible)?

  1. Create a Deployment with at least 1 replica in it
  2. Create a service that matches the labels of that Deployment
  3. Create a second service that matches the labels of that Deployment
  4. Query the PTR record for the pod's IP address

Anything else we need to know?

This was discovered in trying to understand the behaviour in #124207
There is more context here: https://kubernetes.slack.com/archives/C09QYUH5W/p1713551070931899?thread_ts=1713088975.540039&cid=C09QYUH5W

Kubernetes version

$ kubectl version
Client Version: v1.30.0
Kustomize Version: v5.0.4-0.20230601165947-6ce0bf390ce3
Server Version: v1.29.2```

</details>


### Cloud provider

<details>
Discovered using kind
Reproduced on GKE
</details>


### OS version

<details>

```console
# On Linux:
$ cat /etc/os-release
# paste output here
$ uname -a
# paste output here

# On Windows:
C:\> wmic os get Caption, Version, BuildNumber, OSArchitecture
# paste output here

Install tools

Container runtime (CRI) and version (if applicable)

Related plugins (CNI, CSI, ...) and versions (if applicable)

@adrianmoisey adrianmoisey added the kind/bug Categorizes issue or PR as related to a bug. label Apr 20, 2024
@k8s-ci-robot k8s-ci-robot added the needs-sig Indicates an issue or PR lacks a `sig/foo` label and requires one. label Apr 20, 2024
@k8s-ci-robot
Copy link
Contributor

This issue is currently awaiting triage.

If a SIG or subproject determines this is a relevant issue, they will accept it by applying the triage/accepted label and provide further guidance.

The triage/accepted label can be added by org members by writing /triage accepted in a comment.

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.

@k8s-ci-robot k8s-ci-robot added the needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. label Apr 20, 2024
@neolit123
Copy link
Member

/sig network

@k8s-ci-robot k8s-ci-robot added sig/network Categorizes an issue or PR as relevant to SIG Network. and removed needs-sig Indicates an issue or PR lacks a `sig/foo` label and requires one. labels Apr 20, 2024
@thockin thockin self-assigned this Apr 25, 2024
@thockin thockin added triage/accepted Indicates an issue or PR is ready to be actively worked on. and removed needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. labels Apr 25, 2024
@adrianmoisey
Copy link
Author

Depends on coredns/coredns#6623

@thockin
Copy link
Member

thockin commented May 3, 2024

For bookkeeping I am going to close this, and let coredns/coredns#6623 be the final word. OK with that?

@adrianmoisey
Copy link
Author

Works for me!

@adrianmoisey
Copy link
Author

/close

@k8s-ci-robot
Copy link
Contributor

@adrianmoisey: Closing this issue.

In response to this:

/close

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Categorizes issue or PR as related to a bug. sig/network Categorizes an issue or PR as relevant to SIG Network. triage/accepted Indicates an issue or PR is ready to be actively worked on.
Projects
None yet
Development

No branches or pull requests

4 participants