Fixed puppet buildtest module so unnecessary steps are not run.

* Changed install_Development_Tools condition so it doesn't need
  a temporary file to figure out when to run.
* Removed mode from /opt/traf/build-tool-tgz since this directory
  will inherit its mode from the rsync command
* Removed puppet block to ensure /opt/traf/rsync.out is absent.
  This step pretty much always ran since it was always present.
  Moved this to the command of Exec['rsync-build-tool-tgz']
* Added condition to Exec['rsync-build-tool-tgz'] so it will only
  run if there are changes
* Change Exec['untar-build-tool-tgz'] to only run if
  Exec['rsync-build-tool-tgz'] runs

Change-Id: Ib9f1d0d1224225e71839fde5be6122a5bdb62384