Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

auto-provisioning: improve behavior with power cuts (master) #226

Merged
merged 1 commit into from
Feb 6, 2025

Conversation

jsrc27
Copy link
Collaborator

@jsrc27 jsrc27 commented Feb 5, 2025

Some users when auto-provisioning, power down the system once they see the auto-provisioning service exit successfully. However, in some cases this can cause the files created from the provisioning process to not be fully written to disk, resulting in 0-byte files upon next boot.

Therefore, add some explicit syncs to improve this behavior. Also make use of a temp directory to improve the overall atomicity of the process.

Related-to: TOR-3743

Same as: #217 but for master

Some users when auto-provisioning, power down the system once they see
the auto-provisioning service exit successfully. However, in some cases
this can cause the files created from the provisioning process to not be
fully written to disk, resulting in 0-byte files upon next boot.

Therefore, add some explicit syncs to improve this behavior. Also make use
of a temp directory to improve the overall atomicity of the process.

Related-to: TOR-3743

Signed-off-by: Jeremias Cordoba <[email protected]>
@jsrc27 jsrc27 changed the title auto-provisioning: improve behavior with power cuts auto-provisioning: improve behavior with power cuts (master) Feb 5, 2025
Copy link
Contributor

@rborn-tx rborn-tx left a comment

Choose a reason for hiding this comment

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

LGTM

@rborn-tx rborn-tx merged commit cd46a09 into torizon:master Feb 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants