SuperManifestRefUpdatedListener: Make commit latency timer cumulative

The supermanifest/superproject_commit_latency timer was created with a
Description that is neither cumulative nor has a unit. A Gerrit server
binds MetricMaker to DropWizardMetricMaker, which rejects such timers,
so constructing SuperManifestRefUpdatedListener fails and the plugin
is not loaded:

  Cannot load plugin supermanifest
  com.google.inject.ProvisionException: Unable to provision ...
  Caused by: java.lang.IllegalArgumentException: timer must be cumulative
    at ...SuperManifestRefUpdatedListener.<init>(SuperManifestRefUpdatedListener.java:149)

The acceptance tests don't catch this because their in-memory server
binds MetricMaker to TestMetricMaker, which doesn't validate metric
descriptions.

Mark the timer cumulative and give it the milliseconds unit, as
DropWizardMetricMaker requires for timers.

Change-Id: Ie22f661b3b7522b68145125705299253c3164f1e
1 file changed
tree: 9604cd7198ca6d27b5fb5d8e774df39a8f95e788
  1. java/
  2. javatests/
  3. BUILD
  4. external_plugin_deps.bzl
  5. LICENSE
  6. README.md
README.md

The plugin will update a superproject to contain the submodules defined in the manifest XML file whenever the manifest repo is changed.

For more information see the documentation or the build instructions