Skip to content

Commit

Permalink
Update enhancements/machine-api/short-circuiting-backoff.md
Browse files Browse the repository at this point in the history
Signed-off-by: Michael Shitrit <[email protected]>
  • Loading branch information
mshitrit committed Jun 29, 2021
1 parent 8eb78e3 commit ba99f6c
Showing 1 changed file with 6 additions and 11 deletions.
17 changes: 6 additions & 11 deletions enhancements/machine-api/short-circuiting-backoff.md
Original file line number Diff line number Diff line change
Expand Up @@ -108,8 +108,8 @@ MachineHealthCheck:
```
### Risks and Mitigations
No known risks.
In case `FailedNodeStartupTimeout` is undefined default behaviour is preserved (i.e. remediation is not postponed).
The Pro is that naive users aren't being surprised with a new behavior however, the con is that naive users do benefit from the new behavior.

## Design Details

Expand Down Expand Up @@ -139,10 +139,11 @@ TBD

##### Removing a deprecated feature

TBD

### Upgrade / Downgrade Strategy

### Version Skew Strategy
TBD

## Implementation History

Expand All @@ -154,12 +155,6 @@ no known drawbacks

## Alternatives

- Instead of delaying, canceling the remediation for failed machines.
## Infrastructure Needed [optional]
Use this section if you need things from the project. Examples include a new
subproject, repos requested, github details, and/or testing infrastructure.
In case a machine enters the `Failed` state and does not have a NodeRef or a ProviderID do not perform remediation on that machine.

Listing these here allows the community to get the process for these resources
started right away.
This alternative is simpler since it does not require `FailedNodeStartupTimeout` however it does not allow an option to retain the system previous behaviour and relies completely on manual fix of the failed machines.

0 comments on commit ba99f6c

Please sign in to comment.