Skip to content

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

Merged
elasticsearchmachine merged 1 commit intoelastic:7.17from
rjernst:shutdown/breaker_7.17
May 20, 2022
Merged

Avoid circuit breaker trips in shutdown node actions (#86047)#86994
elasticsearchmachine merged 1 commit intoelastic:7.17from
rjernst:shutdown/breaker_7.17

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 0ab2faf into elastic:7.17 May 20, 2022
@rjernst rjernst deleted the shutdown/breaker_7.17 branch May 20, 2022 18:10
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 v7.17.5

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants