-
Notifications
You must be signed in to change notification settings - Fork 4k
Update api-version #6249
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
Update api-version #6249
Conversation
This looks good to me. @cormacpayne - are the failing tests disabled? Is it possible we can re-enable them for this PR to make sure they are passing after these changes in CI? |
@antoniowmsft @matthchr the tests are currently disabled in the Here's the on-demand job for this PR: https://azuresdkci.westus2.cloudapp.azure.com/job/powershell-demand/858/ |
Hmm, it looks like the on-demand job still failed:
The error was:
I do see this field in the recording files, it may be that we have to actually fully re-record the tests due to it: |
@antoniowmsft assigning back to you to address test failures. |
@antoniowmsft Still seeing some failing test - will keep this PR assigned to you. |
@antoniowmsft Ping on the failing tests above - are you having issues rerecording these tests? |
We can close this PR, I've rebased this PRs contents onto |
Description
Update api-version in playback files
Checklist
CONTRIBUTING.md
platyPS
module