Skip to content
GitLab
Projects Groups Topics Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Register
  • Sign in
  • slapos slapos
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributor statistics
    • Graph
    • Compare revisions
  • Merge requests 129
    • Merge requests 129
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Jobs
  • Commits
Collapse sidebar
  • nexedinexedi
  • slaposslapos
  • Merge requests
  • !1760

recipe/librecipe: use atomic operation to create file

  • Review changes

  • Download
  • Patches
  • Plain diff
Closed Joanne Hugé requested to merge wrapper-atomic into master Mar 10, 2025
  • Overview 8
  • Commits 2
  • Pipelines 0
  • Changes 2

At multiple occasions empty wrappers were generated when servers (ORS) were electrically unplugged, due to IO caching, this required manual intervention.

This is important for ORS, it would be good to merge soon (but not urgent). I'm not sure how to test this in a test suite, what I did is I changed slapos-testing/software.cfg and am using these test suites:

https://erp5js.nexedi.net/#/test_suite_module/2168 https://erp5js.nexedi.net/#/test_suite_module/2169

Then I'm not sure if test suite tests are enough, I can try to test inside an ORS but maybe even more testing is required.

/cc @xavier_thompson @tomo

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: wrapper-atomic
GitLab Nexedi Edition | About GitLab | About Nexedi | 沪ICP备2021021310号-2 | 沪ICP备2021021310号-7