master_manipulator 1.2.4 → 1.2.6

checksums.yaml

@@ -1,15 +1,15 @@
 ---
 !binary "U0hBMQ==":
   metadata.gz: !binary |-
-    YjEyODEzZjdlNjg1NDkwYjE5OWM0OTYzMzIwM2QzYzZkZWVlMjI1ZA==
+    ZDZjYzhmMTQ2OGU3OTdhYWI3OWI3NzRkOWE0NzliZTZiY2NmNDdlOQ==
   data.tar.gz: !binary |-
-    OGMxMGJkNzQ0NTA5OWVjMjdmNDQxYTdlODRkZGRjMDFkOTU2NDAwMg==
+    NjU0MjQ0NTQzMjNhYzM1NzE0MzljN2QwNjI1YTlmMDY2ODAzMWUwNA==
 SHA512:
   metadata.gz: !binary |-
-    NDlmMWFiY2I3YzM2ODQ0NDg0MzhmYjIwNzYzZTFkNjU0ZmNmYzQyZDRmYmYx
-    ZWFiZDUxMzA2MWU1NmJhMzYwMjc0ZDI1MTBmNmI4OWRkMGIwNmRiOTY4OWEy
-    ZmJhNzE5Y2Y4NWMxYmE3NWQ4N2M1MGMyNDExMjcwOGY4YjBhYjM=
+    ZTZlNDBkZTk3ZjJkYWZkNGQwNTUwYmJmMzYxOTk0ZGM0NjJhNDM1ZThkNjc5
+    YmE0ZDhjY2ZiZGMwYWM3ZGYxYjFiZTYzMWI4OGIxODc3YTc4OGVhMmI1NzNk
+    OTZiNzkyOWUwMmU2YTYzNWZlNWVmNDk1YjZhODMwZTNjNjk2OTE=
   data.tar.gz: !binary |-
-    MjI4ZmQyYTkyZmQ3ZjZmMDQ2MTg4MWEwNjRmMTI0NDNlZDAxNDUwNmZmMWUx
-    YmYwYzc5M2UzYTNkNWE2YzJhNDE2NGIzMWFkYTEwMjY2M2VlNDUzNmQ4YmQ2
-    YzdhZDc4MjM3M2QyOTYwZjU1YjI3NDAyZDBjZTBkNzQ2YmQzYzc=
+    MjcxMzIxZDgzYzk1NzJiOTA1MzQyYjhkMTFiNDE5NmRjMjU1ODRhNTcyNzA0
+    NjRmOTQ1YTlhOTcyYWM2MzUwMDkzYjg3NzU1NDI1YWI1MmY2MTgyOGE5YmVk
+    MDRjYTU0OTdhNDk5YTVkZDhiNjYwYjE5NzRiNjcxNmEzZGY3ZDk=

data/HISTORY.md

@@ -1,6 +1,7 @@
 # default - History
 ## Tags
