Skip to content

Ansible enhanced descriptions and documentation#31

Merged
whitemerch merged 15 commits intomainfrom
chakib.hamie/ansible_descriptions_and_doc
Mar 27, 2026
Merged

Ansible enhanced descriptions and documentation#31
whitemerch merged 15 commits intomainfrom
chakib.hamie/ansible_descriptions_and_doc

Conversation

@whitemerch
Copy link
Copy Markdown
Contributor

@whitemerch whitemerch commented Mar 10, 2026

Tip

This PR is best reviewed commit by commit.

Motivation

This PR enhances the description of the Ansible rules that we are planning on adding support for, and generates documentation for them

Changes

  • Descriptions: Enhanced rule descriptions with secure examples and remediation guidance for Ansible
  • Documentation: Generated rule documentation under documentation/rules/ansible/ for Ansible

Author Checklist

  • I have reviewed my own PR.
  • I have added or updated relevant unit tests where necessary. If no tests are added, I've explained why.
  • All new and existing tests pass.
  • I have tested my changes on staging (if applicable).

QA Instruction

This PR should be acted upon as a description review PR.

Blast Radius

Datadog IaC Scanner only: new Ansible rules documentation; no changes to other platforms.

Additional Notes

I submit this contribution under the Apache-2.0 license.

@whitemerch whitemerch changed the base branch from main to chakib.hamie/ansible_fix_resource March 10, 2026 07:49
@whitemerch whitemerch changed the title Ansible support: rules, resource naming, descriptions, and documentation Ansible enhanced descriptions and documentation Mar 10, 2026
@datadog-prod-us1-3

This comment has been minimized.

@whitemerch whitemerch marked this pull request as ready for review March 10, 2026 08:52
@whitemerch whitemerch requested review from a team as code owners March 10, 2026 08:52
Base automatically changed from chakib.hamie/ansible_fix_resource to main March 10, 2026 08:58
@drichards-87 drichards-87 added the editorial review Docs Team editorial review label Mar 10, 2026
@drichards-87 drichards-87 self-requested a review March 10, 2026 15:54
@whitemerch whitemerch force-pushed the chakib.hamie/ansible_descriptions_and_doc branch from 6e9be3c to 078ef47 Compare March 11, 2026 07:21
drichards-87
drichards-87 previously approved these changes Mar 17, 2026
@whitemerch whitemerch force-pushed the chakib.hamie/ansible_descriptions_and_doc branch from 0426542 to 5c6aaba Compare March 19, 2026 18:37
@whitemerch whitemerch force-pushed the chakib.hamie/ansible_descriptions_and_doc branch from 607adf8 to 2453a89 Compare March 20, 2026 09:41
ChouraquiBen
ChouraquiBen previously approved these changes Mar 20, 2026
Copy link
Copy Markdown
Contributor

@ChouraquiBen ChouraquiBen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I checked quickly the changes over the files, that all the changes were inside the scope of this PR, it looks good to me

@whitemerch whitemerch marked this pull request as draft March 20, 2026 14:26
@whitemerch whitemerch marked this pull request as ready for review March 20, 2026 14:36
@whitemerch whitemerch force-pushed the chakib.hamie/ansible_descriptions_and_doc branch from 2453a89 to e7c3d69 Compare March 20, 2026 14:53
Copy link
Copy Markdown
Contributor

@ChouraquiBen ChouraquiBen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Thanks for fixing the sync metadata workflow!

@whitemerch whitemerch mentioned this pull request Mar 20, 2026
5 tasks
@whitemerch whitemerch merged commit 9bbf309 into main Mar 27, 2026
19 checks passed
@whitemerch whitemerch deleted the chakib.hamie/ansible_descriptions_and_doc branch March 27, 2026 09:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

editorial review Docs Team editorial review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants