feat: Argument spec implementation for logging role

[DonatSzabo]

Enhancement: Added argument spec and assert role spec validation to the ad integration role. Also wrote tests for it found in tests/tests_invalid_input.

Reason: Because it is a good addition to the linux-system-roles project.

Result: Successfully added it and prepared tests for it. I used AI during this implementation.

Issue Tracker Tickets (Jira or BZ if any): linux-system-roles/postfix#206 https://redhat.atlassian.net/browse/RHELMISC-16008

Notes: logging_domain now defaults to an empty string and is filled from Ansible facts in tasks/set_vars.yml only when it is still empty, so the argument spec can declare it as a plain string.

[github-actions]

CI tests do not run automatically on pull requests. A role repository
maintainer can start them by posting a /citest slash command in a
pull request comment.

See GitHub CI testing using /citest
for details.

Run every available CI workflow:

/citest all

Run the linting and other lightweight checks:

/citest linters

Run the integration tests (QEMU/container and Testing Farm):

/citest integration

Run one or more selected workflows by separating their names with spaces:

/citest ansible-lint
/citest ansible-lint markdownlint

Command	Check name	Description
/citest all	All checks listed below	Run every CI test available for this role
/citest linters	Lint and lightweight checks	Run ansible-lint, ansible-test, ansible-managed-var-comment, codespell, markdownlint, pr-title-lint, test_converting_readme, and codeql, python-unit-test, and shellcheck when those workflows exist
/citest integration	QEMU/container and Testing Farm checks	Run qemu-kvm-integration-tests and tft
/citest ansible-lint	Ansible Lint / ansible_lint (<ansible-lint>, <ansible>, <python>) (pull_request)	Lint Ansible content after converting the role to collection format
/citest ansible-managed-var-comment	Check for ansible_managed variable use in comments / ansible_managed_var_comment (pull_request)	Fail if ansible_managed is used in comments
/citest ansible-test	Ansible Test / ansible_test (<ansible>, <python>) (pull_request)	Run ansible-test sanity tests
/citest codespell	Codespell / Check for spelling errors (pull_request)	Check for spelling errors
/citest markdownlint	Markdown Lint / markdownlint (pull_request)	Lint Markdown files
/citest pr-title-lint	PR Title Lint / commit-checks	Check that the pull request title follows the required format
/citest qemu-kvm-integration-tests	Test / scenario (<image>, <env>) (pull_request)	Run role integration tests in QEMU VMs and containers
/citest test_converting_readme	Test converting README.md to README.html / test_converting_readme (pull_request)	Convert README.md to HTML
/citest tft	<platform>|ansible-<version>	Run integration tests in Testing Farm
/citest woke	Woke / Detect non-inclusive language (pull_request)	Detect non-inclusive language

Post another /citest comment at any time to run another selection.

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

• Configuration used: Repository: linux-system-roles/logging/.coderabbit.yaml
• Review profile: CHILL
• Plan: Advanced
• Run ID: 52c6c389-d0fd-4117-8bc1-e306fede3b6d

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

• Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[DonatSzabo]

/citest all

[richm]

/citest all