Skip to content

Avoid circuit breaker trips in shutdown node actions (#86047)#86993

Merged
elasticsearchmachine merged 2 commits intoelastic:8.2from
rjernst:shutdown/breaker_8.2
May 23, 2022
Merged

Avoid circuit breaker trips in shutdown node actions (#86047)#86993
elasticsearchmachine merged 2 commits intoelastic:8.2from
rjernst:shutdown/breaker_8.2

Conversation

@rjernst
Copy link
Copy Markdown
Member

@rjernst rjernst commented May 20, 2022

When adding or deleting node shutdown metadata, it is important that as
few things go wrong as possible. Since these actions do not do anything
with memory, there is no reason they should trip the circuit breaker,
but they might accidentally if there is pressure on the system. This
commit declares the rest and transport actions as not capable of
tripping the circuit breaker.

closes #84847

When adding or deleting node shutdown metadata, it is important that as
few things go wrong as possible. Since these actions do not do anything
with memory, there is no reason they should trip the circuit breaker,
but they might accidentally if there is pressure on the system. This
commit declares the rest and transport actions as not capable of
tripping the circuit breaker.

closes elastic#84847
@rjernst rjernst added backport auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) labels May 20, 2022
@elasticsearchmachine elasticsearchmachine merged commit 94436c6 into elastic:8.2 May 23, 2022
@rjernst rjernst deleted the shutdown/breaker_8.2 branch May 23, 2022 14:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport v8.2.2

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants