(PUP-3070) Update vimfiles included in Puppet
(maint) Simplify how noop events are checked and handled
(PUP-900) Don't process refresh events for noop'ed resources

When resources were processing queued events, the event callbacks would
be invoked if any of the events were not noop regardless of the state of
the resource in question. This means that a resource that had `noop =>
true` would have the refresh action triggered on refresh, bypassing the
noop statement.

This commit changes the refresh behavior to ensure that resources that
are set to noop are not refreshed.
(maint) Remove dead Transaction#handle_qualified_tags

Commit 0412515e2 removed all invocations of #handle_qualified_tags; we
can remove the remaining stub method.
(PUP-3243) There's no way the 'bsd' provider will work on OpenBSD
Merge pull request #3091 from adrienthebo/issue/master/pup-900-noop-refresh

Issue/master/pup 900 noop refresh
(PUP-3260) Remove rrdgraph report processor
(PUP-3260) Remove RRD functionality from Metric class
(PUP-3260) Remove rrd directory permissions test
(PUP-3260) remove librrd-ruby from Debian deps
(PUP-3249) Remove async storeconfigs and puppet queue
(PUP-3249) Remove puppet queue from packaging
(PUP-3249) Remove puppetqd init and default in postinst
Merge pull request #3103 from adrienthebo/removal/master/pup-3260-remove-rrdgraph-report-processor

(PUP-3260) Remove RRD functionality
(PUP-2560) Convert acceptance test for #7117 to unit test

The acceptance test for Redmine issur #7117 assumed that we would always
be able to reach the inventory the service and blindly did an HTTP
request against that to verify that directives in auth.conf could be
restricted to an environment. This is a very heavyweight way to test
this functionality, and it turns out that removing the inventory service
broke this test anyways. This commit adds a unit test to verify the
behavior of network rights with environments and removes the acceptance
test in favor of the unit test.
(maint) Actually test expected rights behavior

A number of tests were spuriously passing because we were doing a blind
check for truthiness, and `expect(:dunno).to be_true` will pass. In
addition we were checking a negative test that could never fail because
we were passing unmatchable arguments. This commit corrects the tests to
verify the correct behavior.
Merge remote-tracking branch 'upstream/pr/3102'

* upstream/pr/3102:
  (maint) Actually test expected rights behavior
  (PUP-2560) Convert acceptance test for #7117 to unit test
(PUP-3249) Remove packaging references to puppet-queue man pages

Packaging failed due to references to puppet-queue man pages that
were removed earlier.
(PUP-3249) Add back missing comma

Commit c33dc8d5958 removed the trailing comma as the "Suggests"
dependencies wrap. This commit adds it back.
(PUP-3249) Remove async storeconfigs and queue from man pages

Update man pages now that async_storeconfigs and queue_* properties have
been removed.