Skip to content

Remove container registry migration phase 1 feature flags

João Pereira requested to merge 335703-cleanup into master

What does this MR do and why?

Reverts !63907 (merged), !67852 (merged), and !68357 (merged).

This effectively removes all feature flags (container_registry_migration_phase1, container_registry_migration_phase1_allow, and container_registry_migration_phase1_deny) and the source code previously introduced to support the GitLab.com container registry migration Phase 1. Phase 1 has been completed a couple of months ago and therefore these flags and source code should now be removed.

For context, #335260 (closed) provides a detailed explanation of how/why this code was implemented.

Please note that these feature flags and source code did not represent any GitLab Rails feature. These were only used as a means to communicate metadata to the container registry (using the JWT tokens as the communication channel). This is why I didn't add a changelog entry to this MR.

Rollout issues:

How to set up and validate locally

See !63907 (merged). With this MR, the code should behave as described in Current behavior.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Related to #335703 (closed)

Edited by David Fernandez

Merge request reports