Added Access Reviews schema #10010

shubhamguptacal · 2020-07-01T21:53:31Z

IGA (Identity Governance and Administration group) has a service called access reviews. The access reviews scenario for ARM involves reviewing access of user resource assignments. There is documentation for the scenario available here.

This PR includes exposure of Access Review APIs in the existing resource provider Microsoft.Authorization. We already had a preliminary review meeting with Kristopher Bash in this regard and the PR against the private repo was already reviewed : https://github.com/Azure/azure-rest-api-specs-pr/pull/1213

openapi-pipeline-app · 2020-07-01T21:53:34Z

[Staging] Swagger Validation Report

️️✔️BreakingChange [Detail]

 There are no breaking changes.

️️✔️LintDiff [Detail]

 Validation passes for LintDiff.

️️✔️Avocado [Detail]

 Validation passes for Avocado.

_{Posted by Swagger Pipeline | How to fix these errors?}

azure-pipelines · 2020-07-01T21:53:46Z

Azure Pipelines successfully started running 1 pipeline(s).

openapi-sdkautomation · 2020-07-01T21:55:14Z

Trenton Generation

No readme.md specification configuration files were found that are associated with the files modified in this pull request, or swagger_to_sdk section in readme.md is not configured

openapi-sdkautomation · 2020-07-01T21:55:15Z

azure-sdk-for-go

️✔️ succeeded [Logs] [Expand Details]

️✔️ Generate from 9efaec3 with merge commit bb9847a223172a9cae7cab51a09b660b28450247. SDK Automation 13.0.17.20200619.4
️✔️authorization/mgmt/2015-07-01 [View full logs] [Preview SDK Changes]
- authorization.mgmt.2015-07-01.zip
️✔️preview/authorization/mgmt/2017-10-01-preview [View full logs] [Preview SDK Changes]
- preview.authorization.mgmt.2017-10-01-preview.zip
️✔️preview/authorization/mgmt/2018-01-01-preview [View full logs] [Preview SDK Changes]
- preview.authorization.mgmt.2018-01-01-preview.zip
️✔️preview/authorization/mgmt/2018-07-01-preview [View full logs] [Preview SDK Changes]
- preview.authorization.mgmt.2018-07-01-preview.zip
️✔️preview/authorization/mgmt/2018-09-01-preview [View full logs] [Preview SDK Changes]
- preview.authorization.mgmt.2018-09-01-preview.zip

openapi-sdkautomation · 2020-07-01T21:55:16Z

azure-sdk-for-java

️✔️ succeeded [Logs] [Expand Details]

️✔️ Generate from 9efaec3 with merge commit bb9847a223172a9cae7cab51a09b660b28450247. SDK Automation 13.0.17.20200619.4
️✔️sdk/authorization/mgmt-v2015_06_01 [View full logs] [Preview SDK Changes]
️✔️sdk/authorization/mgmt-v2018_07_01_preview [View full logs] [Preview SDK Changes]
️✔️sdk/authorization/mgmt-v2018_09_01_preview [View full logs] [Preview SDK Changes]

openapi-sdkautomation · 2020-07-01T21:55:16Z

Azure CLI Extension Generation

No readme.md specification configuration files were found that are associated with the files modified in this pull request, or swagger_to_sdk section in readme.md is not configured

openapi-sdkautomation · 2020-07-01T21:55:17Z

azure-sdk-for-js

️✔️ succeeded [Logs] [Expand Details]

️✔️ Generate from 9efaec3 with merge commit bb9847a223172a9cae7cab51a09b660b28450247. SDK Automation 13.0.17.20200619.4

️✔️@azure/arm-authorization [View full logs] [Preview SDK Changes]

azure-arm-authorization-8.3.2.tgz

Only show 100 items here, please refer to log for details.

[npmPack] npm WARN deprecated rollup-plugin-node-resolve@5.2.0: This package has been deprecated and is no longer maintained. Please use @rollup/plugin-node-resolve.
[npmPack] npm WARN deprecated resolve-url@0.2.1: https://github.com/lydell/resolve-url#deprecated
[npmPack] npm WARN deprecated urix@0.1.0: Please see https://github.com/lydell/urix#deprecated
[npmPack] npm notice created a lockfile as package-lock.json. You should commit this file.
[npmPack] loaded rollup.config.js with warnings
[npmPack] (!) Unused external imports
[npmPack] default imported from external module 'rollup' but never used
[npmPack] 
[npmPack] ./esm/authorizationManagementClient.js → ./dist/arm-authorization.js...
[npmPack] created ./dist/arm-authorization.js in 357ms
[npmPack] npm notice 
[npmPack] npm notice package: @azure/arm-authorization@8.3.2
[npmPack] npm notice === Tarball Contents === 
[npmPack] npm notice 87.1kB  dist/arm-authorization.js                                      
[npmPack] npm notice 29.3kB  dist/arm-authorization.min.js                                  
[npmPack] npm notice 2.1kB   esm/authorizationManagementClient.js                           
[npmPack] npm notice 2.5kB   esm/authorizationManagementClientContext.js                    
[npmPack] npm notice 2.5kB   esm/operations/classicAdministrators.js                        
[npmPack] npm notice 470B    esm/models/classicAdministratorsMappers.js                     
[npmPack] npm notice 9.5kB   esm/operations/denyAssignments.js                              
[npmPack] npm notice 489B    esm/models/denyAssignmentsMappers.js                           
[npmPack] npm notice 1.6kB   esm/operations/globalAdministrator.js                          
[npmPack] npm notice 414B    esm/models/globalAdministratorMappers.js                       
[npmPack] npm notice 345B    esm/models/index.js                                            
[npmPack] npm notice 623B    esm/operations/index.js                                        
[npmPack] npm notice 22.5kB  esm/models/mappers.js                                          
[npmPack] npm notice 5.2kB   esm/models/parameters.js                                       
[npmPack] npm notice 4.8kB   esm/operations/permissions.js                                  
[npmPack] npm notice 439B    esm/models/permissionsMappers.js                               
[npmPack] npm notice 3.5kB   esm/operations/providerOperationsMetadataOperations.js         
[npmPack] npm notice 530B    esm/models/providerOperationsMetadataOperationsMappers.js      
[npmPack] npm notice 13.0kB  esm/operations/roleAssignments.js                              
[npmPack] npm notice 484B    esm/models/roleAssignmentsMappers.js                           
[npmPack] npm notice 6.0kB   esm/operations/roleDefinitions.js                              
[npmPack] npm notice 464B    esm/models/roleDefinitionsMappers.js                           
[npmPack] npm notice 1.0kB   rollup.config.js                                               
[npmPack] npm notice 1.7kB   package.json                                                   
[npmPack] npm notice 457B    tsconfig.json                                                  
[npmPack] npm notice 216.9kB dist/arm-authorization.js.map                                  
[npmPack] npm notice 27.6kB  dist/arm-authorization.min.js.map                              
[npmPack] npm notice 860B    esm/authorizationManagementClient.d.ts.map                     
[npmPack] npm notice 1.1kB   esm/authorizationManagementClient.js.map                       
[npmPack] npm notice 493B    esm/authorizationManagementClientContext.d.ts.map              
[npmPack] npm notice 1.4kB   esm/authorizationManagementClientContext.js.map                
[npmPack] npm notice 1.1kB   esm/operations/classicAdministrators.d.ts.map                  
[npmPack] npm notice 1.5kB   esm/operations/classicAdministrators.js.map                    
[npmPack] npm notice 227B    esm/models/classicAdministratorsMappers.d.ts.map               
[npmPack] npm notice 240B    esm/models/classicAdministratorsMappers.js.map                 
[npmPack] npm notice 4.4kB   esm/operations/denyAssignments.d.ts.map                        
[npmPack] npm notice 5.9kB   esm/operations/denyAssignments.js.map                          
[npmPack] npm notice 235B    esm/models/denyAssignmentsMappers.d.ts.map                     
[npmPack] npm notice 248B    esm/models/denyAssignmentsMappers.js.map                       
[npmPack] npm notice 621B    esm/operations/globalAdministrator.d.ts.map                    
[npmPack] npm notice 891B    esm/operations/globalAdministrator.js.map                      
[npmPack] npm notice 197B    esm/models/globalAdministratorMappers.d.ts.map                 
[npmPack] npm notice 210B    esm/models/globalAdministratorMappers.js.map                   
[npmPack] npm notice 13.3kB  esm/models/index.d.ts.map                                      
[npmPack] npm notice 275B    esm/operations/index.d.ts.map                                  
[npmPack] npm notice 126B    esm/models/index.js.map                                        
[npmPack] npm notice 290B    esm/operations/index.js.map                                    
[npmPack] npm notice 1.2kB   esm/models/mappers.d.ts.map                                    
[npmPack] npm notice 12.2kB  esm/models/mappers.js.map                                      
[npmPack] npm notice 1.2kB   esm/models/parameters.d.ts.map                                 
[npmPack] npm notice 4.0kB   esm/models/parameters.js.map                                   
[npmPack] npm notice 2.1kB   esm/operations/permissions.d.ts.map                            
[npmPack] npm notice 2.8kB   esm/operations/permissions.js.map                              
[npmPack] npm notice 204B    esm/models/permissionsMappers.d.ts.map                         
[npmPack] npm notice 217B    esm/models/permissionsMappers.js.map                           
[npmPack] npm notice 1.5kB   esm/operations/providerOperationsMetadataOperations.d.ts.map   
[npmPack] npm notice 2.0kB   esm/operations/providerOperationsMetadataOperations.js.map     
[npmPack] npm notice 280B    esm/models/providerOperationsMetadataOperationsMappers.d.ts.map
[npmPack] npm notice 293B    esm/models/providerOperationsMetadataOperationsMappers.js.map  
[npmPack] npm notice 6.2kB   esm/operations/roleAssignments.d.ts.map                        
[npmPack] npm notice 8.3kB   esm/operations/roleAssignments.js.map                          
[npmPack] npm notice 225B    esm/models/roleAssignmentsMappers.d.ts.map                     
[npmPack] npm notice 238B    esm/models/roleAssignmentsMappers.js.map                       
[npmPack] npm notice 2.9kB   esm/operations/roleDefinitions.d.ts.map                        
[npmPack] npm notice 3.8kB   esm/operations/roleDefinitions.js.map                          
[npmPack] npm notice 222B    esm/models/roleDefinitionsMappers.d.ts.map                     
[npmPack] npm notice 235B    esm/models/roleDefinitionsMappers.js.map                       
[npmPack] npm notice 3.3kB   README.md                                                      
[npmPack] npm notice 1.4kB   esm/authorizationManagementClient.d.ts                         
[npmPack] npm notice 2.3kB   src/authorizationManagementClient.ts                           
[npmPack] npm notice 809B    esm/authorizationManagementClientContext.d.ts                  
[npmPack] npm notice 2.3kB   src/authorizationManagementClientContext.ts                    
[npmPack] npm notice 2.3kB   esm/operations/classicAdministrators.d.ts                      
[npmPack] npm notice 4.6kB   src/operations/classicAdministrators.ts                        
[npmPack] npm notice 160B    esm/models/classicAdministratorsMappers.d.ts                   
[npmPack] npm notice 421B    src/models/classicAdministratorsMappers.ts                     
[npmPack] npm notice 12.3kB  esm/operations/denyAssignments.d.ts                            
[npmPack] npm notice 22.4kB  src/operations/denyAssignments.ts                              
[npmPack] npm notice 179B    esm/models/denyAssignmentsMappers.d.ts                         
[npmPack] npm notice 450B    src/models/denyAssignmentsMappers.ts                           
[npmPack] npm notice 1.1kB   esm/operations/globalAdministrator.d.ts                        
[npmPack] npm notice 2.2kB   src/operations/globalAdministrator.ts                          
[npmPack] npm notice 104B    esm/models/globalAdministratorMappers.d.ts                     
[npmPack] npm notice 363B    src/models/globalAdministratorMappers.ts                       
[npmPack] npm notice 37.0kB  esm/models/index.d.ts                                          
[npmPack] npm notice 307B    esm/operations/index.d.ts                                      
[npmPack] npm notice 35.3kB  src/models/index.ts

