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

Add srm state operations and apis #912

Merged
merged 3 commits into from
Mar 19, 2025
Merged

Conversation

raylrui
Copy link
Contributor

@raylrui raylrui commented Mar 18, 2025

solve: #879

Add state operations in the SRM states.
State creation:
Post - /api/v1/sequence/{orcabus_id}/state/
update the comment
Patch - /api/v1/sequence/{orcabus_id}/state/{id}

currently it only support 'RESOLVED' for failed sequence run.

valid_states_map = {
        'RESOLVED': ['FAILED']
    }

@raylrui raylrui self-assigned this Mar 18, 2025
@raylrui raylrui changed the title add srm state operations and apis Add srm state operations and apis Mar 18, 2025
@raylrui raylrui marked this pull request as ready for review March 18, 2025 00:46
@raylrui raylrui requested a review from reisingerf March 18, 2025 21:56
@raylrui raylrui linked an issue Mar 19, 2025 that may be closed by this pull request
@raylrui raylrui added this pull request to the merge queue Mar 19, 2025
Merged via the queue into main with commit 2ab10bf Mar 19, 2025
6 checks passed
@raylrui raylrui deleted the feat/add-srm-resolved-state branch March 19, 2025 06:51
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.

SequenceManager: add "resolution" mechanism for "Failed" runs
2 participants