taperole 1.2.8 → 1.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: b222a5a4cae4120e81b66cb9000fe978e757a8c1
-  data.tar.gz: dd1609ccb8a53d633fed08b303fe00285f2c3c5e
+  metadata.gz: 32498ae112df7133fcf2c96346e0665e0c4d139b
+  data.tar.gz: ea3f9bc72d365df38432cbacfc3166bd82c3b054
 SHA512:
-  metadata.gz: 713a7ff37a753a09d8a0a4a449fce988360b958d437c63736e78879fc5f263a7327670949736825402f68e42a44c08d06f9d0b6bac6f9f8bcb653a30c2ff1e2f
-  data.tar.gz: dc899db4d9cefe5439f57a0925743449d64795599a21a74d1806eb2d5aca774ce9c75b255ba14477749b9d8c1e647531a3f1bf501746914b9894a2088d8147a4
+  metadata.gz: be6c223a0f1aee40640b6b8f80c37534915953034eef3884a76876ad8e1bb9dcf65a4d2b5d45d19cd9646dbdd0215c8fa17dedcf74542a4e3f64cd421cac000e
+  data.tar.gz: 69d9871011dff3e0522b3dc335146e982f97e83add1343263df1542d14f0df2d44ecec9d19b8e4e8902d59d8492b3538c52802b427cd10b365850be997557b3a

data/.hound.yml

@@ -0,0 +1,7 @@
+fail_on_violations: true
+
+ruby:
+  config_file: config/style_guides/ruby.yml
+
+AllCops:
+  RunRailsCops: false

data/CHANGELOG.md

@@ -1,3 +1,15 @@
+### 1.3.0
+* Puts all tape focused files for a repo into a taperole/ directory
+* Installs mosh
+* Installs htop
+* Removes vagrant runner
+* Auto-detect dev_keys
+* Only try to fe_deploy if fe_app_repo is defined
+* Control nginx with monit
+* Updates Readme
+* Disable retry files
+* tape will not check that requires vars are defined before proceeding with provisioning
+
 ### 1.2.3
 * Fixed issue where users who were using vagrant could not ssh into the deployer user
 * Fixed issue where vagrant boxes init script was failing because .ssh dir already existed 

data/README.md

