Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

OCTOPUS-625:modified files and names and var files as per ansible-lint #18

Merged
merged 1 commit into from
Mar 14, 2024

Conversation

pkenchap
Copy link
Collaborator

This PR contains : Below changes made as per Ansible-lint .

  • Changes made to filename.
  • Changes made to playbook with adding the hosts parameter .
  • Changes made to variable name as per the role name .

@pkenchap
Copy link
Collaborator Author

pkenchap commented Mar 14, 2024

Ansible lint action need collection installation the openstack packages .
Because its passing in my local:

punithkenchappa@Puniths-MacBook-Pro-2 ocp4-upi-multiarch-compute % ansible-lint

Passed: 0 failure(s), 0 warning(s) on 63 files. Last profile that met the validation criteria was 'production'.
punithkenchappa@Puniths-MacBook-Pro-2 ocp4-upi-multiarch-compute %

@pkenchap pkenchap self-assigned this Mar 14, 2024
@pkenchap pkenchap requested a review from prb112 March 14, 2024 10:28
@prb112 prb112 merged commit 5fa143d into ocp-power-automation:main Mar 14, 2024
0 of 2 checks passed
@prb112
Copy link
Collaborator

prb112 commented Mar 14, 2024

/lgtm

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants