Update ansible lint configuration

[arkid15r]

Proposed change

Fix ansible local linter

Checklist

• Required: I followed the contributing workflow
• Required: I verified that my code works as intended and resolves the issue as described
• Required: I ran make check-test locally: all warnings addressed, tests passed
• I used AI for code, documentation, tests, or communication related to this PR

[coderabbitai]

Summary by CodeRabbit

• Chores
  • Updated deployment pipeline configuration to reference updated dependency file formats
  • Enhanced linting configuration with additional dependencies and configuration settings
  • Streamlined continuous integration workflow processes

Walkthrough

Updates CI/CD configuration files to enhance Ansible linting and deployment workflows. Adds ansible-lint offline flag, updates Ansible Galaxy file references from requirements.yaml to requirements.yml across deployment jobs, removes a pre-commit collection installation step, and configures pre-commit ansible-lint hook with additional dependencies and custom config path.

Changes

Cohort / File(s)	Summary
Ansible Lint Configuration .github/ansible/.ansible-lint.yaml	Added offline: false configuration entry to ansible-lint settings.
CI/CD Workflow .github/workflows/run-ci-cd.yaml	Removed pre-commit job's Ansible collection installation step; updated Ansible Galaxy file references from requirements.yaml to requirements.yml in deploy-staging-nest, deploy-staging-nest-proxy, deploy-production-nest, and deploy-production-nest-proxy jobs.
Pre-commit Configuration .pre-commit-config.yaml	Added additional_dependencies: ansible-core to ansible-lint hook and introduced -c .github/ansible/.ansible-lint.yaml config argument.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

• CI: Add ansible-lint workflow for Ansible playbooks #3796: Directly overlaps with ansible-lint configuration, requirements.yml usage, and pre-commit hook setup changes.

Suggested labels

ci

Suggested reviewers

• kasya

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title 'Update ansible lint configuration' directly matches the primary changes: updates to .ansible-lint.yaml, .github/workflows/run-ci-cd.yaml, and .pre-commit-config.yaml to fix ansible linting configuration.
Description check	✅ Passed	The description 'Fix ansible local linter' is directly related to the changeset, which focuses on updating Ansible lint configuration and fixing linter issues.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch ark/update-ansible

---

No actionable comments were generated in the recent review. 🎉

🧹 Recent nitpick comments

.github/ansible/.ansible-lint.yaml (1)

1-1: Minimal but functional config anchor.

offline: false is already the default for ansible-lint, but having this file as an explicit config target for the -c flag in the pre-commit hook is reasonable. Consider adding a brief comment explaining the file's purpose or noting that additional rules can be configured here in the future.

---

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 and usage tips.}

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[cubic-dev-ai]

No issues found across 4 files

Confidence score: 5/5

• Automated review surfaced no issues in the provided summaries.
• No files require special attention.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.13%. Comparing base (aff1dee) to head (dfa9c4e).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #3880   +/-   ##
=======================================
  Coverage   93.13%   93.13%           
=======================================
  Files         463      463           
  Lines       14531    14531           
  Branches     2061     2061           
=======================================
  Hits        13533    13533           
  Misses        538      538           
  Partials      460      460           

Flag	Coverage Δ
backend	95.65% <ø> (ø)
frontend	86.28% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

---

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update aff1dee...dfa9c4e. Read the comment docs.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.