Skip to content
This repository was archived by the owner on Jun 29, 2022. It is now read-only.

baremetal: let installer.service retry on failure#1490

Merged
pothos merged 1 commit intomasterfrom
kai/installer-retry
Jun 17, 2021
Merged

baremetal: let installer.service retry on failure#1490
pothos merged 1 commit intomasterfrom
kai/installer-retry

Conversation

@pothos
Copy link
Contributor

@pothos pothos commented Jun 9, 2021

In case the network only has a temporary problem, the installer service
should start again on failure.
Convert it to a oneshot service to set RemainAfterExist, so that it is
not triggered twice if anything that depends on it is pulling it in
again after it finished.

@pothos pothos requested a review from ipochi June 9, 2021 12:16
ipochi
ipochi previously approved these changes Jun 16, 2021
Copy link
Member

@ipochi ipochi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@ipochi
Copy link
Member

ipochi commented Jun 16, 2021

@pothos Please rebase, that should fix the tinkerbell CI (fix was merged)

@pothos pothos force-pushed the kai/installer-retry branch from 7e99e7f to 5d74929 Compare June 16, 2021 09:53
@pothos pothos requested a review from ipochi June 16, 2021 09:53
In case the network only has a temporary problem, the installer service
should start again on failure.
Convert it to a oneshot service to set RemainAfterExist, so that it is
not triggered twice if anything that depends on it is pulling it in
again after it finished.
@pothos pothos force-pushed the kai/installer-retry branch from 5d74929 to 1a346e6 Compare June 17, 2021 10:16
Copy link
Member

@ipochi ipochi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@pothos pothos merged commit c94916a into master Jun 17, 2021
@pothos pothos deleted the kai/installer-retry branch June 17, 2021 10:46
@invidian invidian added the kind/enhancement New feature or request label Jul 16, 2021
@invidian invidian added this to the v0.9.0 milestone Jul 16, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

kind/enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants