tweek 0.9.9 → 1.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: c6f72d6d59a8e68f01ced40dd39696822515e0d3
-  data.tar.gz: 8b97932c5381d852aa09fcc0066d601b5df5c3bb
+  metadata.gz: 7068bcfbf27d73e5c1a20c2f5d52671a8d0ce968
+  data.tar.gz: 8458e63c00fd12cc564a7e7aa3198aecfefe65b7
 SHA512:
-  metadata.gz: 284e773675805ef2882269ebea781d58f057c905292cd9e400905e7c7af3334d6d8bc5703531b91e6398e113400a56c8aaefe80217fdd1363c2013cfb8d2f283
-  data.tar.gz: 6a9b3db52163945915d8923fd0e979db24a6a386568d8fc3a9ad9a07024a5120c2caeb1fee7b58c66f9100021b348d0fce442990b51342a6d506662dce3d2330
+  metadata.gz: c09dfbf71eca18073c90a87616d46084f2ec6572d659c5a23beaf9c1044e33d984030d30ac24ff74a4e871f066ddf994529373cfdbe3f69af018c06de22d2b84
+  data.tar.gz: 7c85c2be58f9a2de5383a7f5caa0c433d30678f6a46eb12e7a32191d0c91a65eba57858ef6bb7b07894659958ab9d98bb046836b179623f72c32b071182d890d

data/README.md

@@ -5,7 +5,7 @@ be recorded and checked against a file of settings. Think of it as a series of
 configuration tests that can be executed, with the results being presented as a number of
 passes and failures. The number of failures is returned as the exit code of the process
 which allows it's use in scripts. This is particularly useful when checking multiple
-machines via a utility like `pssh` or `mush`.
+machines using a utility like `pssh` or `mush`.
 
 Although existing configuration management systems such as Puppet, Chef or Ansible can
 perform the same checks, they tend to require a high level of support infrastructure to be
@@ -118,7 +118,7 @@ For a discussion of which parameters are supported please see the sample setting
 
 Parameter lines can be suffixed by the word `if` and one or more conditions. These are
 used to limit the scope of the parameter check to those systems where the condition(s) hold
-true. The list of conditions is logicall AND'ed together. A condition takes the form:
+true. The list of conditions is logically AND'ed together. A condition takes the form:
 
 ```
 [k|d|v] op value
@@ -126,10 +126,10 @@ true. The list of conditions is logicall AND'ed together. A condition takes the
 
 Where:
  * `k` is the Kernel Version, `d` is the Distro name, and `v` is the Distro version.
- * The operator `op` can be: equality '==', regex equality '=~'or a version comparator as
+ * The operator `op` can be: equality `==`, regex equality `=~` or a version comparator as
    per the [Rubygems](http://ruby-doc.org/stdlib-2.0.0/libdoc/rubygems/rdoc/Gem/Version.html) specification.
- * The value is one of: a plain string in quotes (for equality), a regex between /
-   delimiters, or a plain version string.
+ * The value is one of: a plain string in `'` quotes (for equality), a regex between `/`
+   delimiters, or a plain version string (eg. `4.3.1`).
 
 NOTE: The determination of the current Distro, Distro Version and Kernel Version is done
 using the `lsb_release` and `uname` commands. Since these may not be installed on all

data/bin/tweek

@@ -9,7 +9,7 @@ require 'rubygems'
 require 'optparse'
 require 'ostruct'
 require 'tweek'
-require 'tweek/app'
+require 'tweek/file'
 require 'tweek/version'
 
 trap('INT') do
@@ -83,7 +83,7 @@ if kernel_ver.nil?
     exit 2
   end
 end
-cs = Tweek::App.new(gflag, true, distro, distro_ver, kernel_ver)
+cs = Tweek::File.new(gflag, true, distro, distro_ver, kernel_ver)
 if ARGV.empty?
   fh = File.open(File.expand_path('../../settings/sample.set',__FILE__))
 else

data/lib/tweek/{app.rb → file.rb}

@@ -4,7 +4,7 @@ require 'ostruct'
 require 'tweek'
 require 'tweek/section'
 
-class Tweek::App
+class Tweek::File
 
   attr_reader :distro, :distro_version, :kernel_version
 
@@ -22,12 +22,6 @@ class Tweek::App
     @tty = tty
   end
 
-  # Define data-collection methods so we can easily stub them for tests
-  #
-  def self.bootline
-    @bootline ||= File.open("/proc/cmdline", &:read).chomp rescue 'nowt'
-  end
-
   def generate type, e
     return unless @gflag
     case type
@@ -37,11 +31,21 @@ class Tweek::App
       @generated.push "#{e.type} #{e.list} #{e.comment}"
     when :entry
       cond = e.cond.nil? ? "": " if #{e.cond}"
-      comment = e.comment.nil? ? "#": e.comment
-      if e.actual
-        @generated.push "#{e.param} = #{e.actual}#{cond} #{comment} [expected #{e.value}]"
+      if e.actual.nil?
+        line = "#{e.param} = #{e.value}#{cond}"
+        note = "[condition not met]"
       else
-        @generated.push "#{e.param} = #{e.value}#{cond} #{comment} [skipped]"
+        line = "#{e.param} = #{e.actual}#{cond}"
+        if e.value === e.actual
+          note = nil
+        else
+          note = "[expected #{e.value}]"
+        end
+      end
+      if e.comment.nil?
+        @generated.push (note ? line + " # " + note : line)
+      else
+        @generated.push (note ? line + " " + comment + " " + note : line + " " + comment)
       end
     end
   end
@@ -65,6 +69,7 @@ class Tweek::App
 
   def condfail cond
     return false unless cond
+    cond.strip!
     failures = 0
     reqs = cond.split(/\s*,\s*/)
     reqs.each do |req|
@@ -123,9 +128,13 @@ class Tweek::App
         next
       end
 
-      if /^\s*(.+?)\s*=\s*(.*?)\s*(if\s+(.*))?$/ =~ line
-        section.push OpenStruct.new( :lineno => handle.lineno, :param => $1, :value => $2,
-                                     :cond => $4, :comment => comment )
+      if /^\s*(.+?)\s*=\s*(.*?)\s*(\bif\b(.*))?$/ =~ line
+        if $4 and $4.strip.empty?
+          error "#{handle.lineno}: Missing condition after 'if'"
+        else
+          section.push OpenStruct.new( :lineno => handle.lineno, :param => $1, :value => $2,
+                                       :cond => $4, :comment => comment )
+        end
         next
       end
       error "#{handle.lineno}: Unrecognized line: #{line}"
@@ -136,11 +145,14 @@ class Tweek::App
 
   def results
     STDERR.puts "\nDistro: #{@distro} Version: #{@distro_version} Kernel: #{@kernel_version}"
-    STDERR.puts "\n#{@nparams} parameters checked, #{@skips.size} skipped, #{@mismatches.size} mismatches, #{@warns.size} warnings, #{@errors.size} errors"
+    STDERR.puts "\n#{@nparams} parameters checked, #{@skips.size} conditions not met, #{@mismatches.size} mismatches, #{@warns.size} warnings, #{@errors.size} errors"
     STDERR.puts "\n#{@errors.join("\n")}" unless @errors.empty?
     STDERR.puts "\n#{@warns.join("\n")}" unless @warns.empty?
     STDERR.puts "\n#{@mismatches.join("\n")}" unless @mismatches.empty?
-    @generated.each { |l| STDOUT.puts l}
+    if @gflag
+      @generated.unshift "# Generated at #{Time.now.strftime("%c %Z")}\n# Distro: #{@distro} Version: #{@distro_version} Kernel: #{@kernel_version}"
+      @generated.each { |l| STDOUT.puts l}
+    end
     return @mismatches.size
   end
 end

data/lib/tweek/section.rb

@@ -63,7 +63,7 @@ class Tweek::Section < Array
 
       devices.each do |blk|
         each_entry (cs) do |entry|
-          entry.actual = File.open("/sys/block/#{blk}/queue/#{entry.param}", &:read).chomp rescue $!.message
+          entry.actual = Tweek.blk_param(blk, entry.param)
           if entry.param == 'scheduler'
             entry.actual = entry.actual.slice(/\[(.*?)\]/,1) unless entry.actual == 'none'
           end
@@ -74,7 +74,7 @@ class Tweek::Section < Array
 
     when 'KERNEL' # KERNEL PARAMETERS
       each_entry (cs) do |entry|
