add /api/providers/#/metric_rollups

Author: kbrock

app/controllers/api/providers_controller.rb

@@ -25,6 +25,7 @@ class ProvidersController < BaseController
     include Subcollections::Folders
     include Subcollections::Lans
     include Subcollections::LoadBalancers
+    include Subcollections::MetricRollups
     include Subcollections::Networks
     include Subcollections::NetworkServices
     include Subcollections::Policies

app/controllers/api/subcollections/metric_rollups.rb

@@ -2,7 +2,8 @@ module Api
   module Subcollections
     module MetricRollups
       RESOURCE_TYPES = {
-        'vms' => 'VmOrTemplate'
+        'vms'       => 'VmOrTemplate',
+        'providers' => 'ExtManagementSystem'
       }.freeze
 
       def metric_rollups_query_resource(object)

config/api.yml

@@ -2782,6 +2782,7 @@
     - :cloud_volume_types
     - :custom_attributes
     - :load_balancers
+    - :metric_rollups
     - :network_services
     - :security_groups
     - :security_policies
@@ -2904,6 +2905,12 @@
         :identifier:
         - ems_infra_show
         - ems_storage_show
+    :metric_rollups_subcollection_actions:
+      :get:
+      - :name: read
+        :identifier:
+        - ems_infra_show
+        - ems_storage_show_list
     :networks_subcollection_actions:
       :get:
       - :name: read

spec/requests/providers_spec.rb

@@ -1453,6 +1453,42 @@ def gen_import_request
     end
   end
 
+  describe "metric rollups subcollection" do
+    let(:provider) { FactoryBot.create(:ems_redhat) }
+    let(:provider1) { FactoryBot.create(:ems_redhat) }
+    let(:url) { api_provider_metric_rollups_url(nil, provider) }
+
+    before do
+      FactoryBot.create_list(:metric_rollup_cm_hr, 3, :resource => provider, :timestamp => 1.hour.ago)
+      FactoryBot.create_list(:metric_rollup_cm_daily, 1, :resource => provider)
+      FactoryBot.create_list(:metric_rollup_cm_hr, 1, :resource => provider1)
+    end
+
+    it 'returns the metric rollups for the provider' do
+      api_basic_authorize subcollection_action_identifier(:providers, :metric_rollups, :read, :get)
+
+      provider.metric_rollups.where(:capture_interval_name => 'hourly').name
+      get(url, :params => { :capture_interval => 'hourly', :start_date => Time.zone.today.to_s })
+
+      expected = {
+        'count'    => 5,
+        'subcount' => 3,
+        'pages'    => 1
+      }
+      expect(response).to have_http_status(:ok)
+      expect(response.parsed_body).to include(expected)
+      expect(response.parsed_body['links'].keys).to match_array(%w[self first last])
+    end
+
+    it 'will not return metric rollups without an appropriate role' do
+      api_basic_authorize
+
+      get(url, :params => { :capture_interval => 'hourly', :start_date => Time.zone.today.to_s })
+
+      expect(response).to have_http_status(:forbidden)
+    end
+  end
+
   context 'cloud volume types subcollection' do
     before do
       @provider = FactoryBot.create(:ems_storage)