-
Notifications
You must be signed in to change notification settings - Fork 107
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
Wait for NFS file synchronization #7927
Conversation
68e9f2b
to
3e4ef61
Compare
6f62342
to
e3c98e6
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #7927 +/- ##
==========================================
+ Coverage 86.51% 86.57% +0.06%
==========================================
Files 383 383
Lines 23742 23830 +88
Branches 629 635 +6
==========================================
+ Hits 20540 20632 +92
+ Misses 3126 3125 -1
+ Partials 76 73 -3 ☔ View full report in Codecov by Sentry. |
eae7f66
to
1afef4a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just some initial comments to simplify reading to make it easier to look for errors.
9a5a685
to
029556f
Compare
5513084
to
a1543c6
Compare
Looks like one of the tests are flaky:
https://github.com/equinor/ert/actions/runs/9414224549/job/25932738059?pr=7927 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good now!
Ensure that the files generated by a job match their expected checksums. It waits for the job's runpath to appear in the scheduler's checksum dictionary, verifies the checksums of the files, and logs any errors or discrepancies.
Issue
Resolves #7788
Resolves #7715
When applicable