3scale_toolbox 0.18.3 → 0.19.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 0a3084c0b2b684a252c7e07c4c9a87e3087ada1ad6a677766aa508ac98e1f313
-  data.tar.gz: 158b1b008a67836c2b27ccd8cbcd49f59995e4ac776eae93c59bf44b002cbf3a
+  metadata.gz: d2a44bdf128bf6f8d4d0a592a4eee49c82c34415017e03366ff2bcc241fe18c0
+  data.tar.gz: 0e94df4747331e9092ae0c961709cfcc334a553eb54209378941b8239307eceb
 SHA512:
-  metadata.gz: '02789c23291a2191617a56c3456297539b09e0922bbf62b57b613065b04d8951800d91f49989d510258c0c3b1baa308d1219f4c29373879c8a65f9a93bd4653a'
-  data.tar.gz: abaca7089c51b5de57c118f5154a74466dd6671a96d7ec043b47ba0d4772761a2d9896b1195e762115ff121296c16cc2f195a5d39e9c20191393f45e6ee756bc
+  metadata.gz: 8dd7a6efc6567598d87e4ebbc18c486eba1ea9e6766ca6c0241f78e73a4287408d9e18ac838788e81a073c8f3a8fd93d299fac4ea9f21a42571e3cff68f2b0bb
+  data.tar.gz: c4ed75a0b46414606208a5fb6e1374e72d5fe06fbb5a237b3793bc2471d20667c489acace51cdf93d4971bfe59b42dc2919631b90074a8af72bac1668c663d01

data/3scale_toolbox.gemspec

@@ -8,8 +8,8 @@ Gem::Specification.new do |spec|
   spec.name          = '3scale_toolbox'
   spec.version       = ThreeScaleToolbox::VERSION
   spec.licenses      = ['MIT']
-  spec.authors       = ['Michal Cichra', 'Eguzki Astiz Lezaun']
-  spec.email         = ['michal@3scale.net', 'eastizle@redhat.com']
+  spec.authors       = ['Miguel Soriano', 'Eguzki Astiz Lezaun']
+  spec.email         = ['msoriano@redhat.com', 'eastizle@redhat.com']
 
   spec.summary       = %q{3scale Toolbox.}
   spec.description   = %q{3scale tools to manage your API from the terminal.}
@@ -34,7 +34,7 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency 'rake', '~> 13.0'
   spec.add_development_dependency 'rspec', '~> 3.8'
   spec.add_development_dependency 'webmock', '~> 3.4'
-  spec.required_ruby_version = '>= 2.5'
+  spec.required_ruby_version = '>= 2.6'
 
   spec.add_dependency '3scale-api', '~> 1.4'
   spec.add_dependency 'cri', '~> 2.15'

data/README.md

@@ -39,7 +39,6 @@
 ## Requirements
 Supported Ruby interpreters
 
-* MRI 2.5
 * MRI 2.6
 * MRI 2.7
 
