alt-orchestra/talos
1
0
Fork 0
Code Issues Pull Requests Actions 29 Packages Projects Releases Wiki Activity

fix: reset inputs back to initial state in secrets.APIController

Browse Source 
This fixes a bug when after an error generating certificates controller
gets into a state of not being able to read its expected inputs
(NetworkStatus specifically).

Signed-off-by: Andrey Smirnov <andrey.smirnov@talos-systems.com>
This commit is contained in:
Andrey Smirnov 2021-09-29 19:57:51 +03:00
parent 64cb873ec4
commit 0f60ef6d38
No known key found for this signature in database
GPG Key ID: 7B26396447AB6DFD
1 changed files with 5 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
internal/app/machined/pkg/controllers/secrets/api.go
View File

@ -76,6 +76,11 @@ func (ctrl *APIController) Run(ctx context.Context, r controller.Runtime, logger
case <-r.EventCh():
}
// reset inputs back to what they were initially
if err := r.UpdateInputs(ctrl.Inputs()); err != nil {
return err
}
machineTypeRes, err := r.Get(ctx, resource.NewMetadata(config.NamespaceName, config.MachineTypeType, config.MachineTypeID, resource.VersionUndefined))
if err != nil {
if state.IsNotFoundError(err) {
@ -121,11 +126,6 @@ func (ctrl *APIController) Run(ctx context.Context, r controller.Runtime, logger
if err = ctrl.teardownAll(ctx, r); err != nil {
return err
}
// reset inputs back to what they were initially
if err = r.UpdateInputs(ctrl.Inputs()); err != nil {
return err
}
}
}