openapi-sdkautomation · 2020-07-01T21:55:18Z

azure-sdk-for-python

️✔️ succeeded [Logs] [Expand Details]

️✔️ Generate from 9efaec3 with merge commit bb9847a223172a9cae7cab51a09b660b28450247. SDK Automation 13.0.17.20200619.4

️✔️azure-mgmt-authorization [View full logs] [Preview SDK Changes]

[build_conf] INFO:packaging_tools:Building template azure-mgmt-authorization
[build_conf] INFO:packaging_tools.conf:Skipping default conf since the file exists
[build_conf] INFO:packaging_tools:Skipping CHANGELOG.md template, since a previous one was found
[build_conf] INFO:packaging_tools:Template done azure-mgmt-authorization
[after_scripts|python] INFO:__main__:Default API version will be: v2018_09_01_preview
[build_package] /usr/lib/python3.6/distutils/dist.py:261: UserWarning: Unknown distribution option: 'long_description_content_type'
[build_package]   warnings.warn(msg)
[build_package] /usr/lib/python3.6/distutils/dist.py:261: UserWarning: Unknown distribution option: 'long_description_content_type'
[build_package]   warnings.warn(msg)
[breaking_change_setup] Ignoring mock: markers 'python_version <= "2.7"' don't match your environment
[ChangeLog] Size of delta 58.355% size of original (original: 38192 chars, delta: 22287 chars)
[ChangeLog] **Features**
[ChangeLog] 
[ChangeLog]   - Added operation group AccessReviewDefaultSettingsOperations
[ChangeLog]   - Added operation group AccessReviewInstanceDecisionsOperations
[ChangeLog]   - Added operation group AccessReviewInstanceOperations
[ChangeLog]   - Added operation group AccessReviewInstancesOperations
[ChangeLog]   - Added operation group AccessReviewScheduleDefinitionsOperations
[ChangeLog]   - Added operation group Operations

