Skip to content

maxTime flag propagates to Bacalhau Job.Spec.Timeout #636

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

Conversation

acashmoney
Copy link
Contributor

Summary

This is a sister PR to #633.

In the previous PR, the maxTime flag updates the time plex (client-side) waits for a Bacalhau job to complete. However, the Bacalhau job spec defaults to a timeout of 1800 sec (30 min).

This change connects the maxTime flag to Bacalhau's Job.Spec.Timeout value. Now Job.Spec.Timeout aligns with plex max wait time for job completion.

Example

Initialize a diffdock job, which generally takes 10+ min.

./plex init -t tools/diffdock.json -i '{"protein": ["testdata/binding/pdbbind_processed_size1/6d08/6d08_protein_processed.pdb"], "small_molecule": ["testdata/binding/pdbbind_processed_size1/6d08/6d08_ligand.sdf"]}' --scatteringMethod=crossProduct                            
Plex version (v0.10.2) up to date.
Pinned IO JSON CID: QmaAZVjACY5bS2eNnvCqYRGxAXDgL4W7WYAuDENKYs3Nbp

Set maxTime flag to 1 min.

./plex run -i QmaAZVjACY5bS2eNnvCqYRGxAXDgL4W7WYAuDENKYs3Nbp -m 1 -a test
Plex version (v0.10.2) up to date.
Created working directory:  /Users/aakaash/Desktop/code/OPENLAB/plex/jobs/c602e946-fb2c-4dea-b60a-aaf272540c9d
Initialized IO file at:  /Users/aakaash/Desktop/code/OPENLAB/plex/jobs/c602e946-fb2c-4dea-b60a-aaf272540c9d/io.json
Processing IO Entries
Starting to process IO entry 0 
Job running...
Bacalhau job id: 0b29d97f-021a-4cbd-b5ad-3bab9b807a09 
Error processing IO entry 0 
error getting Bacalhau job results: bacalhau job did not finish within the expected time (~1 min); please check the job status manually with `bacalhau describe 0b29d97f-021a-4cbd-b5ad-3bab9b807a09`
Finished processing, results written to /Users/aakaash/Desktop/code/OPENLAB/plex/jobs/c602e946-fb2c-4dea-b60a-aaf272540c9d/io.json
Completed IO JSON CID: QmXpgknt7B1XDkiU9V66PDdx5jkNXYcRdcme4NSRxFy5st

Expected behavior of a job timeout occurs.

@acashmoney acashmoney linked an issue Sep 7, 2023 that may be closed by this pull request
@linear
Copy link

linear bot commented Sep 7, 2023

@vercel
Copy link

vercel bot commented Sep 7, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
docs ⬜️ Ignored (Inspect) Visit Preview Sep 7, 2023 8:19am

@acashmoney acashmoney temporarily deployed to ci September 7, 2023 08:15 — with GitHub Actions Inactive
@acashmoney acashmoney temporarily deployed to ci September 7, 2023 08:16 — with GitHub Actions Inactive
@acashmoney acashmoney temporarily deployed to ci September 7, 2023 08:19 — with GitHub Actions Inactive
@acashmoney acashmoney merged commit 273161c into main Sep 7, 2023
@acashmoney acashmoney deleted the 635-lab-596-configure-bacalhau-job-timeout-value-equal-to-maxtime-flag branch September 7, 2023 08:23
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.

[LAB-596] configure bacalhau job timeout value equal to maxTime flag
1 participant