Skip to content

Conversation

@Matbefn
Copy link
Contributor

@Matbefn Matbefn commented Mar 12, 2025

SUMMARY

Adding the field notes from Graph API documentation (https://learn.microsoft.com/en-us/graph/api/resources/application?view=graph-rest-1.0#properties) to support adding Internal notes to a registered app.

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

azure_rm_adapplication

ADDITIONAL INFORMATION

In Azure Portal, per registered app, an internal note can be set for additional information. Setting this was not possible via Ansible.
Ansible_screenshot

- name: Create Azure AD Application
  azure.azcollection.azure_rm_adapplication:
    display_name: "{{ display_name }}"
    tenant: "{{ tenant }}"
    auth_source: cli
    subscription_id: "{{ subscription_id }}"
    notes: "Created via Ansible"

Before:
ok: [localhost] => {
"msg": {
"app_id": "",
"app_roles": [],
"changed": true,
"display_name": "Ansible created app registration",
"failed": false,
"homepage": null,
"identifier_uris": [],
"oauth2_allow_implicit_flow": false,
"object_id": "",
"optional_claims": null,
"public_client_reply_urls": [],
"sign_in_audience": "AzureADMyOrg",
"spa_reply_urls": [],
"web_reply_urls": []
}
}

After:
ok: [localhost] => {
"msg": {
"app_id": "",
"app_roles": [],
"changed": true,
"display_name": "Ansible created app registration",
"failed": false,
"homepage": null,
"identifier_uris": [],
"notes": "Created via Ansible",
"oauth2_allow_implicit_flow": false,
"object_id": "",
"optional_claims": null,
"public_client_reply_urls": [],
"sign_in_audience": "AzureADMyOrg",
"spa_reply_urls": [],
"web_reply_urls": []
}
}

@Fred-sun Fred-sun added medium_priority Medium priority work in In trying to solve, or in working with contributors labels Mar 13, 2025
@Fred-sun
Copy link
Collaborator

@Matbefn Greate! It would be nice to add notes to azure_rm_adapplication_info.py as well. Thank you!

Add to line 230
    notes:
        description:
            - Notes relevant for the management of the application.
        type: str
        returned: always
        sample: "Test value"
Add to line 345
      notes=object.notes,

@Fred-sun Fred-sun added ready_for_review The PR has been modified and can be reviewed and merged and removed work in In trying to solve, or in working with contributors labels Mar 13, 2025
@Matbefn
Copy link
Contributor Author

Matbefn commented Mar 13, 2025

@Fred-sun I have added the notes field to application_info

@xuzhang3 xuzhang3 merged commit f5dfb08 into ansible-collections:dev Mar 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

medium_priority Medium priority ready_for_review The PR has been modified and can be reviewed and merged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants