-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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
[CognitiveServices] PowerShell refresh to use 2024-10-01 version #26951
base: main
Are you sure you want to change the base?
Conversation
️✔️Az.Accounts
❌Az.CognitiveServices
️✔️Az.Network
|
HelpMessage = "Resource Group Name.")] | ||
[ResourceGroupCompleter()] | ||
[ValidateNotNullOrEmpty] | ||
public string ResourceGroupName { get; set; } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can this work? only account name provided without resourcegroupname?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed
/azp run |
Azure Pipelines successfully started running 3 pipeline(s). |
src/CognitiveServices/CognitiveServices/help/Remove-AzCognitiveServicesAccountRaiBlocklist.md
Outdated
Show resolved
Hide resolved
…/azure-powershell into tianxinliu/2024-10-01
/azp run |
Azure Pipelines successfully started running 3 pipeline(s). |
...gnitiveServices/CognitiveServices/help/Get-AzCognitiveServicesAccountDefenderForAISetting.md
Outdated
Show resolved
Hide resolved
src/CognitiveServices/CognitiveServices/help/Get-AzCognitiveServicesAccountDeploymentSku.md
Outdated
Show resolved
Hide resolved
src/CognitiveServices/CognitiveServices/help/Get-AzCognitiveServicesAccountRaiBlocklist.md
Outdated
Show resolved
Hide resolved
src/CognitiveServices/CognitiveServices/help/Get-AzCognitiveServicesAccountRaiBlocklistItem.md
Outdated
Show resolved
Hide resolved
src/CognitiveServices/CognitiveServices/help/Get-AzCognitiveServicesAccountRaiPolicy.md
Outdated
Show resolved
Hide resolved
src/CognitiveServices/CognitiveServices/help/New-AzCognitiveServicesAccountRaiBlocklistItem.md
Outdated
Show resolved
Hide resolved
src/CognitiveServices/CognitiveServices/help/New-AzCognitiveServicesAccountRaiPolicy.md
Outdated
Show resolved
Hide resolved
src/CognitiveServices/CognitiveServices/help/Remove-AzCognitiveServicesAccountRaiBlocklist.md
Outdated
Show resolved
Hide resolved
...ognitiveServices/CognitiveServices/help/Remove-AzCognitiveServicesAccountRaiBlocklistItem.md
Outdated
Show resolved
Hide resolved
src/CognitiveServices/CognitiveServices/help/Remove-AzCognitiveServicesAccountRaiPolicy.md
Outdated
Show resolved
Hide resolved
…rvicesAccountDefenderForAISetting.md
…rvicesAccountRaiBlocklist.md
…rvicesAccountRaiPolicy.md
…rvicesAccountRaiBlocklistItem.md
…rvicesAccountRaiPolicy.md
…eServicesAccountRaiBlocklist.md
…eServicesAccountRaiBlocklistItem.md
…eServicesAccountRaiPolicy.md
…rvicesAccountRaiBlocklistItem.md
…rvicesModelCapacity.md
…rvicesAccountDeploymentSku.md
…rvicesAccountDefenderForAISetting.md
…rvicesAccountRaiBlocklist.md
/azp run |
Azure Pipelines successfully started running 3 pipeline(s). |
Hi @tianxin-ms , please fix CI errors. |
Commenter does not have sufficient privileges for PR 26951 in repo Azure/azure-powershell |
/azp run |
Azure Pipelines successfully started running 3 pipeline(s). |
|
||
### Example 1 | ||
```powershell | ||
C:\> Get-AzCognitiveServicesAccountRaiBlocklist -ResourceGroupName "rgname" -AccountName "accountname" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please remove C:\>
. Check other examples in main branch.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pushed fix, please help review again.
This PR was labeled "needs-revision" because it has unresolved review comments or CI failures. |
/azp run |
Azure Pipelines successfully started running 3 pipeline(s). |
/azp run |
Azure Pipelines successfully started running 3 pipeline(s). |
Hi @tianxin-ms, Could you merge the latest main branch to your code and try run test locally with playback mode? |
/azp run |
Azure Pipelines successfully started running 3 pipeline(s). |
Hi @tianxin-ms, tests can be started and finished now but failed. Please check and fix them. |
This PR was labeled "needs-revision" because it has unresolved review comments or CI failures. |
Description
Update CognitiveServices Powershell to use 2024-10-01 version
Mandatory Checklist
Please choose the target release of Azure PowerShell. (⚠️ Target release is a different concept from API readiness. Please click below links for details.)
Check this box to confirm: I have read the Submitting Changes section of
CONTRIBUTING.md
and reviewed the following information:ChangeLog.md
file(s) appropriatelysrc/{{SERVICE}}/{{SERVICE}}/ChangeLog.md
.## Upcoming Release
header in the past tense.ChangeLog.md
if no new release is required, such as fixing test case only.