Skip to content

[receiver/k8sevents] shutdown instead of standby on leader election loss#44513

Merged
dmitryax merged 11 commits into
open-telemetry:mainfrom
paulojmdias:feat/k8sevents_remove_standby
Dec 11, 2025
Merged

[receiver/k8sevents] shutdown instead of standby on leader election loss#44513
dmitryax merged 11 commits into
open-telemetry:mainfrom
paulojmdias:feat/k8sevents_remove_standby

Conversation

@paulojmdias
Copy link
Copy Markdown
Member

Description

This change ensures that all components using the k8sleaderelector extension use the same behaviour of shutdown instead of standby on leader election loss.

As discussed in the following thread, this is not so beneficial and should be reverted to keep the same behaviour in all the components which use the k8sleaderelector extension.

Signed-off-by: Paulo Dias <paulodias.gm@gmail.com>
paulojmdias and others added 3 commits November 25, 2025 10:51
Signed-off-by: Paulo Dias <paulodias.gm@gmail.com>
Signed-off-by: Paulo Dias <paulodias.gm@gmail.com>
@paulojmdias paulojmdias marked this pull request as draft November 25, 2025 11:12
Signed-off-by: Paulo Dias <paulodias.gm@gmail.com>
@paulojmdias paulojmdias marked this pull request as ready for review November 25, 2025 11:21
Comment thread receiver/k8seventsreceiver/receiver.go
Signed-off-by: Paulo Dias <paulodias.gm@gmail.com>
Comment thread receiver/k8seventsreceiver/receiver.go Outdated
Copy link
Copy Markdown
Member

@ChrsMark ChrsMark left a comment

Choose a reason for hiding this comment

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

LGTM. @dmitryax please take a look too

Comment thread .chloggen/feat_k8sevents_remove_standby.yaml Outdated
Signed-off-by: Paulo Dias <paulodias.gm@gmail.com>
@paulojmdias paulojmdias added the Skip Changelog PRs that do not require a CHANGELOG.md entry label Dec 10, 2025
@dmitryax dmitryax merged commit 693e37b into open-telemetry:main Dec 11, 2025
189 checks passed
@github-actions github-actions Bot added this to the next release milestone Dec 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

receiver/k8sevents Skip Changelog PRs that do not require a CHANGELOG.md entry waiting-for-code-owners

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants