31.2.1 release changes

[naresh-broadcom]

This PR resolves multiple ansible-lint, Python lint, and documentation issues identified during the Red Hat certification review process. The fixes were implemented based on the Galaxy importer results and guidance provided by the Red Hat team to ensure the collection meets certification requirements for release.

Changes Included

Ansible Lint Fixes

• Replaced short module names with Fully Qualified Collection Names (FQCN) such as ansible.builtin.shell, ansible.builtin.command, ansible.builtin.service, ansible.builtin.copy, ansible.builtin.stat, ansible.builtin.template, and others.
• Added missing task names and improved task naming conventions.
• Updated task key ordering according to ansible-lint recommendations.
• Replaced deprecated include statements with ansible.builtin.include_tasks or ansible.builtin.import_tasks.
• Replaced shell with command where shell functionality was not required.
• Added changed_when conditions to ensure idempotent task execution.
• Fixed risky shell pipe usage and improved Jinja formatting in shell commands.
• Replaced free-form module usage with structured parameters.
• Corrected YAML formatting issues including indentation, spacing, comment formatting, and newline requirements.

Python Lint Fixes

• Updated logging statements to use lazy % formatting instead of string interpolation.
• Removed unused imports detected by the linter.
• Fixed undefined variable issues by adding missing imports (e.g., urlparse).

Molecule Test Cleanup

• Addressed lint warnings in molecule/default test scenarios used for CI validation.

Validation

• The updated collection artifact was validated locally using galaxy-importer following the testing procedure recommended by the Red Hat team.

FT