@@ -219,5 +218,5 @@ Install, uninstall and update plugins using tools like [RubyGems](https://guides
 
 ## Contributing
 
-Bug reports and pull requests are welcome on GitHub at https://github.com/3scale/3scale_toolbox.
+If you are interested in contributing to 3scale Toolbox, please refer to instructions available [here](docs/contributing.md)
 

data/lib/3scale_toolbox/commands/backend_command/copy_command/delete_mapping_rules_task.rb

@@ -0,0 +1,18 @@
+module ThreeScaleToolbox
+  module Commands
+    module BackendCommand
+      module CopyCommand
+        class DeleteMappingRulesTask
+          include Task
+
+          # entrypoint
+          def run
+            return unless delete_mapping_rules
+
+            target_backend.mapping_rules.each(&:delete)
+          end
+        end
+      end
+    end
+  end
+end

data/lib/3scale_toolbox/commands/backend_command/copy_command/task.rb

@@ -53,6 +53,10 @@ module ThreeScaleToolbox
             context[:report] ||= {}
           end
 
+          def delete_mapping_rules
+            context.fetch(:delete_mapping_rules, false)
+          end
+
           private
 
           def find_source_backend

data/lib/3scale_toolbox/commands/backend_command/copy_command.rb

@@ -2,6 +2,7 @@ require '3scale_toolbox/commands/backend_command/copy_command/task'
 require '3scale_toolbox/commands/backend_command/copy_command/create_or_update_target_backend_task'
 require '3scale_toolbox/commands/backend_command/copy_command/copy_metrics_task'
 require '3scale_toolbox/commands/backend_command/copy_command/copy_methods_task'
+require '3scale_toolbox/commands/backend_command/copy_command/delete_mapping_rules_task'
 require '3scale_toolbox/commands/backend_command/copy_command/copy_mapping_rules_task'
 
 module ThreeScaleToolbox
@@ -17,7 +18,7 @@ module ThreeScaleToolbox
             summary     'Copy backend'
             description <<-HEREDOC
             This command makes a copy of the referenced backend.
-            Target backend will be searched by source backend system name. System name can be overriden with `--target-system-name` option.
+            Target backend will be searched by the source backend system name. System name can be overridden with `--target-system-name` option.
             If a backend with the selected `system-name` is not found, it will be created.
             \n Components of the backend being copied:
             \nmetrics
@@ -41,6 +42,7 @@ module ThreeScaleToolbox
           # First metrics as methods need 'hits' metric in target backend
           tasks << CopyCommand::CopyMetricsTask.new(context)
           tasks << CopyCommand::CopyMethodsTask.new(context)
+          tasks << CopyCommand::DeleteMappingRulesTask.new(context)
           tasks << CopyCommand::CopyMappingRulesTask.new(context)
           tasks.each(&:call)
         end
@@ -56,6 +58,7 @@ module ThreeScaleToolbox
             source_remote: threescale_client(fetch_required_option(:source)),
             target_remote: threescale_client(fetch_required_option(:destination)),
             source_backend_ref: arguments[:source_backend],
+            delete_mapping_rules: true,
             option_target_system_name: options[:'target-system-name']
           }
         end

data/lib/3scale_toolbox/commands/import_command/import_csv.rb

@@ -104,14 +104,14 @@ module ThreeScaleToolbox
 
               # create a mapping rule
               if (metric_id = metric['id'] || method['id'])
-                mapping_rule = client.create_mapping_rule(service['id'], {
-                                                            metric_id:          metric_id,
-                                                            pattern:            item['endpoint_path'],
-                                                            http_method:        item['endpoint_http_method'],
-                                                            metric_system_name: item['endpoint_system_name'],
-                                                            auth_app_key:       auth_app_key_according_service(service),
-                                                            delta:              1
-                                                          })
+                mapping_rule = client.create_mapping_rule(service['id'],{
+                                                          metric_id:          metric_id,
+                                                          pattern:            item['endpoint_path'],
+                                                          http_method:        item['endpoint_http_method'],
+                                                          metric_system_name: item['endpoint_system_name'],
+                                                          auth_app_key:       auth_app_key_according_service(service),
+                                                          delta:              1
+                })
 
                 if mapping_rule['errors'].nil?
                   stats[:mapping_rules] += 1

data/lib/3scale_toolbox/commands/import_command/openapi/create_activedocs_step.rb

@@ -53,7 +53,7 @@ module ThreeScaleToolbox
             # Other processing steps can work with original openapi spec
             Helper.hash_deep_dup(resource).tap do |activedocs|
               # public production base URL
-              # the basePath field is updated to a new value only when overriden by optional param
+              # the basePath field is updated to a new value only when overridden by optional param
               unless service.proxy['endpoint'].nil?
                 api_spec.set_server_url(activedocs, URI.join(service.proxy.fetch('endpoint'), public_base_path))
               end

