Skip to content

[Quantum] Add "Microsoft.AzureQuantum-" prefix to deployment name#4657

Merged
necusjz merged 4 commits intoAzure:mainfrom
warren-jones:use-quantum-prefix-on-deployment-name
Apr 20, 2022
Merged

[Quantum] Add "Microsoft.AzureQuantum-" prefix to deployment name#4657
necusjz merged 4 commits intoAzure:mainfrom
warren-jones:use-quantum-prefix-on-deployment-name

Conversation

@warren-jones
Copy link
Contributor

To follow our telemetry naming convention and standardize queries, the ARM deployment name should begin with “Microsoft.AzureQuantum-“ when creating a quantum workspace.


This checklist is used to make sure that common guidelines for a pull request are followed.

  • Have you run azdev style <YOUR_EXT> locally? (pip install azdev required)
  • Have you run python scripts/ci/test_index.py -q locally?

@ghost ghost added the Auto-Assign Auto assign by bot label Apr 8, 2022
@ghost ghost requested review from kairu-ms and necusjz April 8, 2022 18:03
@ghost ghost assigned kairu-ms Apr 8, 2022
@ghost ghost added this to the Apr 2022 (2022-04-26) milestone Apr 8, 2022
@ghost ghost requested a review from yonzhan April 8, 2022 18:04
@ghost ghost added the Quantum az quantum label Apr 8, 2022
@ghost ghost requested a review from wangzelin007 April 8, 2022 18:04
@warren-jones warren-jones changed the title [Quantum] [DRAFT -Do not merge] Add "Microsoft.AzureQuantum-" prefix to deployment name [Quantum] [DRAFT - Do not merge] Add "Microsoft.AzureQuantum-" prefix to deployment name Apr 8, 2022
@yonzhan
Copy link
Collaborator

yonzhan commented Apr 8, 2022

Quantum

@yonzhan yonzhan requested a review from jsntcy April 8, 2022 23:32
Copy link
Member

@vxfield vxfield left a comment

Choose a reason for hiding this comment

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

Approved with some feedback to remove to unused commented lines.

Copy link
Member

@vxfield vxfield left a comment

Choose a reason for hiding this comment

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

Approved with some feedback to remove commented lines

@warren-jones warren-jones changed the title [Quantum] [DRAFT - Do not merge] Add "Microsoft.AzureQuantum-" prefix to deployment name [Quantum] Add "Microsoft.AzureQuantum-" prefix to deployment name Apr 12, 2022
@warren-jones warren-jones marked this pull request as ready for review April 12, 2022 22:09
@necusjz necusjz merged commit 85d2295 into Azure:main Apr 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Auto-Assign Auto assign by bot Quantum az quantum

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants