fix: use correct path for wazuh-ansible role

[jackhodgkiss]

No description provided.

[coderabbitai]

Important

Review skipped

Auto incremental 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: stackhpc/stackhpc-kayobe-config/.coderabbit.yaml
• Review profile: CHILL
• Plan: Advanced
• Run ID: 8c140728-9b21-4fe0-be33-5411a4d1225d

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

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Team

Run ID: a8ea482e-5645-432d-8b07-fe1adeb4b84b

📥 Commits

Reviewing files that changed from the base of the PR and between 7e454bb and 473fffe.

📒 Files selected for processing (3)

• etc/kayobe/ansible/deployment/wazuh-agent.yml
• etc/kayobe/ansible/deployment/wazuh-manager.yml
• releasenotes/notes/fix-wazuh-role-path-c9984184fac52f43.yaml

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.

📜 Recent review details

⏰ Context from checks skipped due to timeout. (2)

• GitHub Check: Ansible 2.19 lint with Python 3.12
• GitHub Check: Ansible 2.20 lint with Python 3.12

🔇 Additional comments (3)

etc/kayobe/ansible/deployment/wazuh-agent.yml (1)

8-8: LGTM!

etc/kayobe/ansible/deployment/wazuh-manager.yml (1)

41-41: LGTM!

Also applies to: 52-55

releasenotes/notes/fix-wazuh-role-path-c9984184fac52f43.yaml (1)

1-4: LGTM!

📝 Walkthrough

The deployment playbooks now use corrected Wazuh Ansible role paths. A release note documents the path correction.

Changes: Wazuh role path correction

Layer / File(s)	Summary
Update Wazuh role paths etc/kayobe/ansible/deployment/wazuh-agent.yml, etc/kayobe/ansible/deployment/wazuh-manager.yml, releasenotes/notes/fix-wazuh-role-path-c9984184fac52f43.yaml	The agent and manager playbooks now use wazuh-ansible/roles paths. The release note records the correction.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Merge Risk: ⚪ Minimal · up to 473ff

The Wazuh agent and manager playbooks now load roles from the correct dependency path, restoring deployment role resolution without introducing an identified merge-blocking risk.

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

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.}

[jackhodgkiss]

Appears to mismatch between linting and what works in kayobe which is probably how the error was introduced in the first place.

ERROR! the role 'wazuh-ansible/wazuh-ansible/roles/wazuh/wazuh-indexer' was not found in /usr/home/jph/deployment/src/kayobe-config/etc/kayobe/ansible/deployment/roles:/usr/home/jph/deployment/src/kayobe-config/etc/kayobe/ansible/roles:/usr/home/jph/deployment/venvs/kayobe/share/kayobe/ansible/roles:/usr/home/jph/deployment/src/kayobe-config/etc/kayobe/ansible/deployment

The error appears to be in '/usr/home/jph/deployment/src/kayobe-config/etc/kayobe/ansible/deployment/wazuh-manager.yml': line 41, column 7, but may
be elsewhere in the file depending on the exact syntax problem.

The offending line appears to be:

  roles:
    - role: wazuh-ansible/wazuh-ansible/roles/wazuh/wazuh-indexer
      ^ here

[priteau]

@jackhodgkiss Is there a way we can keep the lint jobs happy?

[github-actions]

Happy Friday @jackhodgkiss, this is a friendly reminder that this PR is waiting for your changes or response. Please take a look when you have a moment!

Note: Once your changes are ready, remove the waiting-author-response label and add the waiting-review label.

[jackhodgkiss]

@jackhodgkiss Is there a way we can keep the lint jobs happy?

I can investigate however the linting isn't useful if it breaks actually functionality.

[Alex-Welsh]

@jackhodgkiss Is there a way we can keep the lint jobs happy?

I can investigate however the linting isn't useful if it breaks actually functionality.

So are the linters actually incorrect here?

You can ignore the file if needed. We've done that in the past for vault-related playbooks