azure icon indicating copy to clipboard operation
azure copied to clipboard

[Feature] improve service principal listing

Open paulczar opened this issue 2 years ago • 1 comments

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

paulczar avatar Jun 24 '22 16:06 paulczar

I can't get the tests to run, hopefully I wrote the tests correctly 🤞

paulczar avatar Jun 24 '22 16:06 paulczar

@Fred-sun Sorry I had forgotten all about this. I've added your changes, and responded to your comment.

paulczar avatar Nov 07 '22 19:11 paulczar

@paulczar Thank you for your update and we will complete the test and move forward with the merger as soon as possible! Thanks!

Fred-sun avatar Nov 09 '22 06:11 Fred-sun

Sorry I let this sit for so long @Fred-sun ! I just merged your suggestion so we should be good to go!

paulczar avatar Jul 31 '23 16:07 paulczar

@paulczar I'm pushing for a merger. Thank you!

Fred-sun avatar Aug 01 '23 02:08 Fred-sun

@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!

Fred-sun avatar Dec 01 '23 01:12 Fred-sun