-        actual = /\b(#{entry.param})(=\S+)?\b/.match(Tweek::App.bootline)
+        actual = /\b(#{entry.param})(=\S+)?\b/.match(Tweek.bootline)
         case
         when (entry.value == 'true' and actual)
           observed = actual.to_s
@@ -99,7 +99,7 @@ class Tweek::Section < Array
 
     when 'CLOCKSOURCE'
       each_entry (cs) do |entry|
-        entry.actual=File.open("/sys/devices/system/clocksource/#{entry.param}/current_clocksource",&:read).chomp rescue $!.message
+        entry.actual = Tweek.clocksource(entry.param)
         cs.assert_equal entry.value, entry.actual, entry.param
         cs.generate :entry, entry
       end
@@ -107,7 +107,7 @@ class Tweek::Section < Array
     when 'SYSCTL' # SYSCTL SETTINGS
       each_entry (cs) do |entry|
         file = entry.param.gsub(/\./,'/')
-        entry.actual = File.open("/proc/sys/#{file}", &:read).chomp.gsub(/\s+/,' ') rescue $!.message
+        entry.actual = Tweek.sysctl(file)
         cs.assert_equal entry.value, entry.actual, entry.param
         cs.generate :entry, entry
       end
@@ -121,84 +121,39 @@ class Tweek::Section < Array
 
 	  case entry.param
 	  when 'driver'
-	    entry.actual = File.basename(File.readlink("/sys/class/net/#{netdev}/device/driver"))
-	    cs.assert_equal entry.value, entry.actual, entry.param
+            entry.actual = Tweek.net_driver(netdev)
 
 	  when 'irqs'
 
-	    irqs=File.open("/proc/interrupts", &:readlines).grep(/\b#{netdev}\b/).map{|m| m.partition(':')[0].strip}
+            balpid = Tweek.balance_pid
 
-	    if irqs.empty?
-	      cs.warn "#{netdev} IRQ check skipped: none found"
-	      entry.actual = 'null'
-	      cs.generate :entry, entry
-	      next
-	    end
-
-	    ncores = `getconf _NPROCESSORS_ONLN`.chomp.to_i rescue 0
-	    mismatches = 0
-	    irqs.each_with_index do |irq, ix|
-	      case entry.value
-	      when 'null','none','ignore'
-		next
-
-	      when 'aws'
-		expected_rps = ("00000000," * 3) + "0000ff00"
-		expected_rss = ("00000000," * 3) + sprintf("%08x", 1<<ix)
-
-	      when 'pin'
-		case ncores
-		when 32 # RPS on cores 10-15 and 18-31, RSS tied to 0-7
-		  case ix
-		  when 0..7
-		    expected_rps = ("00000000," * 3) + "fffefe00"
-		    expected_rss = ("00000000," * 3) + sprintf("%08x", 1<<(ix+1))
-		  else
-		    cs.warn "#{netdev} IRQ strategy '#{entry.value}' can't handle IRQ#{ix}"
-		  end
-
-		when 64 # Split the work across the two nodes
-		  case ix
-		  when 0..3 # alloc on node 0 (cpus 0-3)
-		    expected_rps = ("00000000," * 2) + "0000ffff," + "0000ffe0"
-		    expected_rss = ("00000000," * 3) + sprintf("%08x", 1<<(ix+1))
-		  when 4..7 # alloc on node 1 (
-		    expected_rps = ("00000000," * 2) + "ffff0000," + "ffe00000"
-		    expected_rss = ("00000000," * 3) + sprintf("%08x", 1<<(ix + 9))
-		  else
-		    cs.warn "#{netdev} IRQ strategy '#{entry.value}' can't handle IRQ#{ix}"
-		  end
-
-		else
-		  cs.error "#{entry.lineno}: Can't handle #{ncores} cores in IRQ strategy '#{entry.value}'"
-		end
-
-	      else
-		cs.error "#{entry.lineno}: Unrecognized #{netdev} IRQ strategy '#{entry.value}'"
-	      end
-	      actual_rss = File.open("/proc/irq/#{irq}/smp_affinity",&:read).chomp rescue $!.message
-	      actual_rps = File.open("/sys/class/net/#{netdev}/queues/rx-#{ix}/rps_cpus", &:read).chomp rescue $!.message
-	      mismatches += cs.assert_equal expected_rss, actual_rss, "#{netdev}_irq_#{irq}_rss"
-	      mismatches += cs.assert_equal expected_rps, actual_rps, "#{netdev}_irq_#{irq}_rps"
-	    end
-	    entry.actual = 'null' if mismatches > 0
+            if entry.value == 'balance'
+              entry.actual = balpid.zero? ? 'IRQ balance daemon not running' : 'balance'
+            else
+              if balpid.zero?
+                handle_irqs(netdev, cs, entry)
+                next
+              else
+                entry.actual = 'IRQ balance daemon running'
+              end
+            end
 
-	  else # try getting the name
+	  else # treat the parameter as a name
 	    begin
-	      entry.actual = File.open("/sys/class/net/#{netdev}/#{entry.param}", &:read).chomp.gsub(/\s+/,' ')
-	      cs.assert_equal entry.value, entry.actual, entry.param
-
+              entry.actual = Tweek.net_param(netdev, entry.param)
 	    rescue
 	      cs.error "#{entry.lineno}: Network parameter #{entry.param} not handled: #{$!.message}"
 	      next
 	    end
           end
+
+          cs.assert_equal entry.value, entry.actual, entry.param
 	  cs.generate :entry, entry
         end
       end
 
     when 'EXT4' # EXT4 filesystems info via `dumpe2fs -h /dev/#{device}`
-      mounted = File.open("/proc/mounts", &:readlines).map{|m| m.split(' ')}
+      mounted = Tweek.mounted
       mounts = Dir.glob(self.list.split(' '))
       mounts.each do |mount|
         unless this = mounted.select{|m| (mount == m[0] or mount == m[1]) and m[2] == 'ext4'}.first
@@ -223,8 +178,8 @@ class Tweek::Section < Array
         cs.warn "#{self.lineno}: XFS device check skipped: no mountpoints found"
       end
       mounts.each do |mount|
-        xfsinfo = `xfs_info #{mount} 2>/dev/null`.chomp
-        if $?.exitstatus > 0
+        xfsinfo = Tweek.xfsinfo(mount)
+        unless xfsinfo
           cs.warn "No XFS filesystem at #{mount}"
           each_entry (cs) { |entry| cs.generate :entry, entry }
           next
@@ -248,5 +203,89 @@ class Tweek::Section < Array
     end
 
   end
+
+  def handle_irqs netdev, cs, entry
+    irqs = Tweek.irqs(netdev)
+
+    if irqs.empty?
+      cs.warn "#{netdev} IRQ check skipped: none found"
+      entry.actual = 'null'
+      cs.generate :entry, entry
+      return
+    end
+
+    ncores = Tweek.ncores
+    mismatches = 0
+    expected_rps = []
+    expected_rss = []
+    irqs.each_with_index do |irq, ix|
+      case entry.value
+      when 'null','none','ignore'
+        next
+
+      when 'aws'
+        case ncores
+        when 8..64
+          expected_rps << ("00000000," * 3) + "0000ff00"
+          expected_rss << ("00000000," * 3) + sprintf("%08x", 1<<ix)
+        when 1..8
+          expected_rps << "0000"
+          expected_rss << sprintf("%04x", 1<<ix)
+        else
+          cs.warn "#{netdev} IRQ strategy '#{entry.value}' can't handle #{ncores} cores"
+          return
+        end
+
+      when 'pin'
+        case ncores
+        when 1..8 # not a lot of choice
+          expected_rps << "0000"
+          expected_rss << "0001"
+
+        when 8..32 # RPS on cores 10-15 and 18-31, RSS tied to 0-7
+          case ix
+          when 0..7
+            expected_rps << ("00000000," * 3) + "fffefe00"
+            expected_rss << ("00000000," * 3) + sprintf("%08x", 1<<(ix+1))
+          else
+            cs.warn "#{netdev} IRQ strategy '#{entry.value}' can't handle IRQ#{ix}"
+            return
+          end
+
+        when 32..64 # Split the work across the two nodes
+          case ix
+          when 0..3 # alloc on node 0 (cpus 0-3)
+            expected_rps << ("00000000," * 2) + "0000ffff," + "0000ffe0"
+            expected_rss << ("00000000," * 3) + sprintf("%08x", 1<<(ix+1))
+          when 4..7 # alloc on node 1 (
+            expected_rps << ("00000000," * 2) + "ffff0000," + "ffe00000"
+            expected_rss << ("00000000," * 3) + sprintf("%08x", 1<<(ix + 9))
+          else
+            cs.warn "#{netdev} IRQ strategy '#{entry.value}' can't handle IRQ#{ix}"
+            return
+          end
+
+        else
+          cs.error "#{entry.lineno}: Can't handle #{ncores} cores in IRQ strategy '#{entry.value}'"
+          return
+        end
+
+      else
+        cs.error "#{entry.lineno}: Unrecognized #{netdev} IRQ strategy '#{entry.value}'"
+        return
+      end
+    end
+    irqs.each_with_index do |irq, ix|
+      actual_rss = Tweek.net_rss(irq)
+      mismatches += cs.assert_equal expected_rss[ix], actual_rss, "#{netdev}_irq_#{irq}_rss"
+    end
+    irqs.each_with_index do |irq, ix|
+      actual_rps = Tweek.net_rps(netdev, ix)
+      mismatches += cs.assert_equal expected_rps[ix], actual_rps, "#{netdev}_irq_#{irq}_rps"
+    end
+    entry.actual = 'null' if mismatches > 0
+    return
+  end
+
 end
 

data/lib/tweek/version.rb

@@ -1,3 +1,3 @@
 module Tweek
-  VERSION = "0.9.9"
+  VERSION = "1.0.0"
 end

data/lib/tweek.rb

@@ -1,3 +1,64 @@
+# Define data-collection methods so we can easily stub them for tests
+#
 module Tweek
-  # nothing yet
+
+  def self.bootline
+    @bootline ||= ::File.open("/proc/cmdline", &:read).chomp
+  end
+
+  def self.balance_pid
+    @balance_pid ||= `pgrep irqbalance`.chomp.to_i
+  end
+
+  # Determine the queue processing interrupts for the network device
+  # Empirically we observe that the convention for the name in /proc/interrupts varies
+  # so some hacking required...
+  def self.irqs netdev
+    @irqs = ::File.open("/proc/interrupts", &:readlines).grep(/\b#{netdev}\b/)
+    @irqs.reject! {|i| i !~ /\b#{netdev}-Tx-?Rx/} if @irqs.size > 1
+    @irqs.map{|m| m.partition(':')[0].strip}
+  end
+
+  def self.ncores
+    `getconf _NPROCESSORS_ONLN`.chomp.to_i rescue 0
+  end
+
+  def self.clocksource param
+     ::File.open("/sys/devices/system/clocksource/#{param}/current_clocksource",&:read).chomp rescue $!.message
+  end
+
+  def self.sysctl name
+    ::File.open("/proc/sys/#{name}", &:read).chomp.gsub(/\s+/,' ') rescue $!.message
+  end
+
+  def self.net_driver name
+    ::File.basename(::File.readlink("/sys/class/net/#{name}/device/driver"))
+  end
+
+  def self.blk_param blkdev, name
+    ::File.open("/sys/block/#{blkdev}/queue/#{name}", &:read).chomp rescue $!.message
+  end
+
+  # Receive Side Scaling - hardware
+  def self.net_rss irq
+    ::File.open("/proc/irq/#{irq}/smp_affinity",&:read).chomp rescue $!.message
+  end
+
+  # Receive Packet Steering - software
+  def self.net_rps netdev, ix
+    ::File.open("/sys/class/net/#{netdev}/queues/rx-#{ix}/rps_cpus", &:read).chomp rescue $!.message
+  end
+
+  def self.net_param netdev, name
+    ::File.open("/sys/class/net/#{netdev}/#{name}", &:read).chomp.gsub(/\s+/,' ')
+  end
+
+  def self.mounted
+    ::File.open("/proc/mounts", &:readlines).map{|m| m.split(' ')}
+  end
+
+  def self.xfsinfo mount
+    info = `xfs_info #{mount} 2>/dev/null`.chomp
+    return $?.exitstatus > 0 ? nil : info
+  end
 end

data/settings/sample.set

@@ -1,7 +1,13 @@
-# A documentation-rich sample for the parameter checking script
+# A documentation-rich sample file for Tweek
+#
+# This is NOT meant to be a recommended parameter set, it's purpose
+# is to document the format with real examples and educate on the meaning and effect of
+# various tuning parameters. Use with caution, YMMV!
 #
 # Comments begin with # and extend to the end of the line
-# Block comments are Ruby style =begin and =end
+=begin
+Block comments are Ruby style =begin and =end
+=end
 # Conditionals use 'k' for kernel version, 'v' for distro version and 'd' for distro
 # Version conditionals use the http://guides.rubygems.org/patterns/#semantic-versioning
 # Distro conditionals follow Ruby syntax and you may use string or regex comparisons
@@ -25,12 +31,16 @@ maxcpus = 63 # Set the number of physical cores, eliminates the 'B' hyperthreads
 # intel_idle.max_cstate=1
 numa = off if k>3.9 # won't move memory around between NUMA zones, useful with large working set
 
+
+# Recommended for heavy networking activity
+#
 SYSCTL
 vm.min_free_kbytes = 838608
-vm.zone_reclaim = 1 if k~>4.9.0,d=='Ubuntu'
-vm.zone_reclaim_mode = 1 if k~>3.13.0, k<4.9.0
+vm.zone_reclaim_mode = 1
 
 # These parameters taken from Brendan Gregg's Re:Invent talk 2014
+#
+SYSCTL
 net.core.somaxconn = 1000
 net.core.netdev_max_backlog = 5000
 net.core.rmem_max = 16777216
@@ -47,27 +57,21 @@ net.ipv4.tcp_slow_start_after_idle = 0
 net.ipv4.tcp_tw_reuse = 1
 net.ipv4.ip_local_port_range = 1024 65535
 net.ipv4.tcp_abort_on_overflow = 1  # allows a connection that is queued to be reset, not just dropped.
-net.ipv4.tcp_fin_timeout = 50 if d=~/Amazon|RedHat/i
+net.ipv4.tcp_fin_timeout = 50 if d=~/AmazonAMI|RedHat/i
 
+# Page cache flushing optimization (Brendan Gregg, Re:Invent 2014)
 # Smoother page cache flushing: background flush earlier, aggressive later
-# Brendan Gregg, Re:Invent 2014
 #
 vm.dirty_ratio = 80
 vm.dirty_background_ratio = 5
 vm.dirty_expire_centisecs = 12000
 
-=begin
-# These are all the recommended RedHat TCP settings
-# not yet validated, so used as demo of multiline comments
-net.ipv4.tcp_syn_retries = 0
-net.ipv4.tcp_synack_retries = 0
-net.ipv4.tcp_timestamps = 0
-=end
-
-NET eth0 # Define one or more network devices
-# Check the IRQ strategy:
+NET eth0
+# Check the IRQ strategy using the 'irqs' parameter:
+#
 # aws:  Use first numa node, split RSS from RPS
 #       RSS allocated across 0-7 (node 0), RPS on first numa node
+#
 # pin:  Differs depending on number of cores
 #    32 Apportion across the single node as follows:
 #       RPS allocated across cores 10-15 and 18-31
@@ -75,10 +79,14 @@ NET eth0 # Define one or more network devices
 #    64 Use both NUMA nodes, leave first core free on each node
 #       RPS allocated across remaining cores on each node
 #       RSS allocated across 1-4 (node 0) then 17-20 (node 1)
-# null: Ignore this adapter
-irqs = pin
+#
+# balance: Use the irqbalance daemon
+# null: Ignore this device
+#
+irqs = aws
+# Use the 'driver' parameter for the driver name
 driver = ena
-# Check values in /sys/class/net/<device>/ tree
+# Other parameter names match values in /sys/class/net/<device>/ tree
 mtu = 9001
 device/driver/module/version = 1.1.3
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: tweek
 version: !ruby/object:Gem::Version
-  version: 0.9.9
+  version: 1.0.0
 platform: ruby
 authors:
 - Nick Townsend
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-09-20 00:00:00.000000000 Z
+date: 2017-09-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -65,7 +65,7 @@ files:
 - README.md
 - bin/tweek
 - lib/tweek.rb
-- lib/tweek/app.rb
+- lib/tweek/file.rb
 - lib/tweek/section.rb
 - lib/tweek/version.rb
 - settings/sample.set