Merge pull request #200 from Sharpie/SUP-4870-exclude-console-comidi-…

…route-metrics

(SUP-4870) Exclude comidi-route-metrics from Console collection

functions/version_based_excludes.pp

@@ -6,10 +6,20 @@
 function puppet_metrics_collector::version_based_excludes(
   String[1] $metrics_type,
 ) >> Array[String] {
-  if $metrics_type == 'puppetserver' {
-    ['file-sync-storage-service','pe-puppet-profiler','pe-master','pe-jruby-metrics']
-  }
-  else {
-    $excludes = []
+  case $metrics_type {
+    'puppetserver': {
+      # File Sync Storage includes a lot of detail that bloats file sizes.
+      # The pe-* metrics are legacy representations that only duplicate data.
+      ['file-sync-storage-service','pe-puppet-profiler','pe-master','pe-jruby-metrics']
+    }
+    'console': {
+      # PE Console has a lot of parameterized routes that can result in
+      # hundreds of megabytes collected daily from the route metrics.
+      # This data can be extracted from the console access log if needed.
+      ['comidi-route-metrics']
+    }
+    default: {
+      []
+    }
   }
 }