phred
Tweaks to apt-get upgrade and ssh restart
Thanks for writing up your blog post and example playbook.
Coincidentally, before the original "first 5 minutes" post went up, I'd been looking at using ansible to secure some new VPS servers (as well as configure them/deploy apps.)
In playing with your script in vagrant I found a few minor issues which seemed worth sharing back.
Glad you liked the article, Joshua! I'll take a look at this patch. Thank you for trying it out and fixing it for me. :)
Fred Alger @_phred
On Thursday, 14 March 2013 at 12:15, Joshua Barratt wrote:
Thanks for writing up your blog post and example playbook. Coincidentally, before the original "first 5 minutes" post went up, I'd been looking at using ansible to secure some new VPS servers (as well as configure them/deploy apps.) In playing with your script in vagrant I found a few minor issues which seemed worth sharing back. You can merge this Pull Request by running git pull https://github.com/jbarratt/5minbootstrap patch-1 Or view, comment on, or merge it at: https://github.com/phred/5minbootstrap/pull/3 Commit Summary Tweaks to apt-get upgrade and ssh restart
File Changes M bootstrap.yml (https://github.com/phred/5minbootstrap/pull/3/files#diff-0) (10)
Patch Links: https://github.com/phred/5minbootstrap/pull/3.patch https://github.com/phred/5minbootstrap/pull/3.diff
For sure!
Also meant to mention I'd been finding the newer ansible syntax where you use the actual command name much more readable. Not sure if you were using the older one intentionally or not.
As in
- name: Update APT package cache apt: update_cache=yes
instead of
- name: Update APT package cache action: apt update_cache=yes
See what I get for staying out of the loop on Ansible developments for a few months? Glad you're here to school me, I agree that it's much more readable. :-)
Fred Alger @_phred
On Thursday, 14 March 2013 at 20:31, Joshua Barratt wrote:
For sure!
Also meant to mention I'd been finding the newer ansible syntax where you use the actual command name much more readable. Not sure if you were using the older one intentionally or not.
As in
- name: Update APT package cache
apt: update_cache=yesinstead of
- name: Update APT package cache
action: apt update_cache=yes— Reply to this email directly or view it on GitHub (https://github.com/phred/5minbootstrap/pull/3#issuecomment-14937831).