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

fix problem where wrapper unzips input files each time it restarts #5938

Merged
merged 2 commits into from
Dec 7, 2024

Conversation

davidpanderson
Copy link
Contributor

Fixes #5936

wrapper: unzip input files only first time we run
client: set up slot dir only at start of job.
This is a little tricky because of async file copy.
Deal with this by creating a 'boinc_setup_complete' file
after all files are copied.

These fixes address a problem where:

  • wrapper job starts, unzips its output files, deletes the .zip
  • job stops (e.g. client exits)
  • job restarts; client sees that .zip file isn't in slot dir,
    copies it again.
  • wrapper starts, unzips .zip file again.

The fixes address both (new client, old wrapper) and (old client, new wrapper).

client: set up slot dir only at start of job.
    This is a little tricky because of async file copy.
    Deal with this by creating a 'boinc_setup_complete' file
    after all files are copied.

These fixes address a problem where:
- wrapper job starts, unzips its output files, deletes the .zip
- job stops (e.g. client exits)
- job restarts; client sees that .zip file isn't in slot dir,
    copies it again.
- wrapper starts, unzips .zip file again.

The fixes address both (new client, old wrapper) and (old client, new wrapper).

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.

Files not reviewed (7)
  • client/app.h: Language not supported
  • client/app_control.cpp: Language not supported
  • client/app_start.cpp: Language not supported
  • client/client_state.cpp: Language not supported
  • client/client_state.h: Language not supported
  • client/main.cpp: Language not supported
  • samples/wrapper/wrapper.cpp: Language not supported
@AenBleidd AenBleidd added this to the Client/Manager 8.0.5 milestone Dec 7, 2024
@AenBleidd AenBleidd merged commit 54266b2 into master Dec 7, 2024
152 of 153 checks passed
@AenBleidd AenBleidd deleted the dpa_start branch December 7, 2024 12:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Merged
Development

Successfully merging this pull request may close these issues.

Boinc Wrapper unzips inputfiles again after resume from checkpoint?
2 participants