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 1657c1f
Showing 1 changed file with 7 additions and 18 deletions.
25 changes: 7 additions & 18 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 All @@ -125,20 +125,15 @@ The existing remediation tests will be reviewed / adapted / extended as needed.

TBD

#### Examples
TBD
##### Dev Preview -> Tech Preview
#### Dev Preview -> Tech Preview

TBD

##### Tech Preview -> GA
#### Tech Preview -> GA

TBD

##### Removing a deprecated feature
#### Removing a deprecated feature

### Upgrade / Downgrade Strategy

Expand All @@ -154,12 +149,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 1657c1f

Please sign in to comment.