data/lib/3scale_toolbox/commands/import_command/openapi/update_policies_step.rb

@@ -15,7 +15,7 @@ module ThreeScaleToolbox
             # do not update in-place, otherwise changes will not be detected
             policies_settings = source_policies_settings.dup
 
-            add_anonymous_access_policy(policies_settings)
+            reconcile_anonymous_access_policy(policies_settings)
             add_rh_sso_keycloak_role_check_policy(policies_settings)
             add_url_rewritting_policy(policies_settings)
 
@@ -31,15 +31,28 @@ module ThreeScaleToolbox
 
           private
 
-          def add_anonymous_access_policy(policies)
-            # only on 'open api' security req
-            return unless api_spec.security.nil?
-
-            return if policies.any? { |policy| policy['name'] == 'default_credentials' }
-
-            # Anonymous policy should be before apicast policy
-            # hence, adding as a first element
-            policies.insert(0, anonymous_policy)
+          def reconcile_anonymous_access_policy(policies)
+            idx = policies.find_index { |p| p['name'] == 'default_credentials' }
+
+            if api_spec.security.nil?
+              # only on 'open api' security req
+              # Update anonymous policy if exists
+              #
+              if idx.nil?
+                # Anonymous policy should be before apicast policy
+                # hence, adding as a first element
+                policies.insert(0, anonymous_policy)
+              else
+                # only update if different
+                if policies[idx].dig('configuration', 'user_key') != anonymous_policy.dig(:configuration, :user_key)
+                  policies[idx] = anonymous_policy
+                end
+              end
+            else
+              unless idx.nil?
+                policies.slice!(idx)
+              end
+            end
           end
 
           def anonymous_policy

data/lib/3scale_toolbox/commands/import_command/openapi.rb

@@ -108,6 +108,8 @@ module ThreeScaleToolbox
           end
 
           def openapi_parser
+            raise ThreeScaleToolbox::Error, 'only JSON/YAML format is supported' unless openapi_resource.is_a?(Hash)
+
             if openapi_resource.key?('openapi')
               ThreeScaleToolbox::OpenAPI::OAS3.build(openapi_path, openapi_resource, validate: validate)
             else

data/lib/3scale_toolbox/commands/plans_command/export_command.rb

@@ -1,12 +1,3 @@
-require '3scale_toolbox/commands/plans_command/export/step'
-require '3scale_toolbox/commands/plans_command/export/read_app_plan_step'
-require '3scale_toolbox/commands/plans_command/export/read_plan_features_step'
-require '3scale_toolbox/commands/plans_command/export/read_plan_limits_step'
-require '3scale_toolbox/commands/plans_command/export/read_plan_pricing_rules_step'
-require '3scale_toolbox/commands/plans_command/export/read_plan_methods_step'
-require '3scale_toolbox/commands/plans_command/export/read_plan_metrics_step'
-require '3scale_toolbox/commands/plans_command/export/write_artifacts_file_step'
-
 module ThreeScaleToolbox
   module Commands
     module PlansCommand
@@ -31,32 +22,64 @@ module ThreeScaleToolbox
           end
 
           def run
-            tasks = []
-            tasks << ReadAppPlanStep.new(context)
-            tasks << ReadPlanLimitsStep.new(context)
-            tasks << ReadPlanPricingRulesStep.new(context)
-            tasks << ReadPlanFeaturesStep.new(context)
-            tasks << ReadPlanMethods.new(context)
-            tasks << ReadPlanMetrics.new(context)
-            tasks << WriteArtifactsStep.new(context)
-
-            # run tasks
-            tasks.each(&:call)
+            select_output do |output|
+              plan_object = application_plan.to_hash.merge(
+                'created_at' => Time.now.utc.iso8601,
+                'toolbox_version' => ThreeScaleToolbox::VERSION
+              )
+              output.write(plan_object.to_yaml)
+            end
           end
 
           private
 
-          def context
-            @context ||= create_context
+          def remote
+            @remote ||= threescale_client(arguments[:remote])
+          end
+
+          def select_output
+            ios = if file
+                    File.open(file, 'w')
+                  else
+                    $stdout
+                  end
+            begin
+              yield(ios)
+            ensure
+              ios.close
+            end
+          end
+
+          def application_plan
+            @application_plan ||= find_application_plan
+          end
+
+          def find_application_plan
+            Entities::ApplicationPlan.find(service: product, ref: plan_system_name).tap do |p|
+              raise ThreeScaleToolbox::Error, "Application plan #{plan_system_name} does not exist" if p.nil?
+            end
+          end
+
+          def product
+            @product ||= find_product
+          end
+
+          def product_ref
+            arguments[:service_system_name]
+          end
+
+          def plan_system_name
+            arguments[:plan_system_name]
+          end
+
+          def find_product
+            Entities::Service.find(remote: remote, ref: product_ref).tap do |prd|
+              raise ThreeScaleToolbox::Error, "Product #{product_ref} does not exist" if prd.nil?
+            end
           end
 
-          def create_context
-            {
-              file: options[:file],
-              threescale_client: threescale_client(arguments[:remote]),
-              service_system_name: arguments[:service_system_name],
-              plan_system_name: arguments[:plan_system_name],
-            }
+          def file
+            options[:file]
           end
         end
       end

data/lib/3scale_toolbox/commands/plans_command/import/import_backend_metrics_step.rb

@@ -0,0 +1,37 @@
+module ThreeScaleToolbox
+  module Commands
+    module PlansCommand
+      module Import
+        class ImportBackendMetricsStep
+          include Step
+          ##
+          # Writes Plan metrics and methods
+          def call
+            resource_backend_metrics.each(&method(:create_metric))
+            resource_backend_methods.each(&method(:create_method))
+          end
+
+          private
+
+          def create_metric(metric_attrs)
+            backend = find_backend(metric_attrs.fetch('backend_system_name'))
+
+            unless backend.metrics.any? { |m| m.system_name == metric_attrs.fetch('system_name') }
+              Entities::BackendMetric.create(backend: backend, attrs: metric_attrs)
+              puts "Created backend metric: #{metric_attrs.fetch('system_name')}; backend: #{backend.system_name}"
+            end
+          end
+
+          def create_method(method_attrs)
+            backend = find_backend(method_attrs.fetch('backend_system_name'))
+
+            unless backend.methods.any? { |m| m.system_name == method_attrs.fetch('system_name') }
+              Entities::BackendMethod.create(backend: backend, attrs: method_attrs)
+              puts "Created backend method: #{method_attrs.fetch('system_name')}; backend: #{backend.system_name}"
+            end
+          end
+        end
+      end
+    end
+  end
+end

data/lib/3scale_toolbox/commands/plans_command/import/import_plan_limits_step.rb

@@ -2,7 +2,7 @@ module ThreeScaleToolbox
   module Commands
     module PlansCommand
       module Import
-        class ImportMetricLimitsStep
+        class ImportLimitsStep
           include Step
           ##
           # Writes Plan limits
@@ -26,9 +26,18 @@ module ThreeScaleToolbox
 
           def resource_limits_processed
             resource_limits.map do |limit|
-              metric = find_metric_by_system_name(limit.delete('metric_system_name'))
+              metric_system_name = limit.delete('metric_system_name')
+              backend_system_name = limit.delete('metric_backend_system_name')
+              metric_owner = if backend_system_name.nil?
+                               service
+                             else
+                               find_backend(backend_system_name)
+                             end
+              metric = metric_owner.find_metric_or_method(metric_system_name)
               # this ImportMetricLimitsStep step is assuming all metrics/methods have been created
               # in previous step, so finding metric should always succeed.