openapi-sdkautomation · 2020-07-01T21:55:19Z

azure-sdk-for-net

️✔️ succeeded [Logs] [Expand Details]

️✔️ Generate from 9efaec3 with merge commit bb9847a223172a9cae7cab51a09b660b28450247. SDK Automation 13.0.17.20200619.4

[AutoRest] realpath(): Permission denied
[AutoRest] realpath(): Permission denied
[AutoRest] realpath(): Permission denied
[AutoRest] realpath(): Permission denied
[AutoRest] realpath(): Permission denied
[AutoRest] realpath(): Permission denied

️✔️Microsoft.Azure.Management.Authorization [View full logs] [Preview SDK Changes]
- Microsoft.Azure.Management.Authorization.2.11.0-preview.nupkg
- Microsoft.Azure.Management.Authorization.2.11.0-preview.snupkg

azuresdkci · 2020-07-01T22:01:12Z

Can one of the admins verify this patch?

azure-pipelines · 2020-07-01T22:16:15Z

Azure Pipelines successfully started running 1 pipeline(s).

openapi-sdkautomation · 2020-07-01T22:17:57Z

azure-sdk-for-python-track2

No readme.md specification configuration files were found that are associated with the files modified in this pull request, or swagger_to_sdk section in readme.md is not configured

azure-pipelines · 2020-07-01T22:38:11Z

Azure Pipelines successfully started running 1 pipeline(s).

azure-pipelines · 2020-07-02T00:17:20Z

Azure Pipelines successfully started running 1 pipeline(s).

azure-pipelines · 2020-07-02T00:39:24Z

Azure Pipelines successfully started running 1 pipeline(s).

azure-pipelines · 2020-07-02T00:41:56Z

Azure Pipelines successfully started running 1 pipeline(s).

azure-pipelines · 2020-07-02T01:00:43Z

Azure Pipelines successfully started running 1 pipeline(s).

shubhamguptacal · 2020-07-06T17:01:07Z

@qianwens can you please take a look at these changes?

azure-pipelines · 2020-07-07T02:27:07Z

Azure Pipelines successfully started running 1 pipeline(s).

azure-pipelines · 2020-07-07T02:41:53Z

Azure Pipelines successfully started running 1 pipeline(s).

azure-pipelines · 2020-07-07T02:49:14Z

Azure Pipelines successfully started running 1 pipeline(s).

azure-pipelines · 2020-07-07T17:12:28Z

