-
Notifications
You must be signed in to change notification settings - Fork 4k
Fix for New-AzureRmRoleAssignment and Remove-AzureRmRoleAssignment fo… #7228
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
Updated this PR with commits from #7221 which got in to preview already |
@maddieclayton Could you please help why these IOTHub tests are failing... I have already retried.. If they are fixed in preview branch.. can we override and merge this PR? System.Management.Automation.ParameterBindingException : A parameter cannot be found that matches parameter name '-TextExtension'. |
@cormacpayne /@markcowl as well if they know about these IOT hub test failures.. I have already retired. Thanks for looking in to this! |
@bganapa All the admin tests seem to be failing.
|
…e needed psswaggerutilty
@deathly809 good catch.. it was my commit that removed a needed psd1 file in the cleanup script, my yesterday night validation run failed as well. Fixed it with the above commit |
@deathly809 Asides we should enable failing of the job when the pester tests are failing.. |
@maddieclayton @deathly809 The admin tests are passing now |
Fix for New-AzureRmRoleAssignment and Remove-AzureRmRoleAssignment fo…
Description
Fix for New-AzureRmRoleAssignment and Remove-AzureRmRoleAssignment for ADFS scenario
Checklist
CONTRIBUTING.md
platyPS
module