+              raise ThreeScaleToolbox::Error, "metric [#{metric_system_name}, #{backend_system_name}] not found" if metric.nil?
+
               limit.merge('metric_id' => metric.id)
             end
           end

data/lib/3scale_toolbox/commands/plans_command/import/import_plan_metrics_step.rb

@@ -14,13 +14,13 @@ module ThreeScaleToolbox
           private
 
           def missing_metrics
-            ThreeScaleToolbox::Helper.array_difference(resource_metrics, service.metrics) do |a, b|
+            ThreeScaleToolbox::Helper.array_difference(resource_product_metrics, service.metrics) do |a, b|
               a['system_name'] == b.system_name
             end
           end
 
           def missing_methods
-            ThreeScaleToolbox::Helper.array_difference(resource_methods, service.methods) do |a, b|
+            ThreeScaleToolbox::Helper.array_difference(resource_product_methods, service.methods) do |a, b|
               a['system_name'] == b.system_name
             end
           end

data/lib/3scale_toolbox/commands/plans_command/import/import_plan_pricing_rules_step.rb

@@ -26,7 +26,18 @@ module ThreeScaleToolbox
 
           def resource_pr_processed
             resource_pricing_rules.map do |pr|
-              metric = find_metric_by_system_name(pr.delete('metric_system_name'))
+              metric_system_name = pr.delete('metric_system_name')
+              backend_system_name = pr.delete('metric_backend_system_name')
+              metric_owner = if backend_system_name.nil?
+                               service
+                             else
+                               find_backend(backend_system_name)
+                             end
+              metric = metric_owner.find_metric_or_method(metric_system_name)
+              # this ImportMetricLimitsStep step is assuming all metrics/methods have been created
+              # in previous step, so finding metric should always succeed.
+              raise ThreeScaleToolbox::Error, "metric [#{metric_system_name}, #{backend_system_name}] not found" if metric.nil?
+
               pr.merge('metric_id' => metric.id,
                        'cost_per_unit' => pr.fetch('cost_per_unit').to_f)
             end

data/lib/3scale_toolbox/commands/plans_command/import/step.rb

@@ -53,6 +53,22 @@ module ThreeScaleToolbox
             artifacts_resource['methods'] || []
           end
 
+          def resource_product_metrics
+            resource_metrics.reject{ |m| m.has_key? 'backend_system_name' }
+          end
+
+          def resource_product_methods
+            resource_methods.reject{ |mth| mth.has_key? 'backend_system_name' }
+          end
+
+          def resource_backend_metrics
+            resource_metrics.select{ |m| m.has_key? 'backend_system_name' }
+          end
+
+          def resource_backend_methods
+            resource_methods.select{ |mth| mth.has_key? 'backend_system_name' }
+          end
+
           def resource_limits
             artifacts_resource['limits'] || []
           end
@@ -65,10 +81,6 @@ module ThreeScaleToolbox
             artifacts_resource['plan_features'] || []
           end
 
-          def service_metrics_and_methods
-            service.metrics + service.methods
-          end
-
           def service_features
             context[:service_features] ||= service.features
           end
@@ -82,10 +94,6 @@ module ThreeScaleToolbox
             service_features.find { |feature| feature['system_name'] == system_name }
           end
 
-          def find_metric_by_system_name(system_name)
-            service_metrics_and_methods.find { |metric| metric.system_name == system_name }
-          end
-
           private
 
           def find_service
@@ -100,6 +108,13 @@ module ThreeScaleToolbox
               raise ThreeScaleToolbox::Error, "Application plan #{plan_system_name} does not exist" if p.nil?
             end
           end
+
+          def find_backend(backend_system_name)
+            Entities::Backend.find_by_system_name(remote: threescale_client,
+                                                  system_name: backend_system_name).tap do |backend|
+              raise ThreeScaleToolbox::Error, "Backend #{backend_system_name} does not exist" if backend.nil?
+            end
+          end
         end
       end
     end

data/lib/3scale_toolbox/commands/plans_command/import/validate_plan_step.rb

@@ -0,0 +1,126 @@
+module ThreeScaleToolbox
+  module Commands
+    module PlansCommand
+      module Import
+        class ValidatePlanStep
+          include Step
+          ##
+          # Creates if it does not exist, updates otherwise
+          def call
+            validate_product_metric_method_uniqueness!
+
+            validate_backend_metric_method_uniqueness!
+
+            validate_product_backend_usage_references!
+
+            validate_limit_backend_references!
+
+            validate_limit_product_references!
+
+            validate_pricingrule_backend_references!
+
+            validate_pricingrule_product_references!
+          end
+
+          private
+
+          def validate_product_metric_method_uniqueness!
+            system_name_list = (resource_product_metrics + resource_product_methods).map do |m|
+              m.fetch('system_name')
+            end
+            if system_name_list.length != system_name_list.uniq.length
+              raise ThreeScaleToolbox::Error, "Invalid content. " \
+                "Product metrics and method system names must be unique."
+            end
+          end
+
+          def validate_backend_metric_method_uniqueness!
+            metric_list = resource_backend_metrics + resource_backend_methods
+            backend_list = metric_list.map { |m| m.fetch('backend_system_name') }.uniq
+            backend_list.each do |backend_system_name|
+              backend_metric_list = metric_list.select do |m|
+                m.fetch('backend_system_name') == backend_system_name
+              end.map { |m| m.fetch('system_name') }
+
+              if backend_metric_list.length != backend_metric_list.uniq.length
+                raise ThreeScaleToolbox::Error, "Invalid content. " \
+                  "Backend #{backend_system_name} contains metrics and method system names that are not unique"
+              end
+            end
+          end
+
+          def validate_product_backend_usage_references!
+            metric_list = resource_backend_metrics + resource_backend_methods
+            backend_list = metric_list.map { |m| m.fetch('backend_system_name') }.uniq
+            backend_usages_list = service.backend_usage_list.map(&:backend).map(&:system_name)
+
+            backend_list.each do |backend_system_name|
+              unless backend_usages_list.include?(backend_system_name)
+                raise ThreeScaleToolbox::Error, "Invalid content. " \
+                  "Backend usage reference to backend #{backend_system_name} has not been found"
+              end
+            end
+          end
+
+          def validate_limit_backend_references!
+            metric_list = resource_backend_metrics + resource_backend_methods
+            limits_with_backend_ref = resource_limits.select{ |limit| limit.has_key? 'metric_backend_system_name'}
+            limits_with_backend_ref.each do |limit|
+              none = metric_list.none? do |m|
+                m.fetch('system_name') == limit.fetch('metric_system_name') &&
+                m.fetch('backend_system_name') == limit.fetch('metric_backend_system_name')
+              end
+
+              if none
+                raise ThreeScaleToolbox::Error, "Invalid content. " \
+                  "Limit with backend metric [#{limit.fetch('metric_system_name')}, #{limit.fetch('metric_backend_system_name')}] " \
+                  "has not been found in metric or method list"
+              end
+            end
+          end
+
+          def validate_limit_product_references!
+            metric_list = resource_product_metrics + resource_product_methods
+            limits = resource_limits.reject { |limit| limit.has_key? 'metric_backend_system_name'}
+            limits.each do |limit|
+              if metric_list.none? { |m| m.fetch('system_name') == limit.fetch('metric_system_name') }
+                raise ThreeScaleToolbox::Error, "Invalid content. " \
+                  "Limit with product metric [#{limit.fetch('metric_system_name')}] " \
+                  "has not been found in metric or method list"
+              end
+            end
+          end
+
+          def validate_pricingrule_backend_references!
+            metric_list = resource_backend_metrics + resource_backend_methods
+            pr_with_backend_ref = resource_pricing_rules.select{ |pr| pr.has_key? 'metric_backend_system_name'}
+            pr_with_backend_ref.each do |pr|
+              none = metric_list.none? do |m|
+                m.fetch('system_name') == pr.fetch('metric_system_name') &&
+                m.fetch('backend_system_name') == pr.fetch('metric_backend_system_name')
+              end
+
+              if none
+                raise ThreeScaleToolbox::Error, "Invalid content. " \
+                  "PricingRule with backend metric [#{pr.fetch('metric_system_name')}, #{pr.fetch('metric_backend_system_name')}] " \
+                  "has not been found in metric or method list"
+              end
+            end
+          end
+
+          def validate_pricingrule_product_references!
+            metric_list = resource_product_metrics + resource_product_methods
+            prs = resource_pricing_rules.reject { |pr| pr.has_key? 'metric_backend_system_name'}
+            prs.each do |pr|
+              if metric_list.none? { |m| m.fetch('system_name') == pr.fetch('metric_system_name') }
+                raise ThreeScaleToolbox::Error, "Invalid content. " \
+                  "PricingRule with product metric [#{pr.fetch('metric_system_name')}] " \
+                  "has not been found in metric or method list"
+              end
+            end
+          end
+        end
+      end
+    end
+  end
+end

