Skip to content

Draft: Fix creating environment after renaming

What does this MR do?

THIS IS DRAFT: we still rely on deterministic slug, so this can't be merged yet =(

When an environment is renamed, the slug stays the same. This prevents a new environment with the old name from being creted because the slug is deterministically generated from the name and should be unique.

This MR adds a random suffix to all slugs fixing this issue.

Screenshots or Screencasts (strongly suggested)

How to setup and validate locally (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 #31268 (closed)

Edited by Vladimir Shushlin

Merge request reports