Open wdbaruni opened 1 week ago
[!IMPORTANT]
Review skipped
Auto reviews are disabled on this repository.
Please check the settings in the CodeRabbit UI or the
.coderabbit.yaml
file in this repository. To trigger a single review, invoke the@coderabbitai review
command.You can disable this status message by setting the
reviews.review_status
tofalse
in the CodeRabbit configuration file.
not sure how TestDockerRunSuite/TestRun_BadExecutables/bad-image-bad-executable
used to pass before. It relies on node-details being printed which are off default. In the legacy print path we used to print the node details if the flag is set, or error is returned. The new print path doesn't consider errors, which is what causing that test to fail.
I've fixed that in https://github.com/bacalhau-project/bacalhau/pull/4115/commits/e00dc77c92f73eb361cdb8dba59dc3e179900175
This PR improves the visibility of queued jobs by adding a new history event, dedicated job state, and enables updating the job state from running back to queued if necessary.
This change includes:
models.JobHistory
so that events are not coupled with state updates where we can add events even if the job/execution state remains the same. This plays nicely with our intention to add more granular events for job executionIssues
This change breaks existing clients. Planning to add a middleware in a followup PR that will fail requests coming from outdated clients with a user friendly message asking them to upgrade their client.
Example Output
Old Format
New Format
New Format with Queueing
Closes #3991 Closes #4101