Skip to content
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

[AMS] Fix #14687: Mixed resource group and account name in command "az ams streaming-endpoint show" #14687 #15251

Merged
merged 18 commits into from
Sep 30, 2020

Conversation

hivyas
Copy link
Member

@hivyas hivyas commented Sep 22, 2020

Description

Switched resource group and account name in output string.

Testing Guide

Run the following command with an incorrect streaming endpoint name to see the error string that was changed
az ams streaming-endpoint show --resource-group [rg]--account-name [accountName]--name [incorrectName]

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change.
[Component Name 2] az command b: Add some customer-facing feature.


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

@yonzhan
Copy link
Collaborator

yonzhan commented Sep 23, 2020

AMS

@yonzhan yonzhan requested a review from jiasli September 23, 2020 01:33
@hivyas hivyas added this to the S176 milestone Sep 28, 2020
@arrownj arrownj merged commit e015d5b into Azure:dev Sep 30, 2020
Juliehzl pushed a commit to Juliehzl/azure-cli that referenced this pull request Sep 30, 2020
…nd "az ams streaming-endpoint show" Azure#14687 (Azure#15251)

* reverting changes for ASK key

* reverting changes

* updated ams module to dec 2019 swagger

* added test cases

* added ability to update offlinerental

* fixed audio language bug

* fixing requirements*.txt files

* Fixed transforms test case and national bug

* updated content key policy tests

* fixed style and lint errors

* made az ams sp create idempotent

* fixed typo and updated help text

* switched resource group with account name

* updated test cases
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.

3 participants