azure
azure copied to clipboard
[Feature] improve service principal listing
SUMMARY
Fixes #759
This allows the user to list service principals by display name, or just all principals for post-list processing.
examples
- name: Get ad service principal info by object_id
azure_rm_adserviceprincipal_info:
tenant: "{{ tenant_id }}"
register: ad_info
- name: Get ad service principal info by object_id
azure_rm_adserviceprincipal_info:
tenant: "{{ tenant_id }}"
app_display_name: "{{ ad_info.ad_fact.app_display_name }}"
register: ad_info
Signed-off-by: Paul Czarkowski [email protected]
ISSUE TYPE
- Feature Pull Request
COMPONENT NAME
azure_rm_adserviceprincipal_info
I can't get the tests to run, hopefully I wrote the tests correctly 🤞
@Fred-sun Sorry I had forgotten all about this. I've added your changes, and responded to your comment.
@paulczar Thank you for your update and we will complete the test and move forward with the merger as soon as possible! Thanks!
Sorry I let this sit for so long @Fred-sun ! I just merged your suggestion so we should be good to go!
@paulczar I'm pushing for a merger. Thank you!
@paulczar azure-graphrbac has been abandoned and migrated to msgraph(#1325), where the corresponding method is also supported in msgraph, this problem has been fixed, first close this PR, if you have new suggestions for change, you can propose a new PR, thank you!