Skip to content

Assign default name to QueueService #49672

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

Merged
merged 7 commits into from
May 16, 2025

Conversation

kzhen
Copy link
Contributor

@kzhen kzhen commented Apr 28, 2025

Fixes #49898

Updated QueueService to use default for the name property value as per the spec https://learn.microsoft.com/en-us/azure/templates/microsoft.storage/storageaccounts/queueservices?pivots=deployment-language-bicep

Contributing to the Azure SDK

Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.

For specific information about pull request etiquette and best practices, see this section.

@github-actions github-actions bot added Community Contribution Community members are working on the issue customer-reported Issues that are reported by GitHub users external to the Azure organization. Provisioning labels Apr 28, 2025
Copy link

Thank you for your contribution @kzhen! We will review the pull request and get back to you soon.

@kzhen
Copy link
Contributor Author

kzhen commented Apr 28, 2025

I've raised this PR after trying to create storage queues during deployment via Aspire - details with a reproduction

@RussKie
Copy link

RussKie commented May 7, 2025

dotnet/aspire#9137 is blocked by the same issue.

@azure-sdk
Copy link
Collaborator

API change check

API changes are not detected in this pull request.

@RussKie
Copy link

RussKie commented May 12, 2025

Error : When the project has a release version it shouldn't reference any pre-release libraries. Found the following pre-release references: D:\a\_work\1\s\sdk\provisioning\Azure.Provisioning\src\Azure.Provisioning.csproj

@ArcturusZhang
Copy link
Member

We need to merge #50036 first in order to release an individual package.
Now all the provisioning packages are bundled together therefore we have to release all of them at once from the same PR.
#50036 should be able to change this.

@ArcturusZhang ArcturusZhang merged commit 5264b64 into Azure:main May 16, 2025
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Community Contribution Community members are working on the issue customer-reported Issues that are reported by GitHub users external to the Azure organization. Provisioning
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Provisioning] QueueService's Name should have a default value of default in Azure.Provisioning.Storage
5 participants