Skip to content

Migrate group badges when using Bulk Import

George Koltsov requested to merge georgekoltsov/bulk-import-badges into master

What does this MR do?

This MR adds group badges to the list of migrated relations when using Group Migration tool (https://docs.gitlab.com/ee/user/group/import/). This MR also introduced a new data extractor that uses REST API. Till this point Bulk Import used GraphQL extractor that uses GraphQL API to fetch all the data. Badges pipeline uses REST API for several reasons:

  1. Badges type is not available in GraphQL which would require changing GraphQL API first in order to deliver this issue
  2. Badges are quite simple data type and we decided to just implement it in REST
  3. REST extractor is going to be useful in the future once we add more data types that need to be fetched using REST

More info on BulkImport internals can be found here https://gitlab.com/gitlab-org/gitlab/-/blob/master/doc/development/bulk_import.md

Mentions #292431 (closed)

Screenshots (strongly suggested)

badges

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 George Koltsov

Merge request reports

Loading