-* [LATEST - 27 Sep, 2016 (036a99e6)](#LATEST)
+* [LATEST - 10 Jan, 2017 (e3ff6153)](#LATEST)
+* [1.2.4 - 27 Sep, 2016 (48017488)](#1.2.4)
 * [1.2.3 - 20 Nov, 2015 (f31e884e)](#1.2.3)
 * [1.2.2 - 30 Oct, 2015 (31450d3c)](#1.2.2)
 * [1.2.1 - 30 Oct, 2015 (b59fd2cf)](#1.2.1)
@@ -12,7 +13,81 @@
 * [1.0.0 - 6 May, 2015 (8ae50d90)](#1.0.0)
 
 ## Details
-### <a name = "LATEST">LATEST - 27 Sep, 2016 (036a99e6)
+### <a name = "LATEST">LATEST - 10 Jan, 2017 (e3ff6153)
+
+* (GEM) update master_manipulator version to 1.2.6 (e3ff6153)
+
+* Merge pull request #25 from kurtwall/release-gem (838e4850)
+
+
+```
+Merge pull request #25 from kurtwall/release-gem
+
+(GEM) Release 1.2.5 gem
+```
+* (GEM) Release 1.2.5 gem (b6a81655)
+
+* Merge pull request #24 from kurtwall/fix-apiref (cd46a323)
+
+
+```
+Merge pull request #24 from kurtwall/fix-apiref
+
+(QA-2812) API ref missing #pe_version method
+```
+* (QA-2812) API ref missing #pe_version method (4f718a04)
+
+* Merge pull request #23 from kurtwall/release-gem (6f2f8ed9)
+
+
+```
+Merge pull request #23 from kurtwall/release-gem
+
+(maint) Fix YARD markup; modify a couple of variable names
+```
+* Update changelog (88fd7460)
+
+* Fix spec failures (36910dd8)
+
+* Various cleanups ahead of a 1.2.5 release (fdf54a20)
+
+* Merge pull request #22 from kurtwall/seed-the-yard (49f6ca54)
+
+
+```
+Merge pull request #22 from kurtwall/seed-the-yard
+
+(QA-2789) Convert API docs to use YARD markup
+```
+* Merge https://github.com/puppetlabs/master_manipulator into seed-the-yard (38cb1469)
+
+* Merge pull request #21 from kurtwall/update-readme (1d75e909)
+
+
+```
+Merge pull request #21 from kurtwall/update-readme
+
+(QA-2789) Revise Master Manipulator docs
+```
+* Start the initial conversion to YARD (169b5755)
+
+* Update README.md (2159ad76)
+
+* (QA-2789) Revise Master Manipulator docs (fc5cae87)
+
+* Merge pull request #20 from kurtwall/fix-typo (eff5cdf0)
+
+
+```
+Merge pull request #20 from kurtwall/fix-typo
+
+(MAINT) Fix typo
+```
+* (MAINT) Fix typo (d3a63069)
+
+### <a name = "1.2.4">1.2.4 - 27 Sep, 2016 (48017488)
+
+* (HISTORY) update master_manipulator history for gem release 1.2.4 (48017488)
 
 * (GEM) update master_manipulator version to 1.2.4 (036a99e6)
 

data/README.md

@@ -1,127 +1,33 @@
 # Master Manipulator
 
-This Gem extends the Beaker DSL for the purpose of changing things on a
-Puppet Master.
+This gem extends the Beaker DSL for the purpose of changing things on a
+Puppet master
 
 ## Installation
 
+### Using a Gemfile
+
 Add this line to your application's Gemfile:
 
+```
     gem 'master_manipulator'
+```
 
 And then execute:
 
-    $ bundle
-
-Or install it yourself as:
-
-    $ gem install master_manipulator
-
-## Methods
-
-### disable_node_classifier
-
-Disable the node classifier on a Puppet Enterprise master and use the "site.pp"
-manifest instead for node classification.
-
-#### Example
-
-    disable_node_classifier(master)
-
-### disable_env_cache
-
-Disable environment caching on a Puppet Enterprise master to allow for dynamic
-updates to environments without waiting for cache purge.
-
-#### Example
-
-    disable_env_cache(master)
-
-### restart_puppet_server
-
-Restart the Puppet Server service to pickup configuration changes made to the
-"puppet.conf" file.
-
-#### Example
-
-Restart the Puppet Server service on Puppet Enterprise:
-
-    restart_puppet_server(master)
-
-### get_manifests_path
-
-Retrieve the path to the manifests folder for an environment on a master.
-
-#### Example 1
-
-Retrieve the path to the manifests folder for the "production" environment.
-
-    prod_env_manifests_path = get_manifests_path(master)
-
-#### Example 2
-
-Retrieve the path to the manifests folder for the "staging" environment.
-
-    stage_env_manifests_path = get_manifests_path(master, :env => 'staging')
-
-### get_site_pp_path
-
-Retrieve the path to the "site.pp" manifest for an environment on a master.
-
-#### Example 1
-
-Retrieve the path to the "site.pp" manifest for the "production" environment.
-
-    prod_env_site_pp_path = get_site_pp_path(master)
-
-#### Example 2
-
-Retrieve the path to the "site.pp" manifest for the "production" environment.
-
-    stage_env_site_pp_path = get_site_pp_path(master, :env => 'staging')
-
-### create_site_pp
-
-Create a "site.pp" file with file bucket enabled. Also, allow the creation of a
-custom node definition or use the 'default' node definition.
-
-#### Example 1
-
-Create a "site.pp" manifest with the default node definition using a simple
-manifest.
-
-    site_pp = create_site_pp(master, :manifest => 'notify { hello: }')
-
-#### Example 2
-
-Create a "site.pp" manifest with a custom node definition using a simple
-manifest.
-
-    site_pp = create_site_pp(master, :node_def_name => 'puppet_agent', :manifest => 'notify { hello: }')
-
-### set_perms_on_remote
-
-Set permissions and ownership on a remote file.
-
-#### Example 1
-
-Set permissions on the "site.pp" manifest with default ownership of Pupppet
-user and group.
-
-    set_perms_on_remote(master, get_site_pp_path(master), '644')
-
-#### Example 2
+```
+    bundle install
+```
 
-Set permissions on the "site.pp" manifest with root ownership.
+or your preferred invocation for installing gems.
 
-    set_perms_on_remote(master, get_site_pp_path(master), '644', :owner => 'root', :group => 'root')
+### Using Rubygems
 
-### inject_site_pp
+To install the gem yourself:
 
-Inject a "site.pp" manifest onto a master.
 
-#### Example
+```
+    gem install master_manipulator
+```
 
-    prod_env_site_pp_path = get_site_pp_path(master)
-    site_pp = create_site_pp(master, :manifest => 'notify { hello: }')
-    inject_site_pp(master, prod_env_site_pp_path, site_pp)
+Refer to [the API document](doc/apiref.md) for API information.

data/Rakefile

@@ -1,5 +1,4 @@
 require "bundler/gem_tasks"
-
 require 'rspec/core/rake_task'
 
 task :default => :test

data/lib/master_manipulator/config.rb

@@ -1,36 +1,24 @@
 module MasterManipulator
   module Config
 
-    # Disable the Node Classifier on the Puppet master.
-    # (Note: this requires a restart of Puppet Server to take affect.)
-    # ==== Attributes
-    #
-    # * +master_host+ - The master running Puppet Server.
-    #
-    # ==== Returns
-    #
-    # nil
-    #
-    # ==== Examples
-    #
-    # site_pp = disable_node_classifier(master_host)
+    # Disable the node classifier on the Puppet master
+    # @param [Beaker::Host] master_host The master running the Puppet server.
+    # @return nil
+    # @example Disable the node classifier on the master
+    #   disable_node_classifier(master)
+    #   reload_puppetserver(master)
     def disable_node_classifier(master_host)
       on(master_host, puppet('config set node_terminus plain --section master'))
     end
 
-    # Disable environment caching on the Puppet master.
-    # (Note: this requires a restart of Puppet Server to take affect.)
-    # ==== Attributes
-    #
-    # * +master_host+ - The master running Puppet Server.
-    #
-    # ==== Returns
-    #
-    # nil
-    #
-    # ==== Examples
-    #
-    # site_pp = disable_node_classifier(master_host)
+    # Disable environment caching on the Puppet master. Modifying this
+    # this setting requires reloading (preferred) or restarting (not
+    # preferred) the Puppet server
+    # @param [Beaker::Host] master_host The master running the Puppet server.
+    # @return nil
+    # @example Disable the environment cache on master
+    #   disable_env_cache(master)
+    #   reload_puppetserver(master)
     def disable_env_cache(master_host)
       on(master_host, puppet('config set environment_timeout 0 --section main'))
     end

data/lib/master_manipulator/service.rb

@@ -3,31 +3,34 @@ require 'json'
 module MasterManipulator
   module Service
 
-    # Restart the puppet server and wait for it to come back up
-    # ==== Attributes
-    # *+host+ - the host that this should operate on
-    # *+opts+ - an options hash - not required
-    #   *+:wait_cycle+ - the number of cycles to attempt retry
-    #   *+:is_pe?+ - Boolean : if the SUT is PE, defaults to true
-    #
-    # Raises a standard error if the wait is unsuccessful
-    #
-    # ==== Example
-    # restart_puppet_server(master)
-    # restart_puppet_server(master, {:wait_cycles => 20})
-    def restart_puppet_server(host, opts = {})
+    # Restart the puppet server and wait for it to come back up or raise
+    # an error if the wait times out
+    # @param [Beaker::Host] master_host The host to manipulate.
+    # @param [Hash] opts Optional options hash containing options
+    # @option opts [Boolean] :is_pe? true if host is running PE, false otherwise
+    # @option opts [Integer] :wait_cycles How many attempt to make before failing
+    # @return nil
+    # @example Restart the puppetserver process on a PE master
+    #   restart_puppet_server(master)
+    # @example Restart the puppetserver process on a FOSS master
+    #   restart_puppet_server(master, {:is_pe? => false})
+    # @example Restart the puppetserver process on a PE master, timing out after 20 attempts
+    #   restart_puppet_server(master, {:wait_cycles => 20})
+    # @example Restart the puppetserver process on a FOSS master, timing out after 20 attempts
+    #   restart_puppet_server(master, {:is_pe? => false, :wait_cycles => 20})
+    def restart_puppet_server(master_host, opts = {})
 
       start_time = Time.now
 
       opts[:is_pe?] ||= true
       opts[:is_pe?] ? service_name = 'pe-puppetserver' : service_name = 'puppetserver'
 
-      on(host, puppet("resource service #{service_name} ensure=stopped"))
-      on(host, puppet("resource service #{service_name} ensure=running"))
-      hostname = on(host, 'hostname').stdout.chomp
+      on(master_host, puppet("resource service #{service_name} ensure=stopped"))
+      on(master_host, puppet("resource service #{service_name} ensure=running"))
+      hostname = on(master_host, 'hostname').stdout.chomp
       opts[:wait_cycles] ||= 10
 
-      pe_ver = pe_version(host)
+      pe_ver = pe_version(master_host)
       three_eight_regex = /^3\.8/
 
       # This logic is not ideal refactor in the future
@@ -40,8 +43,7 @@ module MasterManipulator
       end
 
       (1..opts[:wait_cycles]).each do |i|
-
-        @result = curl_on(host, curl_call, :acceptable_exit_codes => [0,1,7])
+        @result = curl_on(master_host, curl_call, :acceptable_exit_codes => [0,1,7])
         # parse body if we are using PE and we are not in PE 3.8
         (pe_ver && !pe_ver.match(three_eight_regex)) ? @body = JSON.parse(@result.stdout) : @body = []
 
@@ -50,42 +52,32 @@ module MasterManipulator
             sleep 20
             pe_ver.match(/three_eight_regex/) ? return : (return if @body.all? { |k, v| v['state'] == 'running' })
           when '1', '7'
-            # Exit code 7 is "connection refused"
+            # Exit code 7 is 'connection refused'
             sleep (i**(1.2))
         end
       end
 
       total_time = Time.now - start_time
-
       message = "Attempted to restart #{opts[:wait_cycles]} times, waited #{total_time} seconds total."
-
-      message << "\nHere is the status reported by the puppetserver'"
+      message << "\nHere is the status reported by the puppetserver"
 
       @body.each do |k,v|
         message << "\n'#{k}' state: #{v['state']} "
       end
-
       raise message
 
     end
 
-    # Determine the version of PE installed on the master
-    #
-    # ==== Attributes
-    # *+host+ - the host that this should operate on
-    #
-    # ==== Returns
-    #
-    # +string+ -The version of puppet enterprise, if version can not be determined 'version unknown' is returned
-    #
-    # ==== Examples
-    #
-    # pe_version
-    def pe_version(host)
-      if on(host, 'test -f /opt/puppet/pe_version', :acceptable_exit_codes => [0,1]).exit_code == 0
-        return on(host, 'cat /opt/puppet/pe_version').stdout.chomp
-      elsif on(host, 'test -f /opt/puppetlabs/server/pe_version', :acceptable_exit_codes => [0,1]).exit_code == 0
-        return on(host, 'cat /opt/puppetlabs/server/pe_version').stdout.chomp
+    # Return the version of PE installed on the specified Puppet master
+    # @param [Beaker::Host] master_host the master to query.
+    # @return [String] The version of Puppet Enterprise, or 'version unknown' if undetermined
+    # @example Return the version of PE running on master
+    #   ver = pe_version(master)
+    def pe_version(master_host)
+      if on(master_host, 'test -f /opt/puppet/pe_version', :acceptable_exit_codes => [0,1]).exit_code == 0
+        return on(master_host, 'cat /opt/puppet/pe_version').stdout.chomp
+      elsif on(master_host, 'test -f /opt/puppetlabs/server/pe_version', :acceptable_exit_codes => [0,1]).exit_code == 0
+        return on(master_host, 'cat /opt/puppetlabs/server/pe_version').stdout.chomp
       else
         return 'version unknown'
       end

data/lib/master_manipulator/site.rb

@@ -1,65 +1,51 @@
 module MasterManipulator
   module Site
 
-    # Get the location of an environment's manifests path. (Defaults to "production" environment.)
-    #
-    # ==== Attributes
-    #
-    # * +master_host+ - The master host that contains environments.
-    # * +opts:env+ - The environment from which to discover the manifests path.
-    #
-    # ==== Returns
-    #
-    # +string+ - An absolute path to the manifests for an environment on the master host.
-    #
-    # ==== Examples
-    #
-    # prod_env_manifests_path = get_manifests_path(master)
+    # Get the location of an environment's manifests path; defaults to production environment.
+    # @param [Beaker::Host] master_host The master host that contains environments.
+    # @param [Hash] opts Optional options hash containing options.
+    # @option opts [String] :env The environment to query on the specified master
+    # @return [String] The absolute path to the manifests for the given environment on the given master
+    # @example Get the manifests path for the production environment on master
+    #   prod_env_manifests_path = get_manifests_path(master)
+    # @example Get the manifests path for the testing environment on master
+    #   test_env_manifests_path = get_manifests_path(master, {:env => 'test'})
     def get_manifests_path(master_host, opts = {})
       opts[:env] ||= 'production'
-
       environment_base_path = on(master_host, puppet('config print environmentpath')).stdout.rstrip
 
       return File.join(environment_base_path, opts[:env], 'manifests')
     end
 
-    # Get the location of an environment's "site.pp" path. (Defaults to "production" environment.)
-    #
-    # ==== Attributes
-    #
-    # * +master_host+ - The master host that contains environments.
-    # * +opts:env+ - The environment from which to discover the "site.pp" path.
-    #
-    # ==== Returns
-    #
-    # +string+ - An absolute path to the "site.pp" for an environment on the master host.
-    #
-    # ==== Examples
-    #
-    # prod_env_site_pp_path = get_site_pp_path(master)
+    # Get the path to a given environment's site.pp; defaults to production environment.
+    # @param [Beaker::Host] master_host The master to query.
+    # @param [Hash] opts Optional options hash containing options.
+    # @option opts [String] :env opts The environment to query on the specified master
+    # @return [String] The absolute path to the site.pp for the given environment on the given master
+    # @example Return the path to the site.pp file for the production environment on master
+    #   prod_env_site_pp_path = get_site_pp_path(master)
+    # @example Return the path to the site.pp file for the testing environment on master
+    #   prod_env_site_pp_path = get_site_pp_path(master, {:env => 'testing'})
     def get_site_pp_path(master_host, opts = {})
       opts[:env] ||= 'production'
 
       return File.join(get_manifests_path(master_host, opts), 'site.pp')
     end
 
-    # Create a "site.pp" file with file bucket enabled. Also, allow
-    # the creation of a custom node definition or use the 'default'
+    # Create a site.pp file with file bucket enabled. Also, allow
+    # the creation of a custom node definition or use the default
     # node definition.
-    #
-    # ==== Attributes
-    #
-    # * +master_host+ - The target master for "site.pp" injection.
-    # * +opts:manifest+ - A Puppet manifest to inject into the node definition.
-    # * +opts:node_def_name+ - A node definition pattern or name.
-    #
-    # ==== Returns
-    #
-    # +string+ - A combined manifest with node definition containing input manifest
-    #
-    # ==== Examples
-    #
-    # site_pp = create_site_pp(master, '', node_def_name='agent')
+    # @param [Beaker::Host] master_host the target master for site.pp injection.
+    # @param [Hash] opts Optional options hash containing options.
+    # @option opts [String] :manifest The node definition to lay down
+    # @option opts [String] :node_def_name The node definition name
+    # @return [String] manifest A manifest containing the input node definition to lay down in site.pp
+    # @example Create a site.pp on master using the default node definition and an otherwise empty manifest
+    #   site_pp = create_site_pp(master)
+    # @example Create a site.pp on master for a node named 'mailgrunt' with an otherwise empty manifest
+    #   site_pp = create_site_pp(master, {:node_def_name => 'mailgrunt'})
+    # @example Create a site.pp on master for a node named 'mailgrunt' with a custom node definition in the manifest
+    #   site_pp = create_site_pp(master, {:manifest => manifest_for_mailgrunt, :node_def_name => 'mailgrunt'})
     def create_site_pp(master_host, opts = {})
       opts[:manifest] ||= ''
       opts[:node_def_name] ||= 'default'
@@ -72,8 +58,7 @@ MANIFEST
 
       node_def = <<-MANIFEST
 node #{opts[:node_def_name]} {
-
-#{opts[:manifest]}
+  #{opts[:manifest]}
 }
 MANIFEST
 
@@ -96,50 +81,41 @@ MANIFEST
     end
 
     # Set mode, owner and group on a remote path.
-    #
-    # ==== Attributes
-    #
-    # * +host+ - The remote host containing the target path.
-    # * +path+ - The path to set mode, user and group upon.
-    # * +mode+ - The desired mode to set on the path in as a string.
-    # * +opts:owner+ - The owner to set on the path. (Puppet user if not specified.)
-    # * +opts:group+ - The group to set on the path. (Puppet group if not specified.)
-    #
-    # ==== Returns
-    #
-    # nil
-    #
-    # ==== Examples
-    #
-    # set_perms_on_remote(master, "/tmp/test/site.pp", "777")
-    def set_perms_on_remote(host, path, mode, opts = {})
-      opts[:owner] ||= on(host, puppet('config print user')).stdout.rstrip
-      opts[:group] ||= on(host, puppet('config print group')).stdout.rstrip
-
-      on(host, "chmod -R #{mode} #{path}")
-      on(host, "chown -R #{opts[:owner]}:#{opts[:group]} #{path}")
+    # @param [Beaker::Host] master_host The remote host containing the target path.
+    # @param [String] path The pathname on which to set mode, user and group.
+    # @param [String] mode The desired file mode in 4-digit octal string (e.g. '0644').
+    # @param [Hash] opts Optional options hash containing options.
+    # @option opts [String] :owner The user owner to assign
+    # @option opts [String] :group The group owner to assign
+    # @return nil
+    # @example Set perms on a file on master with the default user and group
+    #   set_perms_on_remote(master, '/tmp/this_is_junk.pp', '0644')
+    # @example Set perms on a file on master with the default user and a custom group
+    #   set_perms_on_remote(master, '/tmp/this_is_junk.pp', '0644', {:group => 'tutones')
+    # @example Set perms on a file on master with a custom user and the default group
+    #   set_perms_on_remote(master, '/tmp/this_is_junk.pp', '0644', {:user => 'tommy')
+    # @example Set perms on a file on master with a custom user and a custom group
+    #   set_perms_on_remote(master, '/tmp/this_is_junk.pp', '0644', {:user => 'tommy', :group => 'tutones')
+    def set_perms_on_remote(master_host, path, mode, opts = {})
+      opts[:owner] ||= on(master_host, puppet('config print user')).stdout.rstrip
+      opts[:group] ||= on(master_host, puppet('config print group')).stdout.rstrip
+
+      on(master_host, "chmod -Rv #{mode} #{path}")
+      on(master_host, "chown -Rv #{opts[:owner]}:#{opts[:group]} #{path}")
     end
 
-    # Inject a "site.pp" manifest onto a master.
-    #
-    # ==== Attributes
-    #
-    # * +master_host+ - The target master for injection.
-    # * +site_pp_path+ - A path on the remote host into which the site.pp will be injected.
-    # * +manifest+ - The manifest content to inject into "site.pp" to the host target path.
-    #
-    # ==== Returns
-    #
-    # nil
-    #
-    # ==== Examples
-    #
-    # site_pp = inject_site_pp(master_host, "/tmp/test/site.pp", manifest)
+    # Inject a site.pp manifest onto a master.
+    # @param [Beaker::Host] master_host the target master for injection.
+    # @param [String] site_pp_path a path on the remote host into which the site.pp will be injected.
+    # @param [String] manifest the manifest content to inject into site.pp to the host target path.
+    # @return nil
+    # @example Place this_is_junk.pp, with the contents defined in manifest, in /tmp on the master named compile_master
+    #   site_pp = inject_site_pp(compile_master, '/tmp/this_is_junk.pp', manifest)
     def inject_site_pp(master_host, site_pp_path, manifest)
       site_pp_dir = File.dirname(site_pp_path)
       create_remote_file(master_host, site_pp_path, manifest)
 
-      set_perms_on_remote(master_host, site_pp_dir, '744')
+      set_perms_on_remote(master_host, site_pp_dir, '0744')
     end
 
   end

data/lib/master_manipulator/version.rb

@@ -1,5 +1,5 @@
 module MasterManipulator
   module Version
-    STRING = '1.2.4'
+    STRING = '1.2.6'
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: master_manipulator
 version: !ruby/object:Gem::Version
-  version: 1.2.4
+  version: 1.2.6
 platform: ruby
 authors:
-- Puppet Labs
+- Puppet, Inc.
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-09-27 00:00:00.000000000 Z
+date: 2017-01-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -136,10 +136,10 @@ dependencies:
     - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
-description: This Gem extends the Beaker DSL for the purpose of changing things on
-  a Puppet Master.
+description: This gem extends the Beaker DSL for the purpose of changing things on
+  a Puppet master
 email:
-- qa@puppetlabs.com
+- qa@puppet.com
 executables: []
 extensions: []
 extra_rdoc_files: []
@@ -178,6 +178,6 @@ rubyforge_project:
 rubygems_version: 2.4.6
 signing_key: 
 specification_version: 4
-summary: Puppet Labs testing library for controlling a Puppet Master
+summary: Puppet test library for modifying a Puppet master
 test_files:
 - spec/spec_helper.rb