Add stuff to use ITK on Debian

[kumy]

Hi,

This commit add support for mpm_itk on Debian.

http://mpm-itk.sesse.net/

Bests

[igalic]

Travis doesn't like this

 1) apache::mod::itk on a Debian OS should contain File[/usr/local/etc/apache22/Modules/itk.conf] with ensure => "file"

Failure/Error: it { should contain_file("/usr/local/etc/apache22/Modules/itk.conf").with_ensure('file') }

expected that the catalogue would contain File[/usr/local/etc/apache22/Modules/itk.conf]

# ./spec/classes/mod/itk_spec.rb:15:in `block (3 levels) in <top (required)>'

Finished in 2 minutes 43.8 seconds

401 examples, 1 failure

[kumy]

Sorry for my mistake, Travis should be ok now.

[blkperl]

Can you rebase these commits into one?

[blkperl]

Can you add a validate_hash for the itk variable as well.

[blkperl]

An rspec-system test like https://github.com/puppetlabs/puppetlabs-apache/blob/master/spec/system/prefork_worker_spec.rb would be awesome as well.

[kumy]

as requested:

• merged commits into one
• use validate_hash
• add the rspec-system test

[blkperl]

The rspec-system tests are failing

Error: You must include the apache base class before using any apache defined resources at /etc/puppet/modules/apache/manifests/vhost.pp:125 on node main.foo.vm

Did you test on Debian or Ubuntu?

RSPEC_SET=ubuntu-server-12042-x64 rake spec:system
Failures:

  1) apache::vhost define apache_directories readme example, adapted should configure a vhost with Files
     Failure/Error: }) { |r| [0,2].should include r.exit_code}


     # ./spec/system/vhost_spec.rb:126:in `block (4 levels) in <top (required)>'
     # ./spec/system/vhost_spec.rb:126:in `block (3 levels) in <top (required)>'

  2) apache::vhost define apache_directories readme example, adapted should answer to files.example.net
     Failure/Error: r.stderr.should == "curl: (22) The requested URL returned error: 403 Forbidden\n"


     # ./spec/system/vhost_spec.rb:136:in `block (4 levels) in <top (required)>'
     # ./spec/system/vhost_spec.rb:135:in `block (3 levels) in <top (required)>'

Finished in 4 minutes 25.4 seconds
55 examples, 2 failures

[blkperl]

Hmm the rspec system tests failing may be unrelated.

[blkperl]

Fixed on of them in #347. Still investigating the other one.

[ptomulik]

https://github.com/blkperl/puppetlabs-apache/blob/92abe84d8ef9fb907582ca84ffb0fe25f0fef7e6/spec/system/vhost_spec.rb#L137

This check seems to be too restrictive - I've seen that the error message returned by apache differs slightly:

curl: (22) The requested URL returned error: 403

maybe change this check to

         r.stderr.should =~ /curl: \(22\) The requested URL returned error: 403/

With this change, the tests run smoothly again.