(PDB-905) Fix containment-path for skipped events

We had removed some < v4 reporting code as part of the last release, but I had
also accidentally removed containment-path for skipped events. This was an
effort to drop older Puppet support basically.

This patch fixes that problem, and fixes the tests.

I've also cleaned up a couple of other typos and style items, plus
removed one more case of < v4 report format checking.

Signed-off-by: Ken Barber <ken@bob.sh>
Revert "Try reverting to c1.medium"

This reverts commit 3aa02cfb5c132b968cc20610061bb65d53411d62.
Merge pull request #1107 from kbarber/maint/stable/c3-large-instances-again

Revert "Try reverting to c1.medium"
Fix for acceptance test issue related to fact path -> fact value
constraint violation
More clarification around using an IP address for ssl-host and host

Setting `ssl-host` and `host` to a hostname often causes problems due to
resolution failing, pointing at the wrong thing or sometimes even resolving
first to an IPv6 address instead of an IPv4.

This patch fixes some documentation and the comment in the configuration
file to forewarn users of this scenario, so they can avoid it.

Signed-off-by: Ken Barber <ken@bob.sh>
Fix the dependency for puppet for the PuppetDB package for the correct version

We had accidentally reverted the parameterized handling around the Puppet
dependency in our Debian package control file. This patch corrects that.

Signed-off-by: Ken Barber <ken@bob.sh>
Fixed big when updating the values for an exsting certname's factset
Merge pull request #1109 from kbarber/maint/stable/avoid-hostnames-in-host-entries

More clarification around using an IP address for ssl-host and host
Merge pull request #1108 from senior/maint/stable/fact-path-constraint-violation-fix

Fix for acceptance test issue related to fact path -> fact value constraint violation
Merge pull request #1111 from kbarber/maint/stable/fix-puppetdb-puppet-dependency

Fix the dependency for puppet for the PuppetDB package for the correct v...
Merge pull request #1099 from kbarber/ticket/stable/pdb-905-null-containing-class-containing-path

(PDB-905) Fix containment-path for skipped events
(PDB-904) Switch fact_values.value_string trgm index to be GIN not GIST

There were reports of crashing due to a bug in pg_trgm indexing when a large
fact value was loaded into PuppetDB.

This patch removes the old index via migration, and will then replace it with
the index handling mechanism.

Signed-off-by: Ken Barber <ken@bob.sh>