data/lib/3scale_toolbox/commands/plans_command/import_command.rb

@@ -1,9 +1,11 @@
 require '3scale_toolbox/commands/plans_command/import/step'
+require '3scale_toolbox/commands/plans_command/import/validate_plan_step'
 require '3scale_toolbox/commands/plans_command/import/create_or_update_app_plan_step'
 require '3scale_toolbox/commands/plans_command/import/import_plan_features_step'
 require '3scale_toolbox/commands/plans_command/import/import_plan_metrics_step'
 require '3scale_toolbox/commands/plans_command/import/import_plan_limits_step'
 require '3scale_toolbox/commands/plans_command/import/import_plan_pricing_rules_step'
+require '3scale_toolbox/commands/plans_command/import/import_backend_metrics_step'
 
 module ThreeScaleToolbox
   module Commands
@@ -31,9 +33,11 @@ module ThreeScaleToolbox
 
           def run
             tasks = []
+            tasks << ValidatePlanStep.new(context)
             tasks << CreateOrUpdateAppPlanStep.new(context)
             tasks << ImportMetricsStep.new(context)
-            tasks << ImportMetricLimitsStep.new(context)
+            tasks << ImportBackendMetricsStep.new(context)
+            tasks << ImportLimitsStep.new(context)
             tasks << ImportPricingRulesStep.new(context)
             tasks << ImportPlanFeaturesStep.new(context)
 

data/lib/3scale_toolbox/commands/product_command/copy_command/copy_backends_task.rb

@@ -27,6 +27,7 @@ module ThreeScaleToolbox
             # First metrics as methods need 'hits' metric in target backend
             tasks << Commands::BackendCommand::CopyCommand::CopyMetricsTask.new(backend_context)
             tasks << Commands::BackendCommand::CopyCommand::CopyMethodsTask.new(backend_context)
+            tasks << Commands::BackendCommand::CopyCommand::DeleteMappingRulesTask.new(backend_context)
             tasks << Commands::BackendCommand::CopyCommand::CopyMappingRulesTask.new(backend_context)
             tasks.each(&:call)
 
@@ -72,12 +73,17 @@ module ThreeScaleToolbox
             context.fetch(:logger)
           end
 
+          def delete_mapping_rules
+            context.fetch(:delete_mapping_rules, false)
+          end
+
           def create_backend_context(source_backend)
             {
               source_remote: source_remote,
               target_remote: target_remote,
               source_backend: source_backend,
               source_backend_ref: source_backend.id,
+              delete_mapping_rules: delete_mapping_rules,
               logger: logger
             }
           end