Skip to content

Conversation

machine424
Copy link
Member

@machine424 machine424 commented Jul 3, 2025

to endpointslice, endpoints, service, ingress and pod roles

after injecting the labels for endpointslice, claude-4-sonnet helped transpose the code and tests to the other roles.

fixes #9510

supersedes #13798

to endpointslice, endpoints and pod roles

after injecting the labels for endpointslice, claude-4-sonnet
helped transpose the code and tests to endpoints and pod roles

fixes prometheus#9510
supersedes prometheus#13798

Signed-off-by: machine424 <ayoubmrini424@gmail.com>
Co-authored-by: Paul BARRIE <paul.barrie.calmels@gmail.com>
@machine424
Copy link
Member Author

cc @bboreham as you reviewed #13798

@machine424
Copy link
Member Author

also cc @simonpasquier

@machine424
Copy link
Member Author

maybe @jan--f can take a look as well.

Copy link
Contributor

@jan--f jan--f left a comment

Choose a reason for hiding this comment

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

The code looks good to me, though I don't know this code well.

Any reason to not extend this to Service and Ingress roles as well?

to ingress and service roles.

with the help of claude-4-sonnet

Signed-off-by: machine424 <ayoubmrini424@gmail.com>
Signed-off-by: machine424 <ayoubmrini424@gmail.com>
@machine424
Copy link
Member Author

The code looks good to me, though I don't know this code well.

Any reason to not extend this to Service and Ingress roles as well?

Just overlooked those (they're not that used). Claude can help with that, done.

@machine424 machine424 requested a review from jan--f July 17, 2025 07:59
Copy link
Contributor

@jan--f jan--f left a comment

Choose a reason for hiding this comment

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

Thanks, I'm happy with this.

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.

Kubernetes SD: Optionally attach namespace and node metadata to targets
2 participants