Azure Pipelines successfully started running 1 pipeline(s).

shubhamguptacal · 2020-07-07T17:32:24Z

@qianwens I've added a DoNotMerge label for the moment since I want to release the SDK after we've pushed out some service updates. I'll remove it as soon as the service updates are deployed and remove this label. We can merge it then.

majastrz

Did diff between approved private PR and this one. Delta looks good to me.

qianwens · 2020-10-23T06:51:00Z

Is this PR still needed? If so, Please close this PR and create a new one since this one has exceeded 3 months.

JackTn · 2021-08-23T03:39:09Z

Hi, @shubhamguptacal. The PR has be closed for a long time and it's related branch still exist. If no further used for over 14 days, I will delete this related branch. Please tell me if you still need this branch.

JackTn · 2022-06-16T03:01:35Z

Hi, @shubhamguptacal. The PR has be closed for a long time and it's related branch still exist. Please tell me if you still need this branch or i will delete it in 14 days.

JackTn · 2022-06-16T03:12:14Z

Hi, @shubhamguptacal. The PR has be closed for a long time and it's related branch still exist. Please tell me if you still need this branch or i will delete it in 14 days.

JackTn · 2022-06-16T03:26:19Z

Hi, @shubhamguptacal. The PR has be closed for a long time and it's related branch still exist. Please tell me if you still need this branch or i will delete it in 14 days.

shubhamguptacal requested a review from stankovski as a code owner July 1, 2020 21:53

openapi-assignment-bot bot assigned qianwens Jul 1, 2020

shubhamguptacal force-pushed the accessreviews branch from e876f9c to 74e7450 Compare July 1, 2020 22:15

shubhamguptacal force-pushed the accessreviews branch from 74e7450 to 534ae71 Compare July 1, 2020 22:37

shubhamguptacal force-pushed the accessreviews branch from 534ae71 to ea0885f Compare July 2, 2020 00:17

shubhamguptacal force-pushed the accessreviews branch from ea0885f to 01d8895 Compare July 2, 2020 00:39

shubhamguptacal force-pushed the accessreviews branch from 01d8895 to 84562ba Compare July 2, 2020 00:41

openapi-sdkautomation bot mentioned this pull request Jul 2, 2020

[AutoPR azure-mgmt-authorization] Added Access Reviews schema AzureSDKAutomation/azure-sdk-for-python#1522

Open

shubhamguptacal force-pushed the accessreviews branch from 84562ba to 67fffef Compare July 2, 2020 01:00

qianwens approved these changes Jul 7, 2020

View reviewed changes

shubhamguptacal closed this Jul 7, 2020

shubhamguptacal reopened this Jul 7, 2020

shubhamguptacal force-pushed the accessreviews branch from 67fffef to 88288a6 Compare July 7, 2020 02:41

shubhamguptacal force-pushed the accessreviews branch from 88288a6 to 5923e10 Compare July 7, 2020 02:48

Added Access Reviews schema

9efaec3

shubhamguptacal force-pushed the accessreviews branch from 5923e10 to 9efaec3 Compare July 7, 2020 17:12

shubhamguptacal added the DoNotMerge <valid label in PR review process> use to hold merge after approval label Jul 7, 2020

majastrz added the ARMSignedOff <valid label in PR review process>add this label when ARM approve updates after review label Aug 4, 2020

majastrz approved these changes Aug 4, 2020

View reviewed changes

shubhamguptacal mentioned this pull request Aug 12, 2020

Addendum | Access Reviews reviewer and stats APIs #10445

Merged

qianwens closed this Nov 5, 2020

JackTn deleted the accessreviews branch July 5, 2022 01:18

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Added Access Reviews schema #10010

Added Access Reviews schema #10010

shubhamguptacal commented Jul 1, 2020 •

edited

Loading

openapi-pipeline-app bot commented Jul 1, 2020 •

edited

Loading

azure-pipelines bot commented Jul 1, 2020

openapi-sdkautomation bot commented Jul 1, 2020 •

edited

Loading

openapi-sdkautomation bot commented Jul 1, 2020 •

edited

Loading

openapi-sdkautomation bot commented Jul 1, 2020 •

edited

Loading

openapi-sdkautomation bot commented Jul 1, 2020 •

