[ServiceBus] Fix #24741: Remove invalid arguments for entity update commands

[PramodValavala-MSFT]

Related command

az servicebus queue update --resource-group {} --namespace-name {} --name {} --enable-partitioning --enable-duplicate-detection --enable-session
az servicebus topic update --resource-group {} --namespace-name {} --name {} --enable-partitioning --enable-duplicate-detection
az servicebus topic subscription update --resource-group {} --namespace-name {} --name {} --enable-session

Description

The following properties of an entity cannot be updated post creation but the update the commands for entities have these arguments, which are invalid and fail as such

• Partitioning (Queues & Topics)
• Duplicate Detection (Queues & Topics)
• Sessions (Queues & Subscriptions)

This PR removes the arguments for the update commands.

resolves #24741

Testing Guide

az servicebus queue update --resource-group {} --namespace-name {} --name {} --enable-partitioning --enable-duplicate-detection --enable-session
az servicebus topic update --resource-group {} --namespace-name {} --name {} --enable-partitioning --enable-duplicate-detection
az servicebus topic subscription update --resource-group {} --namespace-name {} --name {} --enable-session

The above commands currently fail with a 400 Bad Request, which is expected due to the invalid arguments that can't be updated post entity creation. With this PR, they should be removed from the documentation, command help, and error as invalid arguments directly.

History Notes

[ServiceBus] az servicebus queue update: Remove invalid arguments for queue update command
[ServiceBus] az servicebus topic update: Remove invalid arguments for topic update command
[ServiceBus] az servicebus topic subscription update: Remove invalid arguments for subscription update command

---

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

• The PR title and description has followed the guideline in Submitting Pull Requests.

• I adhere to the Command Guidelines.

• I adhere to the Error Handling Guidelines.

[yonzhan]

ServiceBus

[damodaravadhani]

Hi @PramodValavala-MSFT , thanks for working on this. There might need to be testcase updates that need to be made to this, I can take this from here if you invite me as a collaborator on your azure-cli fork.

[PramodValavala-MSFT]

@damodaravadhani Sure. I've added you as a contributor to my fork. Let me know what tests you have in mind and I could help out too, if needed.

[evelyn-ys]

Hi @PramodValavala-MSFT , this is breaking change and we only allow breaking change twice a year. Can you add deprecation and warnings for now and totally remove them in next breaking change window?

[evelyn-ys]

Oh I've already seen #24937 adding the deprecation warning. I'll close this PR then