search

alertlogic / al-agents-ansible-playbooks

Ansible Playbooks to install and provision the Alert Logic Unified Agent
Apache License 2.0
8 stars 14 forks source link

provision agent yml fix #28

Closed gallernd closed 4 years ago

gallernd commented 5 years ago

I could not run the install/provision agent task. After looking at the code, I found a small issue.

Please take a look at this PR, as I hope it will be acceptable to you.

sparkacus commented 4 years ago

Wish I had looked here before spending my time debugging and working on a fix to find someone had already done it :)

eexit commented 4 years ago

I confirm this fix works. Please merge when you get a chance. Thanks :)

muram commented 4 years ago

Added user story to test/review and merge. TAT: < 1 week

muram commented 4 years ago

All checks passed (working on re-activating travis-ci for future PRs checks)

PLAY RECAP ***********************************************************************************************************
i-023b5bf3f9da04df6        : ok=13   changed=5    unreachable=0    failed=0    skipped=24   rescued=0    ignored=0
i-0364d8ba4b6429114        : ok=15   changed=7    unreachable=0    failed=0    skipped=23   rescued=0    ignored=0
i-082f300e4a7d6e2d3        : ok=15   changed=7    unreachable=0    failed=0    skipped=23   rescued=0    ignored=0
===== LOADING ROLE ===== 
Checking role metadata tags 
Checking role platforms 
Checking role cloud platforms 
Checking role dependencies 

===== LINTING ROLE ===== 
yamllint OK. 
ansible-lint OK. 

===== IMPORTING ROLE: al_agents ===== 

Updating repository versions... 
Import completed with 0 warnings and 0 errors
muram commented 4 years ago

Thanks @gallernd this looks good.