@@ -1,6 +1,8 @@
 [![Stories in Ready](https://badge.waffle.io/smashingboxes/taperole.png?label=ready&title=Ready)](https://waffle.io/smashingboxes/tape)
 # Infrastructure Management
 
+[![Join the chat at https://gitter.im/smashingboxes/taperole](https://badges.gitter.im/Join%20Chat.svg)](https://gitter.im/smashingboxes/taperole?utm_source=badge&utm_medium=badge&utm_campaign=pr-badge&utm_content=badge)
+
 ## Deploying & provisioning with tape
 **Use Unbuntu precise64 (12.04 x64)** 
 
@@ -36,7 +38,7 @@ tape installer install
 ```
 
 ### Custom roles
-You can write app specific roles in the roles files storred in the `roles` directory
+You can write app specific roles in the roles files stored in the `roles` directory
 
 You must then specify the roles you want to use in `omnibox.yml` or `deploy.yml`
 
@@ -55,7 +57,7 @@ production
 staging
 ```
 
-then use the `-l` option to specify the staging
+Then use the `-l` option to specify the staging
 
 ```sh
 tape ansible deploy -l staging
@@ -65,31 +67,20 @@ tape ansible deploy -l staging
 ### With vagrant
 
 
-1. `vagrant up` or `tape vagrant create`
+1. `vagrant up`
 2. Put the following into your [hosts inventory file](http://docs.ansible.com/intro_inventory.html):
 
 ```
 [vagrant]
-<192.168.13.37> ansible_ssh_private_key_file=~/.vagrant.d/insecure_private_key
+localhost:2222 ansible_ssh_private_key_file=~/.vagrant.d/insecure_private_key
 ```
 
 The port number might be different if other vagrant machines are running, run `vagrant ssh-config`  to find the correct configuration.
-You can speicfy a port using the `ansible_ssh_port` in your hosts inventory file.
+You can specify a port using the `ansible_ssh_port` in your hosts inventory file.
 
 3. Update `tape_vars.yml` with information to a [rails app you want to deploy](https://github.com/BrandonMathis/vanilla-rails-app)
 4. `tape ansible everything -l vagrant`
 
-
-### With QEMU
-
-1. `tape qemu create --name fe_test`
-2. `tape qemu start --name fe_test -p2255`
-3. `ssh-add ./id_rsa_sb_basebox`
-4. `echo 'localhost:2255' >test_hosts`
-5. `tape ansible everything`
-
-Run `tape -h` for a quick rundown of the tool's modules and options.
-
 ## Development
 
 ```sh

data/config/style_guides/ruby.yml

@@ -0,0 +1,238 @@
+AllCops:
+  Exclude:
+    - "vendor/**/*"
+    - "db/schema.rb"
+  UseCache: false
+Style/CollectionMethods:
+  Description: Preferred collection methods.
+  StyleGuide: https://github.com/bbatsov/ruby-style-guide#map-find-select-reduce-size
+  Enabled: false
+  PreferredMethods:
+    collect: map
+    collect!: map!
+    find_all: select
+    reduce: inject
+Style/DotPosition:
+  Description: Checks the position of the dot in multi-line method calls.
+  StyleGuide: https://github.com/bbatsov/ruby-style-guide#consistent-multi-line-chains
+  Enabled: true
+  EnforcedStyle: trailing
+  SupportedStyles:
+  - leading
+  - trailing
+Style/FileName:
+  Description: Use snake_case for source file names.
+  StyleGuide: https://github.com/bbatsov/ruby-style-guide#snake-case-files
+  Enabled: false
+  Exclude: []
+Style/GuardClause:
+  Description: Check for conditionals that can be replaced with guard clauses
+  StyleGuide: https://github.com/bbatsov/ruby-style-guide#no-nested-conditionals
+  Enabled: false
+  MinBodyLength: 1
+Style/IfUnlessModifier:
+  Description: Favor modifier if/unless usage when you have a single-line body.
+  StyleGuide: https://github.com/bbatsov/ruby-style-guide#if-as-a-modifier
+  Enabled: false
+  MaxLineLength: 100
+Style/OptionHash:
+  Description: Don't use option hashes when you can use keyword arguments.
+  Enabled: false
+Style/PercentLiteralDelimiters:
+  Description: Use `%`-literal delimiters consistently
+  StyleGuide: https://github.com/bbatsov/ruby-style-guide#percent-literal-braces
+  Enabled: false
+  PreferredDelimiters:
+    "%": "()"
+    "%i": "()"
+    "%q": "()"
+    "%Q": "()"
+    "%r": "{}"
+    "%s": "()"
+    "%w": "()"
+    "%W": "()"
+    "%x": "()"
+Style/PredicateName:
+  Description: Check the names of predicate methods.
+  StyleGuide: https://github.com/bbatsov/ruby-style-guide#bool-methods-qmark
+  Enabled: true
+  NamePrefix:
+  - is_
+  - has_
+  - have_
+  NamePrefixBlacklist:
+  - is_
+  Exclude:
+  - spec/**/*
+Style/RaiseArgs:
+  Description: Checks the arguments passed to raise/fail.
+  StyleGuide: https://github.com/bbatsov/ruby-style-guide#exception-class-messages
+  Enabled: false
+  EnforcedStyle: exploded
+  SupportedStyles:
+  - compact
+  - exploded
+Style/SignalException:
+  Description: Checks for proper usage of fail and raise.
+  StyleGuide: https://github.com/bbatsov/ruby-style-guide#fail-method
+  Enabled: false
+  EnforcedStyle: semantic
+  SupportedStyles:
+  - only_raise
+  - only_fail
+  - semantic
+Style/SingleLineBlockParams:
+  Description: Enforces the names of some block params.
+  StyleGuide: https://github.com/bbatsov/ruby-style-guide#reduce-blocks
+  Enabled: false
+  Methods:
+  - reduce:
+    - a
+    - e
+  - inject:
+    - a
+    - e
+Style/SingleLineMethods:
+  Description: Avoid single-line methods.
+  StyleGuide: https://github.com/bbatsov/ruby-style-guide#no-single-line-methods
+  Enabled: false
+  AllowIfMethodIsEmpty: true
+Style/StringLiterals:
+  Enabled: false
+Style/StringLiteralsInInterpolation:
+  Description: Checks if uses of quotes inside expressions in interpolated strings
+    match the configured preference.
+  Enabled: false
+  EnforcedStyle: single_quotes
+  SupportedStyles:
+  - single_quotes
+  - double_quotes
+Style/TrailingComma:
+  Description: Checks for trailing comma in parameter lists and literals.
+  StyleGuide: https://github.com/bbatsov/ruby-style-guide#no-trailing-array-commas
+  Enabled: false
+  EnforcedStyleForMultiline: no_comma
+  SupportedStyles:
+  - comma
+  - no_comma
+Metrics/AbcSize:
+  Description: A calculated magnitude based on number of assignments, branches, and
+    conditions.
+  Enabled: false
+  Max: 15
+Metrics/ClassLength:
+  Description: Avoid classes longer than 100 lines of code.
+  Enabled: false
+  CountComments: false
+  Max: 100
+Metrics/ModuleLength:
+  CountComments: false
+  Max: 100
+  Description: Avoid modules longer than 100 lines of code.
+  Enabled: false
+Metrics/CyclomaticComplexity:
+  Description: A complexity metric that is strongly correlated to the number of test
+    cases needed to validate a method.
+  Enabled: false
+  Max: 6
+Metrics/MethodLength:
+  Description: Avoid methods longer than 10 lines of code.
+  StyleGuide: https://github.com/bbatsov/ruby-style-guide#short-methods
+  Enabled: false
+  CountComments: false
+  Max: 10
+Metrics/ParameterLists:
+  Description: Avoid parameter lists longer than three or four parameters.
+  StyleGuide: https://github.com/bbatsov/ruby-style-guide#too-many-params
+  Enabled: false
+  Max: 5
+  CountKeywordArgs: true
+Metrics/PerceivedComplexity:
+  Description: A complexity metric geared towards measuring complexity for a human
+    reader.
+  Enabled: false
+  Max: 7
+Metrics/LineLength:
+  Max: 100
+  # To make it possible to copy or click on URIs in the code, we allow lines
+  # contaning a URI to be longer than Max.
+  AllowURI: true
+  URISchemes:
+    - http
+    - https
+Lint/AssignmentInCondition:
+  Description: Don't use assignment in conditions.
+  StyleGuide: https://github.com/bbatsov/ruby-style-guide#safe-assignment-in-condition
+  Enabled: false
+  AllowSafeAssignment: true
+Style/ClassAndModuleChildren:
+  Enabled: false
+Style/InlineComment:
+  Description: Avoid inline comments.
+  Enabled: false
+Style/AccessorMethodName:
+  Description: Check the naming of accessor methods for get_/set_.
+  Enabled: fals
+Style/Alias:
+  Description: Use alias_method instead of alias.
+  StyleGuide: https://github.com/bbatsov/ruby-style-guide#alias-method
+  Enabled: false
+Style/Documentation:
+  Description: Document classes and non-namespace modules.
+  Enabled: false
+Style/DoubleNegation:
+  Description: Checks for uses of double negation (!!).
+  StyleGuide: https://github.com/bbatsov/ruby-style-guide#no-bang-bang
+  Enabled: false
+Style/EachWithObject:
+  Description: Prefer `each_with_object` over `inject` or `reduce`.
+  Enabled: false
+Style/EmptyLiteral:
+  Description: Prefer literals to Array.new/Hash.new/String.new.
+  StyleGuide: https://github.com/bbatsov/ruby-style-guide#literal-array-hash
+  Enabled: false
+Style/ModuleFunction:
+  Description: Checks for usage of `extend self` in modules.
+  StyleGuide: https://github.com/bbatsov/ruby-style-guide#module-function
+  Enabled: false
+Style/OneLineConditional:
+  Description: Favor the ternary operator(?:) over if/then/else/end constructs.
+  StyleGuide: https://github.com/bbatsov/ruby-style-guide#ternary-operator
+  Enabled: false
+Style/PerlBackrefs:
+  Description: Avoid Perl-style regex back references.
+  StyleGuide: https://github.com/bbatsov/ruby-style-guide#no-perl-regexp-last-matchers
+  Enabled: false
+Style/Send:
+  Description: Prefer `Object#__send__` or `Object#public_send` to `send`, as `send`
+    may overlap with existing methods.
+  StyleGuide: https://github.com/bbatsov/ruby-style-guide#prefer-public-send
+  Enabled: false
+Style/SpecialGlobalVars:
+  Description: Avoid Perl-style global variables.
+  StyleGuide: https://github.com/bbatsov/ruby-style-guide#no-cryptic-perlisms
+  Enabled: false
+Style/VariableInterpolation:
+  Description: Don't interpolate global, instance and class variables directly in
+    strings.
+  StyleGuide: https://github.com/bbatsov/ruby-style-guide#curlies-interpolate
+  Enabled: false
+Style/WhenThen:
+  Description: Use when x then ... for one-line cases.
+  StyleGuide: https://github.com/bbatsov/ruby-style-guide#one-line-cases
+  Enabled: false
+Lint/EachWithObjectArgument:
+  Description: Check for immutable argument given to each_with_object.
+  Enabled: true
+Lint/HandleExceptions:
+  Description: Don't suppress exception.
+  StyleGuide: https://github.com/bbatsov/ruby-style-guide#dont-hide-exceptions
+  Enabled: false
+Lint/LiteralInCondition:
+  Description: Checks of literals used in conditions.
+  Enabled: false
+Lint/LiteralInInterpolation:
+  Description: Checks for literals used in interpolation.
+  Enabled: false
+Rails/Output:
+  Enabled: false

data/lib/tape/ansible_runner.rb

@@ -4,45 +4,45 @@ class AnsibleRunner < ExecutionModule
   TapeBoxer.register_module :ansible, self
 
   action :configure_dj_runner,
-    proc {ansible '-t configure_dj_runner -e force_dj_runner_restart=true'},
-    "Configures and restarts the delayed job runner"
+         proc { ansible '-t configure_dj_runner -e force_dj_runner_restart=true' },
+         "Configures and restarts the delayed job runner"
   action :restart_unicorn,
-    proc {ansible '-t unicorn_restart'}, 
-    "Restarts the unicorns running on the app servers"
+         proc { ansible '-t unicorn_restart' },
+         "Restarts the unicorns running on the app servers"
   action :stop_unicorn,
-    proc {ansible '-t unicorn_stop -e kill_unicorn=true'}, 
-    "Stops the unicorns running on the app servers"
+         proc { ansible '-t unicorn_stop -e kill_unicorn=true' },
+         "Stops the unicorns running on the app servers"
   action :force_stop_unicorn,
-    proc {ansible '-t unicorn_force_stop -e kill_unicorn=true'}, 
-    "Stops the unicorns running on the app servers"
+         proc { ansible '-t unicorn_force_stop -e kill_unicorn=true' },
+         "Stops the unicorns running on the app servers"
   action :start_unicorn,
-    proc {ansible '-t unicorn_start'},
-    "Starts the unicorns running on the app servers"
+         proc { ansible '-t unicorn_start' },
+         "Starts the unicorns running on the app servers"
   action :restart_nginx,
-    proc {ansible '-t restart_nginx'},
-    "Restarts Nginx"
+         proc { ansible '-t restart_nginx' },
+         "Restarts Nginx"
   action :configure_deployer_user,
-    proc {ansible '-t deployer'},
-    "Ensures the deployer user is present and configures his SSH keys"
+         proc { ansible '-t deployer' },
+         "Ensures the deployer user is present and configures his SSH keys"
   action :reset_db,
-    proc {ansible '-t db_reset -e force_db_reset=true'},
-    "wipes and re-seeds the DB"
+         proc { ansible '-t db_reset -e force_db_reset=true' },
+         "wipes and re-seeds the DB"
   action :bundle,
-    proc {ansible '-t bundle -e force_bundle=true'},
-    "Bundles the gems running on the app servers"
+         proc { ansible '-t bundle -e force_bundle=true' },
+         "Bundles the gems running on the app servers"
   action :be_deploy,
-    proc {ansible_deploy '-t be_deploy'},
-    "Re-deploys fe code"
+         proc { ansible_deploy '-t be_deploy' },
+         "Re-deploys fe code"
   action :fe_deploy,
-    proc {ansible_deploy '-t fe_deploy'},
-    "Re-deploys fe code"
+         proc { ansible_deploy '-t fe_deploy' },
+         "Re-deploys fe code"
   action :deploy,
-    proc {ansible_deploy '-t be_deploy,fe_deploy'},
-    "Checks out app code, installs dependencies and restarts unicorns for "\
-    "both FE and BE code."
+         proc { ansible_deploy '-t be_deploy,fe_deploy' },
+         "Checks out app code, installs dependencies and restarts unicorns for "\
+         "both FE and BE code."
   action :everything,
-    proc { ansible if valid_preconfigs },
-    "This does it all."
+         proc { ansible if valid_preconfigs },
+         "This does it all."
 
   def initialize(*args)
     super
@@ -53,7 +53,11 @@ class AnsibleRunner < ExecutionModule
 
   def valid_preconfigs
     if rails_app?
-      return valid_gems
+      valid_gems
+    elsif fe_app?
+      true
+    else
+      false
     end
   end
 
@@ -71,11 +75,11 @@ class AnsibleRunner < ExecutionModule
   end
 
   def ansible(cmd_str = '')
-    exec_ansible('omnibox.yml', cmd_str)
+    exec_ansible("#{tapefiles_dir}/omnibox.yml", cmd_str)
   end
 
   def ansible_deploy(cmd_str = '')
-    exec_ansible('deploy.yml', cmd_str)
+    exec_ansible("#{tapefiles_dir}/deploy.yml", cmd_str)
   end
 
   def exec_ansible(playbook, args)
@@ -89,9 +93,13 @@ class AnsibleRunner < ExecutionModule
 
   def enforce_roles_path!
     Dir.mkdir('.tape') unless Dir.exists?('.tape')
-    File.open('.tape/ansible.cfg', 'w') do |f|
+
+    File.open("#{local_dir}/.tape/ansible.cfg", 'w') do |f|
       f.puts '[defaults]'
-      f.puts "roles_path=./roles:#{tape_dir}/roles:#{tape_dir}/vendor"
+      f.puts "roles_path=.tape/roles:#{tape_dir}/roles:#{tape_dir}/vendor"
+      f.puts "inventory=#{tapefiles_dir}/hosts"
+      f.puts "retries-dir=/dev/null"
+      f.puts "retry_files_enabled = False"
       f.puts '[ssh_connection]'
       f.puts 'ssh_args = -o ForwardAgent=yes'
     end
@@ -102,7 +110,7 @@ class AnsibleRunner < ExecutionModule
   end
 
   def inventory_file
-    opts.inventory_file || "#{local_dir}/hosts"
+    opts.inventory_file || "#{tapefiles_dir}/hosts"
   end
 end
 end

data/lib/tape/installer.rb

@@ -3,11 +3,11 @@ module TapeBoxer
     TapeBoxer.register_module :installer, self
 
     action :install,
-      proc {install},
-      'Creates all nessisary hosts and config files'
+           proc { install },
+           'Creates all necessary hosts and config files'
     action :uninstall,
-      proc {uninstall},
-      'Cleans up files generated by the installer'
+           proc { uninstall },
+           'Cleans up files generated by the installer'
 
     def initialize(*args)
       super
@@ -17,7 +17,7 @@ module TapeBoxer
 
     def install
       File.open('.gitignore', 'r+') { |f| f.puts '.tape' unless f.read =~/^\.tape$/ }
-      mkdir 'roles'
+      mkdir tapefiles_dir
       if fe_app? && !rails_app?
         puts '🔎  JS/HTML app detected'.pink
         copy_static_app_examples
@@ -25,14 +25,28 @@ module TapeBoxer
         puts '🔎  Rails app detected'.pink
         copy_basic_examples
       end
-      copy_example 'templates/base/hosts.example', 'hosts'
-      mkdir 'dev_keys'
+      create_roles_dir
+      create_inventory_file
+      create_ssh_keys_dir
       print 'Are you going to use vagrant? (y/n): '
       if gets.chomp == 'y'
         copy_example 'Vagrantfile', 'Vagrantfile'
       end
     end
 
+    def create_roles_dir
+      mkdir "#{tapefiles_dir}/roles"
+      `touch #{tapefiles_dir}/roles/.keep`
+    end
+
+    def create_ssh_keys_dir
+      mkdir "#{tapefiles_dir}/dev_keys"
+    end
+
+    def create_inventory_file
+      copy_example 'templates/base/hosts.example', "#{tapefiles_dir}/hosts"
+    end
+
     def fe_app?
       !Dir["#{local_dir}/gulpfile.*"].empty?
     end
@@ -42,25 +56,43 @@ module TapeBoxer
     end
 
     def copy_static_app_examples
-      copy_example 'templates/static_html/omnibox.example.yml', 'omnibox.yml'
-      copy_example 'templates/static_html/deploy.example.yml', 'deploy.yml'
-      copy_example 'templates/static_html/tape_vars.example.yml', 'tape_vars.yml'
+      copy_example(
+        'templates/static_html/omnibox.example.yml',
+        "#{tapefiles_dir}/omnibox.yml"
+      )
+      copy_example(
+        'templates/static_html/deploy.example.yml',
+        "#{tapefiles_dir}/deploy.yml"
+      )
+      copy_example(
+        'templates/static_html/tape_vars.example.yml',
+        "#{tapefiles_dir}/tape_vars.yml"
+      )
     end
 
     def copy_basic_examples
-      copy_example 'templates/base/omnibox.example.yml', 'omnibox.yml'
-      copy_example 'templates/base/deploy.example.yml', 'deploy.yml'
-      copy_example 'templates/base/tape_vars.example.yml', 'tape_vars.yml'
+      copy_example(
+        'templates/base/omnibox.example.yml',
+        "#{tapefiles_dir}/omnibox.yml"
+      )
+      copy_example(
+        'templates/base/deploy.example.yml',
+        "#{tapefiles_dir}/deploy.yml"
+      )
+      copy_example(
+        'templates/base/tape_vars.example.yml',
+        "#{tapefiles_dir}/tape_vars.yml"
+      )
     end
 
     def uninstall
-      rm 'omnibox.yml'
-      rm 'deploy.yml'
-      rm 'tape_vars.yml'
-      rm 'roles'
-      rm 'hosts'
-      rm 'dev_keys'
-      rm 'Vagrantfile'
+      rm "#{tapefiles_dir}/omnibox.yml"
+      rm "#{tapefiles_dir}/deploy.yml"
+      rm "#{tapefiles_dir}/tape_vars.yml"
+      rm "#{tapefiles_dir}/roles"
+      rm "#{tapefiles_dir}/hosts"
+      rm "#{tapefiles_dir}/dev_keys"
+      rm "Vagrantfile"
     end
 
     def rm(file)
@@ -70,7 +102,7 @@ module TapeBoxer
     end
 
     def mkdir(name)
-      print "#{name}: "
+      print "#{Pathname.new(name).basename}: "
       begin
         FileUtils.mkdir name
         success
@@ -82,17 +114,13 @@ module TapeBoxer
       end
     end
 
-    def touch(file)
-      File.new "#{local_dir}/#{file}", 'w'
-    end
-
     def copy_example(file, cp_file)
-      print "#{cp_file}: "
+      print "#{Pathname.new(cp_file).basename}: "
       begin
-        if File.exists?("#{local_dir}/#{cp_file}")
+        if File.exist?("#{cp_file}")
           exists
         else
-          FileUtils.cp("#{tape_dir}/#{file}", "#{local_dir}/#{cp_file}")
+          FileUtils.cp("#{tape_dir}/#{file}", "#{cp_file}")
           success
         end
       rescue Exception => e

data/lib/tape.rb

@@ -78,6 +78,14 @@ module TapeBoxer
     def local_dir
       Dir.pwd
     end
+
+    def tapefiles_dir
+      local_dir + '/taperole'
+    end
+
+    def tapecfg_dir
+      local_dir + '/.tape'
+    end
   end
 end
 

data/requirements.yml

@@ -10,7 +10,10 @@
 - src: https://github.com/ANXS/postgresql
   name: ANXS.postgresql
 
-- src: leonelgalan.node
-  version: '0.2'
+- src: williamyeh.nodejs
 
 - src: bennojoy.memcached
+
+- src: jnv.mosh
+
+- src: tersmitten.htop

data/roles/backend_install_essentials/meta/main.yml

@@ -2,4 +2,4 @@
 dependencies:
   - role: bennojoy.memcached
   - role: zzet.rbenv
-  - role: leonelgalan.node
+  - role: williamyeh.nodejs

data/roles/database_load/tasks/db_reset.yml

@@ -1,5 +1,5 @@
-- name: stop unicorns
-  service: name=unicorn_{{app_name}} state=stopped
+- name: stop everything
+  command: bash -lc "monit stop all"
 
 - name: Reset DB
   command: chdir={{ be_app_path }}
@@ -11,4 +11,4 @@
   when: db_reset.stderr
 
 - name: start unicorns
-  service: name=unicorn_{{app_name}} state=started
+  command: bash -lc "monit start all"

data/roles/deployer_user/tasks/keys.yml

@@ -9,11 +9,12 @@
   command: "cp /root/.ssh/authorized_keys /home/{{ deployer_user.name }}/.ssh/authorized_keys"
 
 - name: Ensure devs keys are present
-  authorized_key: key="{{ lookup('file', local_dir + '/' + item) }}"
+  authorized_key: key=
                   manage_dir=yes
                   state=present
                   user=deployer
-  with_items: dev_key_files
+  with_fileglob:
+    - "{{tapefiles_dir}}/dev_keys/*"
 
 - name: Ensure DO pubkey is present
   authorized_key: key="{{ lookup('file', 'id_rsa_digital_ocean.pub') }}"