Skip to content

Drop license_management artifact [RUN ALL RSPEC] [RUN AS-IF-FOSS]

Tetiana Chupryna requested to merge 299114-drop-flag into master

What does this MR do?

Drop drop_license_management_artifact feature flag as well as license_management artifact itself and it's usage.

We need to drop license_management artifact type as well because tests are highly coupled on the list of available types. Also, that's a good thing for the codebase as right now it's quite confusing to see license_management in the list of supported artifacts, where in fact, we don't support it since 13.0

We dropped support for license_management in 13.0 and since then we don't create such artifacts and don't store them. We also finished migration of existing license_management artifacts to license_scanning so we don't have them in the database.

So this is a safe step to remove license_management from the list of artifact types.

This MR can be separated into smaller ones as this is a breaking change.

Screenshots (strongly suggested)

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

Does this MR contain changes to processing or storing of credentials or tokens, authorization and authentication methods or other items described in the security review guidelines? If not, then delete this Security section.

  • [-] 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

Related to #299114 (closed)

Edited by Tetiana Chupryna

Merge request reports