-
Notifications
You must be signed in to change notification settings - Fork 4.1k
Bug fix: "Get-AzAutomationJob fails for some jobIds" #19155
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
Conversation
Co-authored-by: Beisi Zhou <zhoubeisi@gmail.com>
Co-authored-by: Beisi Zhou <zhoubeisi@gmail.com>
|
Thank you for your contribution Ab0907! We will review the pull request and get back to you soon. |
|
@Ab0907 please resolve the conflict in ChangeLog.md that's blocking CI. |
|
Joined Azure org and conflict blocking CLI has been resolved |
isra-fel
left a comment
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 fix the test cases.
Also, consider using AzurePSCmdlet.PowerShellVersion property to avoid duplication.
|
Used AzurePSCmdlet.PowerShellVersion property to avoid duplication. |
|
/azp run azure-powershell - security-tools |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Description
Get-AzAutomationJob failed for some jobIds with the error - "Conversion from JSON failed with error: Unexpected character encountered while parsing value: D. Path '', line 0, position 0."
The bug was specific to Powershell 7 and was occurring due to ConvertFrom-Json not identifying strings without quotes around them
Checklist
CONTRIBUTING.mdand reviewed the following information:generationbranch.ChangeLog.mdfile(s) appropriatelyChangeLog.mdfile can be found atsrc/{{SERVICE}}/{{SERVICE}}/ChangeLog.md## Upcoming Releaseheader in the past tense. Add changelog in description section if PR goes intogenerationbranch.ChangeLog.mdif no new release is required, such as fixing test case only.