add install_method to vagrantfile and travis

proper naming of vm hostname

rename hostname in inventory as well

.travis.yml

@@ -9,6 +9,12 @@ install:
 script:
   - echo localhost > inventory
   - ansible-playbook --syntax-check -i inventory test.yml
+  - ansible-playbook -i inventory -e nodejs_install_method=binary test.yml --connection=local --sudo
+  - >
+    ansible-playbook -i inventory -e nodejs_install_method=binary test.yml --connection=local --sudo
+    | grep -q 'changed=0.*failed=0'
+    && (echo 'Idempotence test: pass' && exit 0)
+    || (echo 'Idempotence test: fail' && exit 1)
   - ansible-playbook -i inventory test.yml --connection=local --sudo
   # Taken from https://servercheck.in/blog/testing-ansible-roles-travis-ci-github
   # Run the role/playbook again, checking to make sure it's idempotent.

README.md

@@ -5,8 +5,7 @@ Ansible role for installing nodejs, from package or by building it from source.
 
 #### Requirements & Dependencies
 - Tested on Ansible 1.4 or higher.
-- Depends on ANXS.build-essential
-
+- Depends on ANXS.build-essential when using install_method of "source"
 
 #### Variables
 

Vagrantfile

@@ -1,17 +1,22 @@
 # -*- mode: ruby -*-
 # vi: set ft=ruby :
 
+nodejs_install_method = ENV.key?('NODEJS_INSTALL_METHOD') ? ENV['NODEJS_INSTALL_METHOD'] : 'source'
+
 Vagrant.configure('2') do |config|
   config.vm.define 'anxs' do |c|
     c.vm.box = 'anxs-vbox-linux'
-    c.vm.box_url = "http://cloud-images.ubuntu.com/vagrant/precise/current/precise-server-cloudimg-amd64-vagrant-disk1.box"
-    c.vm.network :private_network, ip: "192.168.88.2"
-    c.vm.hostname = "anxs-nginx.local"
-    c.vm.provision "ansible" do |ansible|
-      ansible.playbook = "test.yml"
+    c.vm.box_url = 'http://cloud-images.ubuntu.com/vagrant/precise/current/precise-server-cloudimg-amd64-vagrant-disk1.box'
+    c.vm.network :private_network, ip: '192.168.88.2'
+    c.vm.hostname = 'anxs.local'
+    c.vm.provision 'ansible' do |ansible|
+      ansible.playbook = 'test.yml'
       ansible.sudo = true
-      ansible.inventory_path = "vagrant-inventory"
+      ansible.inventory_path = 'vagrant-inventory'
       ansible.host_key_checking = false
+      ansible.extra_vars = {
+        nodejs_install_method: nodejs_install_method
+      }
     end
   end
 end

vagrant-inventory

@@ -1,2 +1,2 @@
 [anxs]
-anxs-nodejs.local ansible_ssh_host=192.168.88.2 ansible_ssh_port=22
+anxs.local ansible_ssh_host=192.168.88.2 ansible_ssh_port=22