Skip to content

Add support for job delete operation#758

Merged
rigidit merged 9 commits into
microsoft:mainfrom
v-elegacheva:ekaterina/job-delete-support
Jul 22, 2026
Merged

Add support for job delete operation#758
rigidit merged 9 commits into
microsoft:mainfrom
v-elegacheva:ekaterina/job-delete-support

Conversation

@v-elegacheva

@v-elegacheva v-elegacheva commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

adds support for job deletion

@v-elegacheva
v-elegacheva requested a review from a team as a code owner July 13, 2026 18:47
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
There may be pipelines that require an authorized user to comment /azp run to run.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds first-class support for deleting Azure Quantum jobs through the Workspace API, and wires that operation into the Cirq integration so cirq.Job.delete() uses the delete operation instead of cancel.

Changes:

  • Add Workspace.delete_job() that calls the underlying jobs client delete operation.
  • Update azure.quantum.cirq.Job.delete() to invoke workspace.delete_job(...).
  • Add a unit test intended to validate the new delete behavior.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.

File Description
azure-quantum/azure/quantum/workspace.py Introduces delete_job() API on Workspace calling the jobs delete operation.
azure-quantum/azure/quantum/cirq/job.py Switches Cirq job deletion to use Workspace.delete_job() instead of cancellation.
azure-quantum/tests/test_workspace.py Adds a new test case covering job delete behavior.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread azure-quantum/azure/quantum/workspace.py Outdated
Comment thread azure-quantum/tests/test_workspace.py Outdated
Comment thread azure-quantum/tests/test_workspace.py Outdated
@v-elegacheva
v-elegacheva marked this pull request as draft July 13, 2026 20:45
Comment thread azure-quantum/azure/quantum/cirq/job.py
Comment thread azure-quantum/tests/test_workspace.py Outdated
Comment thread azure-quantum/azure/quantum/workspace.py Outdated
Remove outdated note about job deletion behavior.
@v-elegacheva
v-elegacheva marked this pull request as ready for review July 21, 2026 20:11
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
There may be pipelines that require an authorized user to comment /azp run to run.

@rigidit

rigidit commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 1 pipeline(s).

@v-elegacheva

Copy link
Copy Markdown
Contributor Author

@microsoft-github-policy-service agree

@v-elegacheva

Copy link
Copy Markdown
Contributor Author

@microsoft-github-policy-service agree company="Akvelon"

@rigidit rigidit self-assigned this Jul 21, 2026
Comment thread azure-quantum/tests/test_workspace.py Fixed
@rigidit
rigidit self-requested a review July 21, 2026 21:50
@rigidit
rigidit self-requested a review July 21, 2026 22:22
@v-elegacheva

Copy link
Copy Markdown
Contributor Author

/azp run

@azure-pipelines

Copy link
Copy Markdown
Commenter does not have sufficient privileges for PR 758 in repo microsoft/azure-quantum-python

@rigidit

rigidit commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 1 pipeline(s).

@rigidit
rigidit merged commit 3eb01e9 into microsoft:main Jul 22, 2026
6 checks passed
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.

4 participants