Remove google analytics

.rubocop_todo.yml

@@ -462,12 +462,6 @@ RSpec/IteratedExpectation:
     - 'spec/unit/executor_spec.rb'
     - 'spec/unit/fiber_executor_spec.rb'
 
-# Offense count: 1
-# This cop supports safe autocorrection (--autocorrect).
-RSpec/LeadingSubject:
-  Exclude:
-    - 'spec/unit/analytics_spec.rb'
-
 # Offense count: 20
 # This cop supports safe autocorrection (--autocorrect).
 RSpec/MatchArray:
@@ -500,7 +494,6 @@ RSpec/MultipleDescribes:
   Exclude:
     - 'spec/integration/cli/cli_spec.rb'
     - 'spec/integration/logging_spec.rb'
-    - 'spec/unit/analytics_spec.rb'
     - 'spec/unit/config/options_spec.rb'
 
 # Offense count: 648
@@ -514,7 +507,6 @@ RSpec/MultipleMemoizedHelpers:
 RSpec/NamedSubject:
   Exclude:
     - 'bolt-modules/prompt/spec/functions/prompt_spec.rb'
-    - 'spec/unit/analytics_spec.rb'
     - 'spec/unit/pal/yaml_plan/evaluator_spec.rb'
     - 'spec/unit/plan_creator_spec.rb'
     - 'spec/unit/plugin/env_var_spec.rb'
@@ -535,7 +527,6 @@ RSpec/NoExpectationExample:
   Exclude:
     - 'spec/integration/parallel_spec.rb'
     - 'spec/integration/private_plan_spec.rb'
-    - 'spec/unit/analytics_spec.rb'
     - 'spec/unit/pal/yaml_plan/evaluator_spec.rb'
 
 # Offense count: 11
@@ -654,7 +645,6 @@ RSpec/SubjectDeclaration:
 # Offense count: 20
 RSpec/SubjectStub:
   Exclude:
-    - 'spec/unit/analytics_spec.rb'
     - 'spec/unit/transport/ssh/connection_spec.rb'
 
 # Offense count: 1

Gemfile

@@ -12,9 +12,6 @@ def location_for(place, fake_version = nil)
   end
 end
 
-# Disable analytics when running in development
-ENV['BOLT_DISABLE_ANALYTICS'] = 'true'
-
 # Disable warning that Bolt may be installed as a gem
 ENV['BOLT_GEM'] = 'true'
 

acceptance/setup/common/pre-suite/050_build_bolt_inventory.rb

@@ -37,6 +37,4 @@
 
   on bolt, "mkdir -p #{default_boltdir}"
   create_remote_file(bolt, "#{default_boltdir}/inventory.yaml", inventory.to_yaml)
-
-  create_remote_file(bolt, "#{default_boltdir}/analytics.yaml", { 'disabled' => true }.to_yaml)
 end

bolt-modules/boltlib/lib/puppet/functions/add_facts.rb

@@ -24,9 +24,6 @@ def add_facts(target, facts)
     end
 
     inventory = Puppet.lookup(:bolt_inventory)
-    executor = Puppet.lookup(:bolt_executor)
-    # Send Analytics Report
-    executor.report_function_call(self.class.name)
 
     inventory.add_facts(target, facts)
   end

bolt-modules/boltlib/lib/puppet/functions/add_to_group.rb

@@ -29,9 +29,6 @@ def add_to_group(targets, group)
     end
 
     inventory = Puppet.lookup(:bolt_inventory)
-    executor = Puppet.lookup(:bolt_executor)
-    # Send Analytics Report
-    executor.report_function_call(self.class.name)
 
     inventory.add_to_group(inventory.get_targets(targets), group)
   end

bolt-modules/boltlib/lib/puppet/functions/apply_prep.rb

@@ -36,8 +36,6 @@ def apply_prep(target_spec, options = {})
     options = options.slice(*%w[_catch_errors _required_modules _run_as])
     targets = inventory.get_targets(target_spec)
 
-    executor.report_function_call(self.class.name)
-
     executor.log_action('install puppet and gather facts', targets) do
       executor.without_default_logging do
         install_results = install_agents(targets, options)

bolt-modules/boltlib/lib/puppet/functions/background.rb

@@ -29,7 +29,6 @@ def background(scope, name = nil, &block)
     end
 
     executor = Puppet.lookup(:bolt_executor)
-    executor.report_function_call(self.class.name)
 
     plan_id = executor.get_current_plan_id(fiber: Fiber.current)
     executor.create_future(scope: scope, name: name, plan_id: plan_id) do |newscope|

bolt-modules/boltlib/lib/puppet/functions/catch_errors.rb

@@ -36,10 +36,6 @@ def catch_errors(error_types = nil)
                               action: self.class.name)
     end
 
-    executor = Puppet.lookup(:bolt_executor)
-    # Send Analytics Report
-    executor.report_function_call(self.class.name)
-
     begin
       yield
     rescue Puppet::PreformattedError => e

bolt-modules/boltlib/lib/puppet/functions/download_file.rb

@@ -103,9 +103,6 @@ def download_file_with_description(source, destination, targets, description = n
       FileUtils.rm_r(Dir.glob(destination + '*'), secure: true)
     end
 
-    # Send Analytics Report
-    executor.report_function_call(self.class.name)
-
     # Ensure that that given targets are all Target instances
     targets = inventory.get_targets(targets)
     if targets.empty?

bolt-modules/boltlib/lib/puppet/functions/facts.rb

@@ -21,10 +21,6 @@
 
   def facts(target)
     inventory = Puppet.lookup(:bolt_inventory)
-    # Bolt executor not expected when invoked from apply block
-    executor = Puppet.lookup(:bolt_executor) { nil }
-    # Send Analytics Report
-    executor&.report_function_call(self.class.name)
 
     inventory.facts(target)
   end

bolt-modules/boltlib/lib/puppet/functions/fail_plan.rb

@@ -39,10 +39,6 @@ def from_args(msg, kind = nil, details = nil, issue_code = nil)
         .from_issue_and_stack(Bolt::PAL::Issues::PLAN_OPERATION_NOT_SUPPORTED_WHEN_COMPILING, action: 'fail_plan')
     end
 
-    executor = Puppet.lookup(:bolt_executor)
-    # Send Analytics Report
-    executor.report_function_call(self.class.name)
-
     raise Bolt::PlanFailure.new(msg, kind || 'bolt/plan-failure', details, issue_code)
   end
 

bolt-modules/boltlib/lib/puppet/functions/get_resources.rb

@@ -59,9 +59,6 @@ def get_resources(target_spec, resources)
       end
     end
 
-    # Send Analytics Report
-    executor.report_function_call(self.class.name)
-
     targets = inventory.get_targets(target_spec)
 
     executor.log_action('gather resources', targets) do

bolt-modules/boltlib/lib/puppet/functions/get_target.rb

@@ -19,10 +19,6 @@
 
   def get_target(name)
     inventory = Puppet.lookup(:bolt_inventory)
-    # Bolt executor not expected when invoked from apply block
-    executor = Puppet.lookup(:bolt_executor) { nil }
-    # Send Analytics Report
-    executor&.report_function_call(self.class.name)
 
     unless inventory.version > 1
       raise Puppet::ParseErrorWithIssue

bolt-modules/boltlib/lib/puppet/functions/get_targets.rb

@@ -25,10 +25,6 @@
 
   def get_targets(names)
     inventory = Puppet.lookup(:bolt_inventory)
-    # Bolt executor not expected when invoked from apply block
-    executor = Puppet.lookup(:bolt_executor) { nil }
-    # Send Analytics Report
-    executor&.report_function_call(self.class.name)
 
     inventory.get_targets(names)
   end

bolt-modules/boltlib/lib/puppet/functions/parallelize.rb

@@ -31,7 +31,6 @@ def parallelize(scope, data, &block)
     end
 
     executor = Puppet.lookup(:bolt_executor)
-    executor.report_function_call(self.class.name)
 
     futures = data.map do |object|
       # We're going to immediately wait for these futures, *and* don't want

bolt-modules/boltlib/lib/puppet/functions/puppetdb_command.rb

@@ -78,9 +78,6 @@ def puppetdb_command_with_instance(command, version, payload, instance)
       )
     end
 
-    # Send analytics report.
-    Puppet.lookup(:bolt_executor).report_function_call(self.class.name)
-
     puppetdb_client = Puppet.lookup(:bolt_pdb_client)
 
     # Error if the PDB client does not implement :send_command

bolt-modules/boltlib/lib/puppet/functions/puppetdb_fact.rb

@@ -37,10 +37,6 @@ def puppetdb_fact(certnames)
 
   def puppetdb_fact_with_instance(certnames, instance)
     puppetdb_client = Puppet.lookup(:bolt_pdb_client)
-    # Bolt executor not expected when invoked from apply block
-    executor = Puppet.lookup(:bolt_executor) { nil }
-    # Send Analytics Report
-    executor&.report_function_call(self.class.name)
 
     puppetdb_client.facts_for_node(certnames, instance)
   end

bolt-modules/boltlib/lib/puppet/functions/puppetdb_query.rb

@@ -38,10 +38,6 @@ def make_query(query)
 
   def make_query_with_instance(query, instance)
     puppetdb_client = Puppet.lookup(:bolt_pdb_client)
-    # Bolt executor not expected when invoked from apply block
-    executor = Puppet.lookup(:bolt_executor) { nil }
-    # Send Analytics Report
-    executor&.report_function_call(self.class.name)
 
     puppetdb_client.make_query(query, nil, instance)
   end

bolt-modules/boltlib/lib/puppet/functions/remove_from_group.rb

@@ -31,9 +31,6 @@ def remove_from_group(target, group)
     end
 
     inventory = Puppet.lookup(:bolt_inventory)
-    executor = Puppet.lookup(:bolt_executor)
-    # Send Analytics Report
-    executor.report_function_call(self.class.name)
 
     inventory.remove_from_group(inventory.get_targets(target), group)
   end

bolt-modules/boltlib/lib/puppet/functions/resolve_references.rb

@@ -32,10 +32,6 @@ def resolve_references(references)
         )
     end
 
-    executor = Puppet.lookup(:bolt_executor)
-    # Send Analytics Report
-    executor.report_function_call(self.class.name)
-
     plugins = Puppet.lookup(:bolt_inventory).plugins
     plugins.resolve_references(references)
   end

bolt-modules/boltlib/lib/puppet/functions/resource.rb

@@ -40,9 +40,6 @@
 
   def resource(target, type, title)
     inventory = Puppet.lookup(:bolt_inventory)
-    executor  = Puppet.lookup(:bolt_executor) { nil }
-    # Send Analytics Report
-    executor&.report_function_call(self.class.name)
 
     inventory.resource(target, type, title)
   end

bolt-modules/boltlib/lib/puppet/functions/run_command.rb

