Skip to content

Rename IBM Direct Access API#90

Merged
ohtanim merged 4 commits intoqiskit-community:mainfrom
ohtanim:ibm_api_renaming
Mar 26, 2026
Merged

Rename IBM Direct Access API#90
ohtanim merged 4 commits intoqiskit-community:mainfrom
ohtanim:ibm_api_renaming

Conversation

@ohtanim
Copy link
Collaborator

@ohtanim ohtanim commented Mar 26, 2026

Description of Change

The IBM Direct Access API has been renamed to the IBM Quantum System API.
This PR addresses this API name change only and does not include any functional changes.

Important

The IBM Direct Access API has been renamed to the IBM Quantum System API.
As part of this change, the previously available resource type name has been updated from direct-access to ibm-quantum-system.
During a transition period, both direct-access and ibm-quantum-system resource type names will be supported.
After the transition period ends, support for the direct-access resource type name will be discontinued.

renamed items current name new name
QRMI resource type name direct-access ibm-quantum-system
Environment variable prefix QRMI_IBM_DA_xxx QRMI_IBM_QS_xxx

Checklist ✅

  • Have you included a description of this change?
  • Have you updated the relevant documentation to reflect this change?
  • Have you made sure CI is passing before requesting a review?

Ticket

  • Fixes #
  • Is Part of #

@ohtanim ohtanim self-assigned this Mar 26, 2026
@ohtanim ohtanim added documentation Improvements or additions to documentation CI/CD CI/CD such as Github actions related issues labels Mar 26, 2026
@ohtanim ohtanim marked this pull request as ready for review March 26, 2026 15:33
@yoonho
Copy link
Collaborator

yoonho commented Mar 26, 2026

To further "shield" users during the transition period, should we accept the old qrmi_slurm.json env vars, ie, QRMI_IBM_DA_? I did not see this support in src/ibm/quantum_system.rs.
Is it correct to have https://github.com/Qiskit/quantum-system-client as the repository in the Cargo.toml files? This is not a public repository as far as I know.

@ohtanim
Copy link
Collaborator Author

ohtanim commented Mar 26, 2026

To further "shield" users during the transition period, should we accept the old qrmi_slurm.json env vars, ie, QRMI_IBM_DA_? I did not see this support in src/ibm/quantum_system.rs.

you can find src/ibm/direct_access.rs for old settings.

Is it correct to have https://github.com/Qiskit/quantum-system-client as the repository in the Cargo.toml files? This is not a public repository as far as I know.

Please keep this as it is. Almost all of the source code are come from that repository.

@yoonho
Copy link
Collaborator

yoonho commented Mar 26, 2026

Ah, I scrolled over the src/ibm/direct_access.rs diff! Sure, we can keep the repository in Cargo.toml.

Copy link
Collaborator

@yoonho yoonho left a comment

Choose a reason for hiding this comment

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

Approved for merging

@ohtanim
Copy link
Collaborator Author

ohtanim commented Mar 26, 2026

@awennersteen @OkuyanBoga You were asked to review this PR based on the current CODEOWNERS file. As I described in our slack channel, this PR addresses the API name changes for IBM Direct Access API. I don’t think this will affect other components, but just in case, could you take a quick look if you have time?

I'm going to merge my tomorrow morning.

Thanks,

Copy link
Collaborator

@awennersteen awennersteen left a comment

Choose a reason for hiding this comment

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

As far as I can see nothing that should impact us

@ohtanim ohtanim merged commit 87bb4bd into qiskit-community:main Mar 26, 2026
8 checks passed
@ohtanim ohtanim deleted the ibm_api_renaming branch March 26, 2026 22:22
ohtanim added a commit to ohtanim/qrmi that referenced this pull request Mar 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI/CD CI/CD such as Github actions related issues documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants