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

Extend statuses #14

Merged
merged 14 commits into from
Mar 20, 2024
Merged

Extend statuses #14

merged 14 commits into from
Mar 20, 2024

Conversation

MehmedGIT
Copy link
Collaborator

Add extra Workspace and Workflow Job status values: TRANSFERRING_TO_HPC, TRANSFERRING_FROM_HPC to indicate transfer states of a Workspace or Workflow job status. This is required to lock (i.e., prevent altering) Workspaces when they are being transferred to HPC or back from from the HPC. Since transferring the workspace is a sub step of the workflow job execution, that state should also be part of the Workflow Job status.

Implemented changes:

  1. Introduced Workspace states: READY, QUEUED, RUNNING, TRANSFERRING_TO_HPC, TRANSFERRING_FROM_HPC
    READY: The workspace can be used/downloaded
    QUEUED: The workspace is queued in a workflow job
    RUNNING: The workspace is currently running in a workflow job
    TRANSFERRING_TO_HPC: The workspace is being transferred to the HPC
    TRANSFERRING_FROM_HPC: The workspace is being transferred back from the HPC

  2. Extended Workflow Job statuses with 2 more -> TRANSFERRING_TO_HPC and TRANSFERRING_FROM_HPC

  3. The Operandi server denies user requests on Workspaces that are not in a READY state.

  4. The Operandi server denies user requests on Workflow Jobs that are not in finished state (e.g., trying to download the logs or results of a workflow job that is still running inside the HPC environment, or being transfered to or back from the HPC environment.)

Moreover, the tests were improved.

@MehmedGIT MehmedGIT merged commit b214415 into main Mar 20, 2024
7 of 8 checks passed
@MehmedGIT MehmedGIT deleted the extend-statuses branch March 20, 2024 13:04
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.

1 participant