-
Notifications
You must be signed in to change notification settings - Fork 4k
Added Policy API-version 2019-06-01. Policy assignment cmdlet updated to accept EnforcementMode property #10421
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
… for new enforcementMode property
Can one of the admins verify this patch? |
src/Resources/ResourceManager/Entities/Policy/EnforcementMode.cs
Outdated
Show resolved
Hide resolved
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.
Approving, but I left a couple of comments you should consider.
@sandipsh please fix the failed test case |
src/Resources/ResourceManager/Implementation/Policy/PolicyHelpStrings.cs
Show resolved
Hide resolved
@@ -12,6 +12,8 @@ | |||
// limitations under the License. | |||
// ---------------------------------------------------------------------------------- | |||
|
|||
using System; |
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.
Move this inside the namespace.
src/Resources/ResourceManager/Implementation/Policy/SetAzurePolicyAssignment.cs
Show resolved
Hide resolved
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.
Left a couple of minor comments. Looks good though.
Addressed all comments and updated session record json file for failing test. |
/azp run |
Azure Pipelines successfully started running 3 pipeline(s). |
Remove the new version 1.7.2
… for new enforcementMode property
Description
Updated Policy api version to 2019-06-01. Update New-AzPolicyAssignment and Set-AzPolicyAssignment cmdlets to accept -EnforcementMode parameter. Added test for the new property. Updated help and changelog. Also updated create policy definition help examples.
Checklist
CONTRIBUTING.md
ChangeLog.md
file(s) has been updated:ChangeLog.md
file can be found atsrc/{{SERVICE}}/{{SERVICE}}/ChangeLog.md
## Upcoming Release
header -- no new version header should be added