@@ -77,9 +77,6 @@ def run_command_with_description(command, targets, description = nil, options =
     executor = Puppet.lookup(:bolt_executor)
     inventory = Puppet.lookup(:bolt_inventory)
 
-    # Send Analytics Report
-    executor.report_function_call(self.class.name)
-
     # Ensure that given targets are all Target instances
     targets = inventory.get_targets(targets)
 

bolt-modules/boltlib/lib/puppet/functions/run_container.rb

@@ -36,9 +36,7 @@ def run_container(image, options = {})
         .from_issue_and_stack(Bolt::PAL::Issues::PLAN_OPERATION_NOT_SUPPORTED_WHEN_COMPILING, action: 'run_container')
     end
 
-    # Send Analytics Report
     executor = Puppet.lookup(:bolt_executor)
-    executor.report_function_call(self.class.name)
 
     options = options.transform_keys { |k| k.sub(/^_/, '').to_sym }
     validate_options(options)

bolt-modules/boltlib/lib/puppet/functions/run_plan.rb

@@ -67,16 +67,6 @@ def run_inner_plan(scope, plan_name, targets, args = {})
     options, params = args.partition { |k, _v| k.start_with?('_') }.map(&:to_h)
     options = options.transform_keys { |k| k.sub(/^_/, '').to_sym }
 
-    # Bolt calls this function internally to trigger plans from the CLI. We
-    # don't want to count those invocations.
-    unless options[:bolt_api_call]
-      # Send Analytics Report
-      executor.report_function_call(self.class.name)
-    end
-
-    # Send Analytics Report for bundled content, this should capture plans run from both CLI and Plans
-    executor.report_bundled_content('Plan', plan_name)
-
     loaders = closure_scope.compiler.loaders
     # The perspective of the environment is wanted here (for now) to not have to
     # require modules to have dependencies defined in meta data.
@@ -93,9 +83,6 @@ def run_inner_plan(scope, plan_name, targets, args = {})
     end
 
     closure = func.class.dispatcher.dispatchers[0]
-    if closure.model.is_a?(Bolt::PAL::YamlPlan)
-      executor.report_yaml_plan(closure.model.body)
-    end
 
     # If a TargetSpec parameter is passed, ensure it is in inventory
     inventory = Puppet.lookup(:bolt_inventory)

bolt-modules/boltlib/lib/puppet/functions/run_script.rb

@@ -105,9 +105,6 @@ def run_script_with_description(scope, script, targets, description = nil, optio
     executor = Puppet.lookup(:bolt_executor)
     inventory = Puppet.lookup(:bolt_inventory)
 
-    # Send Analytics Report
-    executor.report_function_call(self.class.name)
-
     # Find the file path if it exists, otherwise return nil
     found = Bolt::Util.find_file_from_scope(script, scope)
     unless found && Puppet::FileSystem.exist?(found)
@@ -120,7 +117,6 @@ def run_script_with_description(scope, script, targets, description = nil, optio
         Puppet::Pops::Issues::NOT_A_FILE, file: script
       )
     end
-    executor.report_file_source(self.class.name, script)
     # Ensure that given targets are all Target instances)
     targets = inventory.get_targets(targets)
 

bolt-modules/boltlib/lib/puppet/functions/run_task.rb

@@ -62,17 +62,6 @@ def run_task_with_description(task_name, targets, description, args = {})
     executor = Puppet.lookup(:bolt_executor)
     inventory = Puppet.lookup(:bolt_inventory)
 
-    # Bolt calls this function internally to trigger tasks from the CLI. We
-    # don't want to count those invocations.
-    unless options[:bolt_api_call]
-      # Send Analytics Report
-      executor.report_function_call(self.class.name)
-    end
-
-    # Report Analytics for bundled content, this should capture tasks run from
-    # both CLI and Plans.
-    executor.report_bundled_content('Task', task_name)
-
     # Ensure that given targets are all Target instances.
     targets = inventory.get_targets(targets)
 
@@ -138,9 +127,6 @@ def run_task_with_description(task_name, targets, description, args = {})
       end
     end
 
-    # Report whether the task was run in noop mode.
-    executor.report_noop_mode(executor.noop || options[:noop])
-
     file_line = Puppet::Pops::PuppetStack.top_of_stack
     result = if executor.in_parallel?
                executor.run_in_thread do

bolt-modules/boltlib/lib/puppet/functions/run_task_with.rb

@@ -79,10 +79,6 @@ def run_task_with_with_description(task_name, targets, description, options = {}
     inventory = Puppet.lookup(:bolt_inventory)
     error_set = []
 
-    # Report to analytics
-    executor.report_function_call(self.class.name)
-    executor.report_bundled_content('Task', task_name)
-
     # Keep valid metaparameters, discarding everything else
     options = options.select { |k, _v| k.start_with?('_') }
                      .transform_keys { |k| k.sub(/^_/, '').to_sym }
@@ -182,9 +178,6 @@ def run_task_with_with_description(task_name, targets, description, options = {}
       end
     end
 
-    # Report whether the task was run in noop mode.
-    executor.report_noop_mode(executor.noop || options[:noop])
-
     # Combine the results from the task run with any failing results that were
     # generated earlier when creating the target mapping
     file_line = Puppet::Pops::PuppetStack.top_of_stack

bolt-modules/boltlib/lib/puppet/functions/set_config.rb

@@ -32,9 +32,6 @@ def set_config(target, key_or_key_path, value = true)
     end
 
     inventory = Puppet.lookup(:bolt_inventory)
-    executor = Puppet.lookup(:bolt_executor)
-    # Send Analytics Report
-    executor.report_function_call(self.class.name)
 
     unless inventory.version > 1
       raise Puppet::ParseErrorWithIssue

bolt-modules/boltlib/lib/puppet/functions/set_feature.rb

@@ -32,9 +32,6 @@ def set_feature(target, feature, value = true)
     end
 
     inventory = Puppet.lookup(:bolt_inventory)
-    executor = Puppet.lookup(:bolt_executor)
-    # Send Analytics Report
-    executor.report_function_call(self.class.name)
 
     inventory.set_feature(target, feature, value)
 

bolt-modules/boltlib/lib/puppet/functions/set_resources.rb

@@ -94,8 +94,6 @@ def set_resources(target, resources)
         )
     end
 
-    # Send Analytics Report
-    Puppet.lookup(:bolt_executor).report_function_call(self.class.name)
     inventory = Puppet.lookup(:bolt_inventory)
 
     resources.uniq.map do |resource|

bolt-modules/boltlib/lib/puppet/functions/set_var.rb

@@ -26,9 +26,6 @@ def set_var(target, key, value)
     end
 
     inventory = Puppet.lookup(:bolt_inventory)
-    executor = Puppet.lookup(:bolt_executor)
-    # Send Analytics Report
-    executor.report_function_call(self.class.name)
 
     var_hash = { key => value }
     inventory.set_var(target, var_hash)