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

feat: Add endpoint for updating job info. #59

Merged
merged 4 commits into from
Nov 19, 2024
Merged

Conversation

katsujukou
Copy link
Contributor

πŸ“ƒ Ticket

#273 (internal)

✍ Description

This PR introduces new Provider API endpoint for updating quantum job info.
With this feature, quantum devices can update the result or reason of failure of their jobs.

πŸ“Έ Test Result

tests/oqtopus_cloud/provider/routers/test_devices.py::test_update_device_pending_jobs PASSED [ 11%]
tests/oqtopus_cloud/provider/routers/test_devices.py::test_update_device_status_available PASSED [ 22%]
tests/oqtopus_cloud/provider/routers/test_devices.py::test_update_device_status_not_available PASSED [ 33%]
tests/oqtopus_cloud/provider/routers/test_jobs.py::test_get_jobs PASSED [ 44%]
tests/oqtopus_cloud/provider/routers/test_jobs.py::test_get_job PASSED [ 55%]
tests/oqtopus_cloud/provider/routers/test_jobs.py::test_update_job PASSED [ 66%]
tests/oqtopus_cloud/provider/routers/test_jobs.py::test_update_job_info_400 PASSED [ 77%]
tests/oqtopus_cloud/provider/routers/test_jobs.py::test_update_job_info_result PASSED [ 88%]
/home/runner/work/oqtopus-cloud/oqtopus-cloud/.venv/lib/python3.12/site-packages/coverage/inorout.py:505: CoverageWarning: Module backend/api was never imported. (module-not-imported)
self.warn(f"Module {pkg} was never imported.", slug="module-not-imported")
tests/oqtopus_cloud/provider/routers/test_jobs.py::test_update_job_info_reason PASSED [100%]

πŸ”— Related PRs

 β””  Endpoint for Updating job info and status.
@github-actions github-actions bot added the feature Adding a new feature label Nov 18, 2024
Copy link
Collaborator

@orangekame3 orangekame3 left a comment

Choose a reason for hiding this comment

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

@katsujukou
Thanks, it may be needed make docs to reflect your change for documentaion.

Thank you.

@katsujukou
Copy link
Contributor Author

Thank you for review. I appended commit of updating docs.

Copy link
Contributor

@shgokita shgokita left a comment

Choose a reason for hiding this comment

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

LGMT!

@shgokita shgokita merged commit 3dca242 into develop Nov 19, 2024
1 check passed
@shgokita shgokita deleted the feat#273/job-result branch November 19, 2024 04:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Adding a new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants