Skip to content

Conversation

ryanwuer
Copy link
Contributor

a change to a service with the same name but from another namespace won't enqueue the endpointSlice

@ryanwuer ryanwuer force-pushed the improve-endpointslice-discovery branch from 06e3cd4 to 9f3965b Compare April 15, 2025 14:12
ryanwuer and others added 3 commits April 15, 2025 22:28
…namespace won't enqueue the endpointSlice

Signed-off-by: Ryan Wu <rongjun0821@gmail.com>
Co-authored-by: Ayoub Mrini <ayoubmrini424@gmail.com>
Signed-off-by: Ryan Wu <rongjun0821@gmail.com>
Signed-off-by: Ryan Wu <rongjun0821@gmail.com>
@ryanwuer ryanwuer force-pushed the improve-endpointslice-discovery branch from 9f3965b to 6fc406e Compare April 15, 2025 14:28
Copy link
Member

@machine424 machine424 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks for this
lgtm

@machine424 machine424 merged commit b4d3c06 into prometheus:main Apr 16, 2025
27 checks passed
h3adex pushed a commit to h3adex/prometheus that referenced this pull request Apr 22, 2025
)

* discovery: a change to a service with the same name but from another namespace won't enqueue the endpointSlice

Signed-off-by: Ryan Wu <rongjun0821@gmail.com>

* Update discovery/kubernetes/endpointslice.go

Co-authored-by: Ayoub Mrini <ayoubmrini424@gmail.com>
Signed-off-by: Ryan Wu <rongjun0821@gmail.com>

* Update endpointslice.go

Signed-off-by: Ryan Wu <rongjun0821@gmail.com>

---------

Signed-off-by: Ryan Wu <rongjun0821@gmail.com>
Co-authored-by: Ayoub Mrini <ayoubmrini424@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants