Update feature categories for 2020-12-31

https://gitlab.com/gitlab-com/www-gitlab-com/-/merge_requests/70219
reorganised some of these categories. Most of the items that were under
`collection` can go to `usage_ping`, but the historical usage export
probably belongs under `provision` (as it's to do with license usage),
and the survey response are - tangentially - under `navigation`.

app/controllers/admin/application_settings_controller.rb

@@ -31,7 +31,7 @@ class Admin::ApplicationSettingsController < Admin::ApplicationController
 
   feature_category :source_code_management, [:repository, :clear_repository_check_states]
   feature_category :continuous_integration, [:ci_cd, :reset_registration_token]
-  feature_category :collection, [:usage_data]
+  feature_category :usage_ping, [:usage_data]
   feature_category :integrations, [:integrations]
   feature_category :pages, [:lets_encrypt_terms_of_service]
 

app/controllers/projects/usage_ping_controller.rb

@@ -3,7 +3,7 @@
 class Projects::UsagePingController < Projects::ApplicationController
   before_action :authenticate_user!
 
-  feature_category :collection
+  feature_category :usage_ping
 
   def web_ide_clientside_preview
     return render_404 unless Gitlab::CurrentSettings.web_ide_clientside_preview_enabled?

app/workers/all_queues.yml

@@ -188,7 +188,7 @@
   :idempotent:
   :tags: []
 - :name: cronjob:gitlab_usage_ping
-  :feature_category: :collection
+  :feature_category: :usage_ping
   :has_external_dependencies:
   :urgency: :low
   :resource_boundary: :unknown

app/workers/gitlab_usage_ping_worker.rb

@@ -8,7 +8,7 @@ class GitlabUsagePingWorker # rubocop:disable Scalability/IdempotentWorker
   include CronjobQueue # rubocop:disable Scalability/CronWorkerContext
   include Gitlab::ExclusiveLeaseHelpers
 
-  feature_category :collection
+  feature_category :usage_ping
   sidekiq_options retry: 3, dead: false
   sidekiq_retry_in { |count| (count + 1) * 8.hours.to_i }
 

config/feature_categories.yml

@@ -9,7 +9,6 @@
 ---
 - accessibility_testing
 - advanced_deployments
-- analysis
 - api
 - attack_emulation
 - audit_events
@@ -25,7 +24,6 @@
 - code_quality
 - code_review
 - code_testing
-- collection
 - compliance_management
 - container_host_security
 - container_network_security
@@ -83,6 +81,7 @@
 - pages
 - pipeline_authoring
 - planning_analytics
+- privacy_control_center
 - product_analytics
 - projects
 - provision
@@ -112,6 +111,7 @@
 - time_tracking
 - tracing
 - usability_testing
+- usage_ping
 - users
 - value_stream_management
 - vulnerability_database

ee/app/controllers/admin/licenses/usage_exports_controller.rb

@@ -7,7 +7,7 @@ module Admin
 
       before_action :require_license, only: :show
 
-      feature_category :collection
+      feature_category :provision
 
       def show
         historical_data = HistoricalData.in_license_term(license)

ee/app/controllers/survey_responses_controller.rb

@@ -7,7 +7,7 @@ class SurveyResponsesController < ApplicationController
 
   skip_before_action :authenticate_user!
 
-  feature_category :collection
+  feature_category :navigation
 
   def index
     track_response if Gitlab.com?

lib/api/usage_data.rb

@@ -4,7 +4,7 @@ module API
   class UsageData < ::API::Base
     before { authenticate! }
 
-    feature_category :collection
+    feature_category :usage_ping
 
     namespace 'usage_data' do
       before do