foreman_leapp 0.0.6 → 0.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a75680a7afc44c4a8e90727d2319a53b231748a6db1f8a1c3627c319b3483902
-  data.tar.gz: 8506877b49e172205eeadb3a898d9ee5e8e1235525155673d52b98efdfb7441f
+  metadata.gz: 133a054cb6f2f0aab562ef7c0e5863e844ede2a3124a738594c03033a06225d6
+  data.tar.gz: ed23743e31e2a42dcf9c5ddaebae7b2ebe38ac8c46cff49cdd4856df07091fc2
 SHA512:
-  metadata.gz: 18a93546178c2252cd530f913d98633e24808dbf66698ddff5d4249d2eec0c9df2513447a304df59cd55e4b54c0bcefb31cae0713dd002ca1895bb78bb19be28
-  data.tar.gz: 4993f008708cdb78a9defd9cd311cadb7396e75ad16f0827675ab32e0361e5ad5154810d3e8b1d5f9504d1db2849c6aebe9addae16006a9ff8698b70719b2247
+  metadata.gz: ca1b768b4c623e9b1d679b0bb67b8bdc882ba74041bf386b81bbb647c186669564fd155f3a1ab8d8df1ff7141535efc044d843c6cd81ae08beba1bf550ad1738
+  data.tar.gz: fe21fa9458002c0ae272d6b2738a8abf758b8434a8aad5c31fbf04e3da79ac230a2ab91a6b37a3d3041560f67c249012ec0a035ee2b0d28320fd53e1de45d752

data/README.md

@@ -1,32 +1,23 @@
 # ForemanLeapp
 
-This plugin allows to run inplace upgrades for rhel7 vms in foreman using leapp tool.
-For more information about leapp check [github](https://github.com/oamg/leapp) or
-[developer docs](https://leapp.readthedocs.io/en/latest/).
+This plugin allows to run inplace upgrades for RHEL 7 hosts in Foreman using Leapp tool.
+For more information about Leapp tool check [github](https://github.com/oamg/leapp) or [developer docs](https://leapp.readthedocs.io/en/latest/).
 
 ## Installation
 
-See [How_to_Install_a_Plugin](http://projects.theforeman.org/projects/foreman/wiki/How_to_Install_a_Plugin)
-for how to install Foreman plugins
+See [Plugins Manual](https://www.theforeman.org/plugins/#2.Installation) for how to install Foreman plugins.
 
 ## Usage
 
-The plugin will add 2 remote execution jobs, "Run preupgrade via leapp" and "Run upgrade via leapp". Only
-preupgrade reports storage and retrieval has been implemented so far.
+The plugin will add following jobs:
+- Run preupgrade via Leapp
+- Remediation plan
+- Run upgrade via Leapp
 
-After running a preupgrade remote execution job on one or more foreman hosts the report can be retrieved from foreman db.
-
-The retrieval api looks like:
-
-- to fetch a specific preupgrade report - GET http://FOREMAN_URL:FOREMAN_PORT/api/v2/preupgrade_reports/REPORT_ID.
-- to fetch all reports for specific job invocation - GET http://FOREMAN_URL:FOREMAN_PORT/api/v2/aggregation/JOB_INVOCATION_ID.
-- to fetch last preupgrade report per host - GET http://FOREMAN_URL:FOREMAN_PORT/api/v2/preupgrade_reports/hosts/HOST_NAME_OR_ID/last.
-
-## TODO
-
-- Unit tests
-- Automate rubocop checks
-- Frontend (either from scratch or adapt the react/patternfly/typescript one for cockpit upgrades)
+## Api
+- `GET /api/preupgrade_reports` List Preupgrade reports
+- `GET /api/preupgrade_reports/:id` Show Preupgrade report
+- `GET /api/job_invocations/:id/preupgrade_reports` List Preupgrade reports for Job invocation
 
 ## Contributing
 

data/app/controllers/api/v2/concerns/api_authorizer.rb

@@ -0,0 +1,27 @@
+# frozen_string_literal: true
+
+module ApiAuthorizer
+  extend ActiveSupport::Concern
+
+  included do
+    before_action :hosts_permission
+  end
+
+  private
+
+  def hosts_permission
+    return if User.current.can?('view_hosts')
+
+    render_error 'access_denied', status: :forbidden,
+                                  locals: { details: _('Missing one of the required permissions: view_hosts'),
+                                            missing_permissions: 'view_hosts' }
+  end
+
+  def resource_scope(_options = {})
+    @resource_scope ||= begin
+      scope = PreupgradeReport.joins(:host).merge(Host.authorized(:view_hosts, Host))
+      scope = scope.where(job_invocation_id: params[:id]) if action_name == 'job_invocation'
+      scope
+    end
+  end
+end

data/app/controllers/api/v2/preupgrade_reports_controller.rb

@@ -3,7 +3,9 @@
 module Api
   module V2
     class PreupgradeReportsController < ::Api::V2::BaseController
-      before_action :find_resource, only: %i[show]
+      include ApiAuthorizer
+
+      layout 'api/v2/layouts/index_layout', except: %i[show]
 
       api :GET, '/preupgrade_reports/', N_('List Preupgrade reports')
       param_group :search_and_pagination, ::Api::V2::BaseController
@@ -13,7 +15,24 @@ module Api
 
       api :GET, '/preupgrade_reports/:id', N_('Show Preupgrade report')
       param :id, :identifier, required: true
-      def show; end
+      def show
+        @preupgrade_report = resource_scope.find(params[:id])
+      end
+
+      api :GET, '/job_invocations/:id/preupgrade_reports', N_('List Preupgrade reports for Job invocation')
+      param :id, :identifier, required: true
+      def job_invocation
+        @preupgrade_reports = resource_scope_for_index.where(job_invocation_id: params[:id])
+      end
+
+      private
+
+      # By overriding path_to_authenticate we can require REX's permission view_job_invocations
+      def path_to_authenticate
+        params['action'] = 'show' if params['action'] == 'job_invocation'
+        Foreman::AccessControl.normalize_path_hash params.slice(:action, :id, :user_id)
+                                                         .merge({ controller: 'api/v2/job_invocations' })
+      end
     end
   end
 end

data/app/controllers/preupgrade_reports_controller.rb

@@ -1,7 +1,17 @@
 # frozen_string_literal: true
 
 class PreupgradeReportsController < ::Api::V2::BaseController
+  include ApiAuthorizer
+
   def index
-    @preupgrade_reports = resource_scope.search_for(*search_options)
+    @preupgrade_reports = resource_scope.includes(:preupgrade_report_entries).search_for(*search_options)
+  end
+
+  private
+
+  # By overriding :path_to_authenticate we can require REX's :view_job_invocations permission
+  def path_to_authenticate
+    Foreman::AccessControl.normalize_path_hash params.slice(:action, :id, :user_id)
+                                                     .merge({ controller: 'job_invocations' })
   end
 end

data/app/lib/actions/preupgrade_job.rb

@@ -8,7 +8,8 @@ module Actions
       end
 
       def plan(job_invocation, host, *_args)
-        return unless ::Helpers::JobHelper.correct_feature?(job_invocation, 'leapp_preupgrade')
+        return unless ::Helpers::JobHelper.correct_feature?(job_invocation, 'leapp_preupgrade') ||
+                      ::Helpers::JobHelper.correct_feature?(job_invocation, 'leapp_remediation_plan')
 
         plan_self(host_id: host.id, job_invocation_id: job_invocation.id)
       end

data/app/models/preupgrade_report.rb

@@ -27,6 +27,7 @@ class PreupgradeReport < ::Report
       leapp_run_id: data['leapp_run_id'],
       summary: entry['summary'],
       tags: entry['tags'],
+      flags: entry['flags'],
       detail: entry['detail'] }
   end
 end

data/app/models/preupgrade_report_entry.rb

@@ -5,6 +5,7 @@ class PreupgradeReportEntry < ApplicationRecord
   belongs_to_host
 
   serialize :tags, Array
+  serialize :flags, Array
   serialize :detail, JSON
 
   validates :preupgrade_report, :host, :hostname, :title, :actor, :audience, :severity, :leapp_run_id, presence: true

data/app/views/api/v2/preupgrade_report_entries/base.json.rabl

@@ -1,4 +1,4 @@
 object @preupgrade_report_entry
 
 attributes :id, :preupgrade_report_id, :host_id, :hostname, :title, :actor, :audience,
-           :severity, :leapp_run_id, :summary, :tags, :created_at, :updated_at
+           :severity, :leapp_run_id, :summary, :tags, :flags, :created_at, :updated_at

data/app/views/api/v2/preupgrade_reports/job_invocation.json.rabl

@@ -0,0 +1,3 @@
+collection @preupgrade_reports
+
+extends 'api/v2/preupgrade_reports/base'

data/app/views/foreman_leapp/job_templates/check.erb

@@ -0,0 +1,14 @@
+<%#
+name: Check Leapp
+description_format: 'Check if Leapp package is installed.'
+kind: job_template
+job_category: Leapp
+provider_type: SSH
+snippet: true
+%>
+
+if ! command -v leapp > /dev/null
+then
+  echo "Leapp is not installed."
+  exit 1
+fi

data/app/views/foreman_leapp/job_templates/preupgrade.erb

@@ -7,8 +7,10 @@ provider_type: SSH
 feature: leapp_preupgrade
 %>
 
+<%= render_template 'Check Leapp' %>
+
 leapp preupgrade
-[ $? -eq 0 ]  || exit 1
 
 echo "===leap_upgrade_report_start==="
 cat /var/log/leapp/leapp-report.json
+rm -f /var/log/leapp/leapp-report.json

data/app/views/foreman_leapp/job_templates/remediation.erb

@@ -12,6 +12,17 @@ template_inputs:
   input_type: user
   value_type: plain
   description: List of remediation ids
+- name: run_preupgrade
+  description: Run preupgrade check again when remediation entries are fixed.
+  input_type: user
+  required: true
+  options: "true\nfalse"
+  default: "true"
 %>
 
+<%= render_template 'Check Leapp' %>
+
 <%= build_remediation_plan(input('remediation_ids').split(','), @host) %>
+<% if input('run_preupgrade') == 'true' %>
+  <%= render_template 'Run preupgrade via Leapp' %>
+<% end %>

data/app/views/foreman_leapp/job_templates/upgrade.erb

@@ -6,18 +6,19 @@ description_format: 'Upgrade RHEL 7 host'
 provider_type: Ansible
 feature: leapp_upgrade
 template_inputs:
-- name: reboot
-  description: reboot the vm automaticaly to continue with the upgrade
+- name: Reboot
+  description: Reboot the host automaticaly to continue with the upgrade
   input_type: user
-  required: false
-  options: "false\ntrue"
+  required: true
+  default: "true"
+  options: "true\nfalse"
 %>
 ---
 - hosts: all
   tasks:
     - name: Run Leapp Upgrade
       command: leapp upgrade
-<% if input('reboot') == "true" %>
+<% if input('Reboot') == "true" %>
     - name: Reboot the machine
       reboot:
         reboot_timeout: 1800

data/config/routes.rb

@@ -4,8 +4,9 @@ Rails.application.routes.draw do
   resources :preupgrade_reports, :only => %i[index]
 
   namespace :api, defaults: { format: 'json' } do
-    scope '(:apiv)', module: :v2, defaults: { apiv: 'v2'}, apiv: /v2/, constraints: ApiConstraints.new( version: 2, default: true) do
+    scope '(:apiv)', module: :v2, defaults: { apiv: 'v2' }, apiv: /v2/, constraints: ApiConstraints.new( version: 2, default: true) do
       resources :preupgrade_reports, only: %i[index show]
+      get 'job_invocations/:id/preupgrade_reports', to: 'preupgrade_reports#job_invocation'
     end
   end
 end

data/db/migrate/20200429080939_report_entries_flags.rb

@@ -0,0 +1,5 @@
+class ReportEntriesFlags < ActiveRecord::Migration[6.0]
+  def change
+    add_column :preupgrade_report_entries, :flags, :text
+  end
+end

data/lib/foreman_leapp/engine.rb

@@ -33,7 +33,10 @@ module ForemanLeapp
                          partial: 'job_invocations/leapp_preupgrade_report',
                          name: _('Leapp preupgrade report'),
                          id: 'leapp_preupgrade_report',
-                         onlyif: proc { |subject| ::Helpers::JobHelper.correct_feature?(subject, 'leapp_preupgrade') }
+                         onlyif: proc { |subject|
+                           ::Helpers::JobHelper.correct_feature?(subject, 'leapp_preupgrade') ||
+                             ::Helpers::JobHelper.correct_feature?(subject, 'leapp_remediation_plan')
+                         }
         end
       end
     end

data/lib/foreman_leapp/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module ForemanLeapp
-  VERSION = '0.0.6'
+  VERSION = '0.1.0'
 end

data/package.json

@@ -6,7 +6,7 @@
   "scripts": {
     "lint": "tfm-lint --plugin -d /webpack",
     "link-leapp-js": "./script/link_leapp_js.sh",
-    "test": "tfm-test --plugin",
+    "test": "tfm-test --plugin --config jest.config.js",
     "test:watch": "tfm-test --plugin --watchAll",
     "test:current": "tfm-test --plugin --watch",
     "publish-coverage": "tfm-publish-coverage",
@@ -32,10 +32,11 @@
     "@theforeman/builder": "^4.2.1",
     "@theforeman/eslint-plugin-foreman": "4.2.1",
     "@theforeman/stories": "^4.2.1",
-    "@theforeman/vendor-dev": "^4.2.1",
     "@theforeman/test": "^4.2.1",
+    "@theforeman/vendor-dev": "^4.2.1",
     "babel-eslint": "^10.0.0",
     "eslint": "^6.8.0",
+    "jest-svg-transformer": "^1.0.0",
     "prettier": "^1.19.1"
   },
   "dependencies": {

data/test/functional/api/v2/preupgrade_reports_controller_test.rb

@@ -7,24 +7,107 @@ module Api
     class PreupgradeReportsControllerTest < ActionController::TestCase
       setup do
         @host = FactoryBot.create(:host)
-        @report = FactoryBot.create(:preupgrade_report, host: @host)
+        @job_invocation = FactoryBot.create(:job_invocation)
+        @report = FactoryBot.create(:preupgrade_report, host: @host, job_invocation: @job_invocation)
         @entry = FactoryBot.create(:preupgrade_report_entry, host: @host, preupgrade_report: @report)
       end
 
-      test 'should get index' do
-        get :index, session: set_session_user
+      test 'should get :index' do
+        get :index
         assert_response :success
-        assert_not_empty ActiveSupport::JSON.decode(@response.body)['results']
+        assert_not_empty JSON.parse(@response.body)['results']
       end
 
-      test 'should get detail of report and its entries' do
+      test 'should get :show' do
         get :show, params: { id: @report.id }
         assert_response :success
 
-        response = ActiveSupport::JSON.decode(@response.body)
+        response = JSON.parse(@response.body)
         assert_equal response['id'], @report.id
         assert_not_empty response['preupgrade_report_entries']
       end
+
+      test 'should get :job_invocation' do
+        get :job_invocation, params: { id: @job_invocation.id }
+        assert_response :success
+        assert_not_empty JSON.parse(@response.body)['results']
+      end
+
+      context 'with permissions' do
+        setup do
+          @user = FactoryBot.create(:user, admin: false)
+          setup_user('view', 'job_invocations', nil, @user)
+          setup_user('view', 'hosts', nil, @user)
+        end
+
+        test 'should get :index' do
+          get :index, session: set_session_user(@user)
+          assert_response :success
+          assert_not_empty JSON.parse(@response.body)['results']
+        end
+
+        test 'should get :show' do
+          get :show, params: { id: @report.id }, session: set_session_user(@user)
+          assert_response :success
+          assert_equal @report.id, JSON.parse(@response.body)['id']
+        end
+
+        test 'should get :job_invocation' do
+          get :job_invocation, params: { id: @job_invocation.id }, session: set_session_user(@user)
+          assert_response :success
+          assert_not_empty JSON.parse(@response.body)['results']
+        end
+      end
+
+      context 'without :view_job_invocations' do
+        setup do
+          @user = FactoryBot.create(:user, admin: false)
+          setup_user('view', 'hosts', nil, @user)
+        end
+
+        test 'should not get :index' do
+          get :index, session: set_session_user(@user)
+          assert_response :forbidden
+          assert_includes JSON.parse(@response.body)['error']['missing_permissions'], 'view_job_invocations'
+        end
+
+        test 'should not get :show' do
+          get :show, params: { id: @report.id }, session: set_session_user(@user)
+          assert_response :forbidden
+          assert_includes JSON.parse(@response.body)['error']['missing_permissions'], 'view_job_invocations'
+        end
+
+        test 'should not get :job_invocation' do
+          get :job_invocation, params: { id: @job_invocation.id }, session: set_session_user(@user)
+          assert_response :forbidden
+          assert_includes JSON.parse(@response.body)['error']['missing_permissions'], 'view_job_invocations'
+        end
+      end
+
+      context 'without :view_hosts' do
+        setup do
+          @user = FactoryBot.create(:user, admin: false)
+          setup_user('view', 'job_invocations', nil, @user)
+        end
+
+        test 'should not get :index' do
+          get :index, session: set_session_user(@user)
+          assert_response :forbidden
+          assert_includes JSON.parse(@response.body)['error']['missing_permissions'], 'view_hosts'
+        end
+
+        test 'should not get :job_invocation' do
+          get :show, params: { id: @report.id }, session: set_session_user(@user)
+          assert_response :forbidden
+          assert_includes JSON.parse(@response.body)['error']['missing_permissions'], 'view_hosts'
+        end
+
+        test 'should not get :job_invocation' do
+          get :job_invocation, params: { id: @job_invocation.id }, session: set_session_user(@user)
+          assert_response :forbidden
+          assert_includes JSON.parse(@response.body)['error']['missing_permissions'], 'view_hosts'
+        end
+      end
     end
   end
 end

data/test/functional/preupgrade_reports_controller_test.rb

@@ -4,13 +4,36 @@ require 'test_plugin_helper'
 
 class PreupgradeReportsControllerTest < ActionController::TestCase
   setup do
+    @user = FactoryBot.create(:user, admin: false)
     @host = FactoryBot.create :host
     FactoryBot.create :preupgrade_report, host: @host
   end
 
-  test 'should get index' do
-    get :index, session: set_session_user
-    response = ActiveSupport::JSON.decode(@response.body)
-    assert_equal @host.id, response['results'].first['host_id']
+  test 'should get :index' do
+    get :index
+    assert_response :success
+    assert_not_empty JSON.parse(@response.body)['results']
+  end
+
+  test 'should get :index with :view_job_invocations & :view_hosts' do
+    setup_user 'view', 'job_invocations', nil, @user
+    setup_user 'view', 'hosts', nil, @user
+    get :index, session: set_session_user(@user)
+    assert_response :success
+    assert_not_empty JSON.parse(@response.body)['results']
+  end
+
+  test 'should not get :index without :view_job_invocations' do
+    setup_user 'view', 'hosts', nil, @user
+    get :index, session: set_session_user(@user)
+    assert_response :forbidden
+    assert_includes JSON.parse(@response.body)['error']['missing_permissions'], 'view_job_invocations'
+  end
+
+  test 'should not get :index without :view_hosts' do
+    setup_user 'view', 'job_invocations', nil, @user
+    get :index, session: set_session_user(@user)
+    assert_response :forbidden
+    assert_includes JSON.parse(@response.body)['error']['missing_permissions'], 'view_hosts'
   end
 end

data/webpack/__mocks__/foremanReact/common/I18n.js

@@ -1 +1 @@
-export const translate = () => jest.fn();
+export const translate = val => val;

data/webpack/__mocks__/foremanReact/components/common/EmptyState.js

@@ -0,0 +1 @@
+export const EmptyStatePattern = () => jest.fn();

data/webpack/components/PreupgradeReports/PreupgradeReports.js

@@ -1,24 +1,118 @@
-import React from 'react';
+import React, { useState } from 'react';
 import MessageBox from 'foremanReact/components/common/MessageBox';
 import { LoadingState, Row } from 'patternfly-react';
 import PropTypes from 'prop-types';
 
 import PreupgradeReportsList from '../PreupgradeReportsList';
-import FixAllButton from './components/FixAllButton';
+import UpgradeAllButton from './components/UpgradeAllButton';
+import EntriesFilter from './components/EntriesFilter';
+import FixSelectedButton from './components/FixSelectedButton';
 
 import {
   flattenEntries,
   isEmpty,
   anyEntriesFixable,
+  filterEntries,
+  idsForInvocationFromEntries,
+  sortEntries,
+  fixableEntries,
 } from './PreupgradeReportsHelpers';
 
+import NoReports from './components/NoReports';
+
 const PreupgradeReports = ({
   preupgradeReports,
-  loading,
-  error,
   csrfToken,
   newJobInvocationUrl,
 }) => {
+  const [filterType, setFilterType] = useState('title');
+  const [filterValue, setFilterValue] = useState('');
+  const [checked, setChecked] = useState([]);
+  const [sort, setSort] = useState({ attribute: '', order: 'desc' });
+
+  const onFilterValueChange = value => {
+    setFilterValue(value);
+  };
+
+  const onFilterValueClear = () => setFilterValue('');
+
+  const onFilterTypeChange = value => {
+    onFilterValueClear();
+    setFilterType(value);
+  };
+
+  const isSelected = entry => checked.some(item => item.id === entry.id);
+
+  const anySelected = checked.length > 0;
+
+  const toggleSelected = (entry, isEntrySelected) => {
+    if (isEntrySelected) {
+      setChecked(checked.filter(item => item.id !== entry.id));
+    } else {
+      setChecked([entry, ...checked]);
+    }
+  };
+
+  const toggleSelectAll = () => {
+    const allFixable = fixableEntries(preupgradeReports);
+
+    if (checked.length === allFixable.length) {
+      setChecked([]);
+    } else {
+      setChecked(allFixable);
+    }
+  };
+
+  const changeSort = value => {
+    setSort({ ...sort, ...value });
+  };
+
+  return (
+    <React.Fragment>
+      <Row>
+        <div className="col-md-8">
+          <EntriesFilter
+            filterType={filterType}
+            onFilterTypeChange={onFilterTypeChange}
+            filterValue={filterValue}
+            onFilterValueChange={onFilterValueChange}
+          />
+        </div>
+        <div className="col-md-4">
+          <div className="btn-toolbar pull-right">
+            <FixSelectedButton
+              postUrl={newJobInvocationUrl}
+              disabled={!anyEntriesFixable(preupgradeReports) || !anySelected}
+              csrfToken={csrfToken}
+              ids={idsForInvocationFromEntries(checked)}
+            />
+            <UpgradeAllButton
+              postUrl={newJobInvocationUrl}
+              csrfToken={csrfToken}
+              preupgradeReports={preupgradeReports}
+            />
+          </div>
+        </div>
+      </Row>
+      <PreupgradeReportsList
+        allEntries={filterEntries(
+          filterType,
+          filterValue,
+          sortEntries(flattenEntries(preupgradeReports), sort)
+        )}
+        isSelected={isSelected}
+        toggleSelected={toggleSelected}
+        sort={sort}
+        changeSort={changeSort}
+        toggleSelectAll={toggleSelectAll}
+      />
+    </React.Fragment>
+  );
+};
+
+const withLoadingState = Component => props => {
+  const { error, loading, preupgradeReports, reportsExpected } = props;
+
   if (!isEmpty(error)) {
     return (
       <MessageBox
@@ -31,30 +125,19 @@ const PreupgradeReports = ({
 
   return (
     <LoadingState loading={loading}>
-      <Row>
-        <div className="title-filter col-md-4">&nbsp;</div>
-        <div id="title_action" className="col-md-8">
-          <div className="btn-toolbar pull-right">
-            <FixAllButton
-              postUrl={newJobInvocationUrl}
-              disabled={!anyEntriesFixable(preupgradeReports)}
-              csrfToken={csrfToken}
-              preupgradeReports={preupgradeReports}
-            />
-          </div>
-        </div>
-      </Row>
-      <PreupgradeReportsList allEntries={flattenEntries(preupgradeReports)} />
+      {preupgradeReports.length > 0 ? (
+        <Component {...props} />
+      ) : (
+        <NoReports reportsExpected={reportsExpected} />
+      )}
     </LoadingState>
   );
 };
 
 PreupgradeReports.propTypes = {
   preupgradeReports: PropTypes.array.isRequired,
-  loading: PropTypes.bool.isRequired,
-  error: PropTypes.object.isRequired,
   csrfToken: PropTypes.string.isRequired,
   newJobInvocationUrl: PropTypes.string.isRequired,
 };
 
-export default PreupgradeReports;
+export default withLoadingState(PreupgradeReports);

data/webpack/components/PreupgradeReports/PreupgradeReportsActions.js

@@ -7,7 +7,7 @@ import {
   PREUPGRADE_REPORTS_FAILURE,
 } from '../../consts';
 
-export const getPreupgradeReports = url => async dispatch => {
+export const getPreupgradeReportsAction = url => async dispatch => {
   dispatch({ type: PREUPGRADE_REPORTS_REQUEST });
 
   try {