edited

Loading

openapi-sdkautomation bot commented Jul 1, 2020 •

edited

Loading

openapi-sdkautomation bot commented Jul 1, 2020 •

edited

Loading

openapi-sdkautomation bot commented Jul 1, 2020 •

edited

Loading

azuresdkci commented Jul 1, 2020

azure-pipelines bot commented Jul 1, 2020

openapi-sdkautomation bot commented Jul 1, 2020 •

edited

Loading

azure-pipelines bot commented Jul 1, 2020

azure-pipelines bot commented Jul 2, 2020

azure-pipelines bot commented Jul 2, 2020

azure-pipelines bot commented Jul 2, 2020

azure-pipelines bot commented Jul 2, 2020

shubhamguptacal commented Jul 6, 2020

azure-pipelines bot commented Jul 7, 2020

azure-pipelines bot commented Jul 7, 2020

azure-pipelines bot commented Jul 7, 2020

azure-pipelines bot commented Jul 7, 2020

shubhamguptacal commented Jul 7, 2020

majastrz left a comment

qianwens commented Oct 23, 2020

JackTn commented Aug 23, 2021

JackTn commented Jun 16, 2022

JackTn commented Jun 16, 2022

JackTn commented Jun 16, 2022

Added Access Reviews schema #10010

Added Access Reviews schema #10010

Conversation

shubhamguptacal commented Jul 1, 2020 • edited Loading

openapi-pipeline-app bot commented Jul 1, 2020 • edited Loading

[Staging] Swagger Validation Report

azure-pipelines bot commented Jul 1, 2020

openapi-sdkautomation bot commented Jul 1, 2020 • edited Loading

Trenton Generation

openapi-sdkautomation bot commented Jul 1, 2020 • edited Loading

azure-sdk-for-go

openapi-sdkautomation bot commented Jul 1, 2020 • edited Loading

azure-sdk-for-java

openapi-sdkautomation bot commented Jul 1, 2020 • edited Loading

Azure CLI Extension Generation

openapi-sdkautomation bot commented Jul 1, 2020 • edited Loading

azure-sdk-for-js

openapi-sdkautomation bot commented Jul 1, 2020 • edited Loading

azure-sdk-for-python

openapi-sdkautomation bot commented Jul 1, 2020 • edited Loading

azure-sdk-for-net

azuresdkci commented Jul 1, 2020

azure-pipelines bot commented Jul 1, 2020

openapi-sdkautomation bot commented Jul 1, 2020 • edited Loading

azure-sdk-for-python-track2

azure-pipelines bot commented Jul 1, 2020

azure-pipelines bot commented Jul 2, 2020

azure-pipelines bot commented Jul 2, 2020

azure-pipelines bot commented Jul 2, 2020

azure-pipelines bot commented Jul 2, 2020

shubhamguptacal commented Jul 6, 2020

azure-pipelines bot commented Jul 7, 2020

azure-pipelines bot commented Jul 7, 2020

azure-pipelines bot commented Jul 7, 2020

azure-pipelines bot commented Jul 7, 2020

shubhamguptacal commented Jul 7, 2020

majastrz left a comment

Choose a reason for hiding this comment

qianwens commented Oct 23, 2020

JackTn commented Aug 23, 2021

JackTn commented Jun 16, 2022

JackTn commented Jun 16, 2022

JackTn commented Jun 16, 2022

shubhamguptacal commented Jul 1, 2020 •

edited

Loading

openapi-pipeline-app bot commented Jul 1, 2020 •

edited

Loading

openapi-sdkautomation bot commented Jul 1, 2020 •

edited

Loading

openapi-sdkautomation bot commented Jul 1, 2020 •

edited

Loading

openapi-sdkautomation bot commented Jul 1, 2020 •

edited

Loading

openapi-sdkautomation bot commented Jul 1, 2020 •

edited

Loading

openapi-sdkautomation bot commented Jul 1, 2020 •

edited

Loading

openapi-sdkautomation bot commented Jul 1, 2020 •

edited

Loading

openapi-sdkautomation bot commented Jul 1, 2020 •

edited

Loading

openapi-sdkautomation bot commented Jul 1, 2020 •

edited

Loading