Skip to content

Add project import usage pings

What does this MR do?

This MR adds usage ping data for Project Imports of the following importers:

  • GitLab Project
  • GitLab.com
  • GitHub
  • Gitea
  • Git (Repo by URL)
  • Manifest file
  • Bitbucket Cloud
  • Bitbucket Server

Mentions #235424 (closed)

Migration Output

Up

== 20200826092324 AddProjectsIndexOnImportTypeCreatorIdCreatedAt: migrating ===
-- transaction_open?()
   -> 0.0000s
-- index_exists?(:projects, [:import_type, :creator_id, :created_at], {:where=>"import_type IS NOT NULL", :name=>"index_imported_projects_on_import_type_creator_id_created_at", :algorithm=>:concurrently})
   -> 0.0150s
-- add_index(:projects, [:import_type, :creator_id, :created_at], {:where=>"import_type IS NOT NULL", :name=>"index_imported_projects_on_import_type_creator_id_created_at", :algorithm=>:concurrently})
   -> 0.0056s
== 20200826092324 AddProjectsIndexOnImportTypeCreatorIdCreatedAt: migrated (0.0210s) 

Down

== 20200826092324 AddProjectsIndexOnImportTypeCreatorIdCreatedAt: reverting ===
-- transaction_open?()
   -> 0.0000s
-- indexes(:projects)
   -> 0.0163s
-- remove_index(:projects, {:algorithm=>:concurrently, :name=>"index_imported_projects_on_import_type_creator_id_created_at"})
   -> 0.0022s
== 20200826092324 AddProjectsIndexOnImportTypeCreatorIdCreatedAt: reverted (0.0189s) 

Query Explain without timerange

https://explain.depesz.com/s/PlBj

Query with timerange

https://explain.depesz.com/s/nHgk

Screenshots

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

If this MR contains changes to processing or storing of credentials or tokens, authorization and authentication methods and other items described in the security review guidelines:

  • [-] Label as security and @ mention @gitlab-com/gl-security/appsec
  • [-] The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • [-] Security reports checked/validated by a reviewer from the AppSec team
Edited by Peter Leitzen

Merge request reports